Changeset 223622 in webkit

Timestamp:

Oct 18, 2017 12:19:38 PM (7 years ago)

Author:

Alan Bujtas

Message:

[FrameView::layout cleanup] Move root/body marking dirty logic to a separate function
​https://bugs.webkit.org/show_bug.cgi?id=178477
<rdar://problem/35056478>

Reviewed by Simon Fraser.

No change in functionality.

• page/FrameView.cpp:

(WebCore::FrameView::markRootOrBodyRendererDirty const):
(WebCore::FrameView::layout):

• page/FrameView.h:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/FrameView.cpp (modified) (2 diffs)
• page/FrameView.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-10-18  Zalan Bujtas  <zalan@apple.com>
+
+        [FrameView::layout cleanup] Move root/body marking dirty logic to a separate function
+        https://bugs.webkit.org/show_bug.cgi?id=178477
+        <rdar://problem/35056478>
+
+        Reviewed by Simon Fraser.
+
+        No change in functionality.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::markRootOrBodyRendererDirty const):
+        (WebCore::FrameView::layout):
+        * page/FrameView.h:
+
 2017-10-18  Keith Miller  <keith_miller@apple.com>
 

trunk/Source/WebCore/page/FrameView.cpp

@@ -1337 +1337 @@
 }
 
+void FrameView::markRootOrBodyRendererDirty() const
+{
+    auto& document = *frame().document();
+    RenderBox* rootRenderer = document.documentElement() ? document.documentElement()->renderBox() : nullptr;
+    auto* body = document.bodyOrFrameset();
+    RenderBox* bodyRenderer = rootRenderer && body ? body->renderBox() : nullptr;
+    if (bodyRenderer && bodyRenderer->stretchesToViewport())
+        bodyRenderer->setChildNeedsLayout();
+    else if (rootRenderer && rootRenderer->stretchesToViewport())
+        rootRenderer->setChildNeedsLayout();
+}
+
 void FrameView::layout(bool allowSubtreeLayout)
 {
@@ -1477 +1489 @@
             }
 
-            LayoutSize oldSize = m_size;
+            auto oldSize = m_size;
             m_size = layoutSize();
-
             if (oldSize != m_size) {
                 LOG(Layout, "  layout size changed from %.3fx%.3f to %.3fx%.3f", oldSize.width().toFloat(), oldSize.height().toFloat(), m_size.width().toFloat(), m_size.height().toFloat());
                 m_needsFullRepaint = true;
-                if (!m_firstLayout) {
-                    RenderBox* rootRenderer = document.documentElement() ? document.documentElement()->renderBox() : nullptr;
-                    auto* body = document.bodyOrFrameset();
-                    RenderBox* bodyRenderer = rootRenderer && body ? body->renderBox() : nullptr;
-                    if (bodyRenderer && bodyRenderer->stretchesToViewport())
-                        bodyRenderer->setChildNeedsLayout();
-                    else if (rootRenderer && rootRenderer->stretchesToViewport())
-                        rootRenderer->setChildNeedsLayout();
-                }
+                if (!m_firstLayout)
+                    markRootOrBodyRendererDirty();
             }
-
             m_layoutPhase = InPreLayout;
         }

trunk/Source/WebCore/page/FrameView.h

@@ -767 +767 @@
     bool frameFlatteningEnabled() const;
     bool isFrameFlatteningValidForThisFrame() const;
+    
+    void markRootOrBodyRendererDirty() const;
 
     bool qualifiesAsVisuallyNonEmpty() const;