From: Jim Blandy <jimb@redhat.com>
Date: Thu, 1 Jul 1993 04:04:32 +0000 (+0000)
Subject: * xterm.c (frame_highlight, frame_unhighlight): Don't test
X-Git-Tag: emacs-19.34~11908
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=b3e1e05cfea48f453113947b06d22924012e24ca;p=emacs.git

* xterm.c (frame_highlight, frame_unhighlight): Don't test
	Vx_no_window_manager.  BLOCK_INPUT around the border manipulation
	calls.
---

diff --git a/src/xterm.c b/src/xterm.c
index b24f2f11f5c..3e80d64253e 100644
--- a/src/xterm.c
+++ b/src/xterm.c
@@ -1245,9 +1245,14 @@ static void
 frame_highlight (frame)
      struct frame *frame;
 {
-  if (! EQ (Vx_no_window_manager, Qnil))
-    XSetWindowBorder (x_current_display, FRAME_X_WINDOW (frame),
-		      frame->display.x->border_pixel);
+  /* We used to only do this if Vx_no_window_manager was non-nil, but
+     the ICCCM (section 4.1.6) says that the window's border pixmap
+     and border pixel are window attributes which are "private to the
+     client", so we can always change it to whatever we want.  */
+  BLOCK_INPUT;
+  XSetWindowBorder (x_current_display, FRAME_X_WINDOW (frame),
+		    frame->display.x->border_pixel);
+  UNBLOCK_INPUT;
   x_display_cursor (frame, 1);
 }
 
@@ -1255,9 +1260,14 @@ static void
 frame_unhighlight (frame)
      struct frame *frame;
 {
-  if (! EQ (Vx_no_window_manager, Qnil))
-    XSetWindowBorderPixmap (x_current_display, FRAME_X_WINDOW (frame),
-			    frame->display.x->border_tile);
+  /* We used to only do this if Vx_no_window_manager was non-nil, but
+     the ICCCM (section 4.1.6) says that the window's border pixmap
+     and border pixel are window attributes which are "private to the
+     client", so we can always change it to whatever we want.  */
+  BLOCK_INPUT;
+  XSetWindowBorderPixmap (x_current_display, FRAME_X_WINDOW (frame),
+			  frame->display.x->border_tile);
+  UNBLOCK_INPUT;
   x_display_cursor (frame, 1);
 }
 #else /* ! defined (HAVE_X11) */