Review and drop old frame resize hack.

* frame.h (struct frame): Remove force_flush_display_p.
* dispnew.c (update_frame): Adjust user and don't call
flush_frame here.  The comment has said that there was an issues
with redisplaying fringes, but I don't see any differences with
and without this hack.  Hopefully we can continue without it.
* xdisp.c (clear_garbaged_frames): Adjust user and do not clear
current frame matrices twice if resized_p is set.

4 files changed, 14 insertions, 19 deletions

diff --git a/src/ChangeLog b/src/ChangeLog
index c6eb58071e8..4b688e43430 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,5 +1,16 @@
 2013-09-09  Dmitry Antipov  <dmantipov@yandex.ru>
 
+        Review and drop old frame resize hack.
+        * frame.h (struct frame): Remove force_flush_display_p.
+        * dispnew.c (update_frame): Adjust user and don't call
+        flush_frame here.  The comment has said that there was an issues
+        with redisplaying fringes, but I don't see any differences with
+        and without this hack.  Hopefully we can continue without it.
+        * xdisp.c (clear_garbaged_frames): Adjust user and do not clear
+        current frame matrices twice if resized_p is set.
+
+2013-09-09  Dmitry Antipov  <dmantipov@yandex.ru>
+
         Do not populate pure Xism x_sync to other ports.
         * frame.h (x_sync): Move under HAVE_X_WINDOWS.
         * frame.c (other_visible_frames) [HAVE_X_WINDOWS]: Use as such.
diff --git a/src/dispnew.c b/src/dispnew.c
index 74ecfa88bde..a42b3da5fc5 100644
--- a/src/dispnew.c
+++ b/src/dispnew.c
@@ -3061,16 +3061,6 @@ update_frame (struct frame *f, bool force_p, bool inhibit_hairy_id_p)
       /* Update windows.  */
       paused_p = update_window_tree (root_window, force_p);
       update_end (f);
-
-      /* This flush is a performance bottleneck under X,
-         and it doesn't seem to be necessary anyway (in general).
-         It is necessary when resizing the window with the mouse, or
-         at least the fringes are not redrawn in a timely manner.  ++kfs */
-      if (f->force_flush_display_p)
-        {
-          flush_frame (f);
-          f->force_flush_display_p = 0;
-        }
     }
   else
     {
diff --git a/src/frame.h b/src/frame.h
index baf8d6b93a9..87e218ee938 100644
--- a/src/frame.h
+++ b/src/frame.h
@@ -185,10 +185,6 @@ struct frame
      Clear the frame in clear_garbaged_frames if set.  */
   unsigned resized_p : 1;
 
-  /* Set to non-zero in when we want for force a flush_display in
-     update_frame, usually after resizing the frame.  */
-  unsigned force_flush_display_p : 1;
-
   /* Set to non-zero if the default face for the frame has been
      realized.  Reset to zero whenever the default face changes.
      Used to see the difference between a font change and face change.  */
diff --git a/src/xdisp.c b/src/xdisp.c
index 960ad18709b..df3fa366803 100644
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -10767,11 +10767,9 @@ clear_garbaged_frames (void)
           if (FRAME_VISIBLE_P (f) && FRAME_GARBAGED_P (f))
             {
               if (f->resized_p)
-                {
-                  redraw_frame (f);
-                  f->force_flush_display_p = 1;
-                }
-              clear_current_matrices (f);
+                redraw_frame (f);
+              else
+                clear_current_matrices (f);
               changed_count++;
               f->garbaged = 0;
               f->resized_p = 0;