Reland "Avoid layout/full-repaint on view height change if possible"

Source/core/frame/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 3162 matching lines @@
 
     ScrollView::frameRectsChanged();
 }
 
 void FrameView::setLayoutSizeInternal(const IntSize& size)
 {
     if (m_layoutSize == size)
         return;
 
     m_layoutSize = size;
-    contentsResized();
+
+    // Update scrollbars. Not calling this->contentsResized() to avoid setNeedsLayout.
+    ScrollView::contentsResized();
+
+    if (RenderView* renderView = this->renderView()) {
+        renderView->viewResized();
+        // If selfNeedsLayout, the next layout will do all the things required on layoutSize change.
+        // Otherwise we need to perform the post-layout tasks now.
+        if (!renderView->selfNeedsLayout()) {
+            // Call scheduleOrPerformPostLayoutTasks() instead of synchronized sendResizeEventIfNeeded()
+            // to avoid hang with resize events in seamless frames.

[esprehn, 2014/03/10 21:32:30]

We don't even have seamless anymore. This code seems suspect.

[Xianzhu, 2014/03/11 00:55:13]

Good news. Removed the comment.

Changed the code and added the comments after I encountered failure of a layout
test containing <iframe seamless> when I was preparing the first version of the
patch.

+            scheduleOrPerformPostLayoutTasks();
+            if (frame().page())
+                frame().page()->chrome().client().layoutUpdated(m_frame.get());
+        }
+    }
 }
 
 void FrameView::didAddScrollbar(Scrollbar* scrollbar, ScrollbarOrientation orientation)
 {
     ScrollableArea::didAddScrollbar(scrollbar, orientation);
     if (AXObjectCache* cache = axObjectCache())
         cache->handleScrollbarUpdate(this);
 }
 
 void FrameView::willRemoveScrollbar(Scrollbar* scrollbar, ScrollbarOrientation orientation)
 {
     ScrollableArea::willRemoveScrollbar(scrollbar, orientation);
     if (AXObjectCache* cache = axObjectCache()) {
         cache->remove(scrollbar);
         cache->handleScrollbarUpdate(this);
     }
 }
 
 } // namespace WebCore