Optimize repaint on FrameView resize

Source/core/frame/FrameView.cpp

Index: Source/core/frame/FrameView.cpp
diff --git a/Source/core/frame/FrameView.cpp b/Source/core/frame/FrameView.cpp
index 95f79cbc724c033eeb430f97c5e0db39788b65c4..cf21b66daa59b592c0cb62bd1c0bf7eaf762f3c1 100644
--- a/Source/core/frame/FrameView.cpp
+++ b/Source/core/frame/FrameView.cpp
@@ -883,7 +883,6 @@ void FrameView::layout(bool allowSubtree)
         return;
     }
 
-    bool shouldDoFullLayout = false;
     FontCachePurgePreventer fontCachePurgePreventer;
     RenderLayer* layer;
     {
@@ -909,7 +908,7 @@ void FrameView::layout(bool allowSubtree)
         ScrollbarMode vMode;
         calculateScrollbarModesForLayoutAndSetViewportRenderer(hMode, vMode);
 
-        shouldDoFullLayout = !inSubtreeLayout && (m_firstLayout || toRenderView(rootForThisLayout)->document().printing());
+        bool shouldDoFullRepaint = !inSubtreeLayout && (m_firstLayout || toRenderView(rootForThisLayout)->document().printing());
 
         if (!inSubtreeLayout) {
             // Now set our scrollbar state for the layout.
@@ -942,7 +941,9 @@ void FrameView::layout(bool allowSubtree)
             m_size = LayoutSize(layoutSize().width(), layoutSize().height());
 
             if (oldSize != m_size) {
-                shouldDoFullLayout = true;
+                if (!renderView()->style()->isHorizontalWritingMode() || oldSize.width() != m_size.width())
+                    shouldDoFullRepaint = true;
+
                 if (!m_firstLayout) {
                     RenderBox* rootRenderer = document->documentElement() ? document->documentElement()->renderBox() : 0;
                     RenderBox* bodyRenderer = rootRenderer && document->body() ? document->body()->renderBox() : 0;
@@ -958,7 +959,7 @@ void FrameView::layout(bool allowSubtree)
 
         // We need to set m_doFullRepaint before triggering layout as RenderObject::checkForRepaint
         // checks the boolean to disable local repaints.
-        m_doFullRepaint |= shouldDoFullLayout;
+        m_doFullRepaint |= shouldDoFullRepaint;
 
         performLayout(rootForThisLayout, inSubtreeLayout);
 
@@ -1746,7 +1747,8 @@ void FrameView::repaintContentRectangle(const IntRect& r)
 void FrameView::contentsResized()
 {
     ScrollView::contentsResized();
-    setNeedsLayout();
+    if (RenderView* renderView = this->renderView())
+        renderView->setNeedsLayout(MarkOnlyThis, 0, false);

[esprehn, 2014/03/26 17:46:17]

You should not pass MarkOnlyThis. It doesn't really matter for a RenderView, but
it's wrong in general so I'd suggest not doing it.

[Xianzhu, 2014/03/26 18:08:39]

Done.

 }
 
 void FrameView::scrollbarExistenceDidChange()