Changeset 223549 in webkit

Timestamp:

Oct 17, 2017 9:17:38 AM (7 years ago)

Author:

Alan Bujtas

Message:

[FrameView::layout cleanup] Merge InPreLayout blocks
​https://bugs.webkit.org/show_bug.cgi?id=178373

Reviewed by Antti Koivisto.

Remove redundant comment and scoping.

No change in functionality.

• page/FrameView.cpp:

(WebCore::FrameView::layout):

Location:

trunk/Source/WebCore

Files:

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

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-10-17  Zalan Bujtas  <zalan@apple.com>
+
+        [FrameView::layout cleanup] Merge InPreLayout blocks
+        https://bugs.webkit.org/show_bug.cgi?id=178373
+
+        Reviewed by Antti Koivisto.
+
+        Remove redundant comment and scoping.
+
+        No change in functionality.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::layout):
+
 2017-10-17  Antti Koivisto  <antti@apple.com>
 

trunk/Source/WebCore/page/FrameView.cpp

@@ -1366 +1366 @@
     Document& document = *frame().document();
     ASSERT(document.pageCacheState() == Document::NotInPageCache);
+    RenderElement* root = nullptr;
+    RenderLayer* layer = nullptr;
+    bool subtree = false;
 
     {
@@ -1399 +1402 @@
             return;
 
-        // Close block here so we can set up the font cache purge preventer, which we will still
-        // want in scope even after we want m_layoutSchedulingEnabled to be restored again.
-        // The next block sets m_layoutSchedulingEnabled back to false once again.
-    }
-
-    m_layoutPhase = InPreLayout;
-
-    RenderLayer* layer = nullptr;
-    bool subtree = false;
-    RenderElement* root = nullptr;
-
-    ++m_nestedLayoutCount;
-
-    {
-        SetForScope<bool> changeSchedulingEnabled(m_layoutSchedulingEnabled, false);
+        m_layoutPhase = InPreLayout;
+
+        ++m_nestedLayoutCount;
 
         autoSizeIfEnabled();