Don't set FRAME_PIXEL_HEIGHT and FRAME_PIXEL_WIDTH in update_various_frame_slots...

* widget.c (update_various_frame_slots): Don't set
FRAME_PIXEL_HEIGHT and FRAME_PIXEL_WIDTH here (Bug#16736).

diff --git a/src/ChangeLog b/src/ChangeLog
index 1db4d4ac3755c040d59a0ec589e70c025f0f4b6f..0fc12fce809aeff3305fdce1dec43a6218072597 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,8 @@
+2014-02-18  Martin Rudalics  <rudalics@gmx.at>
+
+       * widget.c (update_various_frame_slots): Don't set
+       FRAME_PIXEL_HEIGHT and FRAME_PIXEL_WIDTH here (Bug#16736).
+
 2014-02-18  Michael Albinus  <michael.albinus@gmx.de>
 
        * dbusbind.c (xd_close_bus): Apply proper check on busobj.

diff --git a/src/widget.c b/src/widget.c
index ecd42b35719b6555f9cef05eca5f06783fc46c99..838e775f9186127e6fac498bbb63ec37e9fd318e 100644 (file)
--- a/src/widget.c
+++ b/src/widget.c
@@ -599,8 +599,13 @@ update_various_frame_slots (EmacsFrame ew)
 {
   struct frame *f = ew->emacs_frame.frame;
   struct x_output *x = f->output_data.x;
+  /* Don't do that: It confuses the check in change_frame_size_1 whether
+     the pixel size of the frame changed due to a change of the internal
+     border width.  Bug#16736.  */
+#if 0
   FRAME_PIXEL_HEIGHT (f) = ew->core.height + x->menubar_height;
   FRAME_PIXEL_WIDTH (f) = ew->core.width;
+#endif
   f->internal_border_width = ew->emacs_frame.internal_border_width;
 
 }