Add infrastructure for partial layouts

Source/core/page/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 161 matching lines @@
     , m_isTrackingRepaints(false)
     , m_shouldUpdateWhileOffscreen(true)
     , m_deferSetNeedsLayouts(0)
     , m_setNeedsLayoutWasDeferred(false)
     , m_scrollCorner(0)
     , m_shouldAutoSize(false)
     , m_inAutoSize(false)
     , m_didRunAutosize(false)
     , m_hasSoftwareFilters(false)
     , m_visibleContentScaleFactor(1)
+    , m_shouldStopPartialLayout(false)
+    , m_stopLayoutAtRenderer(0)
 {
     init();
 
     // FIXME: Can m_frame ever be null here?
     if (!m_frame)
         return;
 
     Page* page = m_frame->page();
     if (!page)
         return;
@@ skipping 647 matching lines @@
 void FrameView::layout(bool allowSubtree)
 {
     // We should never layout a Document which is not in a Frame.
     ASSERT(m_frame);
     ASSERT(m_frame->view() == this);
     ASSERT(m_frame->page());
 
     if (m_inLayout)
         return;
 
+    ASSERT(!shouldStopPartialLayout());
+
     TRACE_EVENT0("webkit", "FrameView::layout");
     TRACE_EVENT_SCOPED_SAMPLING_STATE("Blink", "Layout");
 
     // Protect the view from being deleted during layout (in recalcStyle)
     RefPtr<FrameView> protector(this);
 
     // Every scroll that happens during layout is programmatic.
     TemporaryChange<bool> changeInProgrammaticScroll(m_inProgrammaticScroll, true);
 
     m_layoutTimer.stop();
@@ skipping 117 matching lines @@
                     else if (rootRenderer && rootRenderer->stretchesToViewport())
                         rootRenderer->setChildNeedsLayout();
                 }
             }
         }
 
         layer = rootForThisLayout->enclosingLayer();
 
         m_actionScheduler->pause();
 
+        // Text Autosizing requires two-pass layout which is incompatible with partial layout. If

[esprehn, 2013/08/18 03:43:15]

This should be a FIXME for text autosizing not to do that.

[pdr., 2013/08/20 06:19:10]

Added fixme and a reference to crbug.com/256657.

+        // enabled, only do partial layout for the second layout.
+        RenderObject* delayPartialLayoutForRenderer = 0;
+        if (m_frame && m_frame->settings() && m_frame->settings()->textAutosizingEnabled()) {

[esprehn, 2013/08/18 03:43:15]

m_frame cannot be null here. I'm pretty sure that'd be catastrophic to do layout
on a frameless document.

[pdr., 2013/08/20 06:19:10]

Done.

+            delayPartialLayoutForRenderer = m_stopLayoutAtRenderer;
+            m_stopLayoutAtRenderer = 0;
+        }
+
         {
             bool disableLayoutState = false;
             if (inSubtreeLayout) {
                 RenderView* view = rootForThisLayout->view();
                 disableLayoutState = view->shouldDisableLayoutStateForSubtree(rootForThisLayout);
                 view->pushLayoutState(rootForThisLayout);
             }
             LayoutStateDisabler layoutStateDisabler(disableLayoutState ? rootForThisLayout->view() : 0);
 
             m_inLayout = true;
             beginDeferredRepaints();
             forceLayoutParentViewIfNeeded();
             rootForThisLayout->layout();
 
+            // If text autosizing is enabled, do a partial layout for the second layout.
+            if (delayPartialLayoutForRenderer) {
+                ASSERT(!shouldStopPartialLayout());
+                m_stopLayoutAtRenderer = delayPartialLayoutForRenderer;
+            }
             bool autosized = document->textAutosizer()->processSubtree(rootForThisLayout);
             if (autosized && rootForThisLayout->needsLayout()) {
                 TRACE_EVENT0("webkit", "2nd layout due to Text Autosizing");
                 rootForThisLayout->layout();
             }
 
             endDeferredRepaints();
             m_inLayout = false;
 
             if (inSubtreeLayout)
                 rootForThisLayout->view()->popLayoutState(rootForThisLayout);
         }
         m_layoutRoot = 0;
     } // Reset m_layoutSchedulingEnabled to its previous value.
 
+    // Reset partial layout state post-layout and exit here if a partial layout was done.
+    if (shouldStopPartialLayout()) {
+        resetPartialLayoutState();
+        return;
+    }
+    resetPartialLayoutState();
+
     bool neededFullRepaint = m_doFullRepaint;
 
     if (!inSubtreeLayout && !toRenderView(rootForThisLayout)->printing())
         adjustViewSize();
 
     m_doFullRepaint = neededFullRepaint;
 
     // Now update the positions of all layers.
     beginDeferredRepaints();
     if (m_doFullRepaint)
@@ skipping 63 matching lines @@
     // however m_inLayout is not set for most of this function, so none of our RELEASE_ASSERTS
     // in Frame/Page will fire. One of the post-layout tasks is disconnecting the Frame from
     // the page in fast/frames/crash-remove-iframe-during-object-beforeload-2.html
     // necessitating this check here.
     ASSERT(frame());
     // ASSERT(frame()->page());
     if (frame() && frame()->page())
         frame()->page()->chrome().client()->layoutUpdated(frame());
 }
 
+void FrameView::checkPartialLayoutComplete(RenderObject* renderer)

[esprehn, 2013/08/18 03:43:15]

This check should be inline.

[pdr., 2013/08/20 06:19:10]

Done.

+{
+    ASSERT(renderer);
+    if (renderer == m_stopLayoutAtRenderer) {
+        // Make sure a partial layout was allowed.
+        do {
+            ASSERT(renderer->supportsPartialLayout());
+        } while ((renderer = renderer->parent()));

[esprehn, 2013/08/18 03:43:15]

Why do you need to check this twice? Doing it either before or after should be
enough?

[pdr., 2013/08/20 06:19:10]

The supportsPartialLayout loop was intended to just be an assert. I've cleaned
this up like so:
1) I've inlined checkPartialLayoutComplete and changed it to return a bool if
layout is complete.
2) I've moved the assertion loop behind an #ifndef NDEBUG check and into a
private function "checkPartialLayoutAllowed". The purpose of
checkPartialLayoutAllowed is just to assert that everything is working properly.

+
+        m_shouldStopPartialLayout = true;
+    }
+}
+
 RenderBox* FrameView::embeddedContentBox() const
 {
     RenderView* renderView = this->renderView();
     if (!renderView)
         return 0;
 
     RenderObject* firstChild = renderView->firstChild();
     if (!firstChild || !firstChild->isBox())
         return 0;
 
@@ skipping 2221 matching lines @@
 }
 
 AXObjectCache* FrameView::axObjectCache() const
 {
     if (frame() && frame()->document())
         return frame()->document()->existingAXObjectCache();
     return 0;
 }
 
 } // namespace WebCore