Title: [224225] trunk/Source/WebCore
Revision
224225
Author
rn...@webkit.org
Date
2017-10-31 01:39:38 -0700 (Tue, 31 Oct 2017)

Log Message

Assert that no script is executed during LayoutPhase::InRenderTreeLayout
https://bugs.webkit.org/show_bug.cgi?id=179046

Reviewed by Antti Koivisto.

Added NoEventDispatchAssertion to LayoutPhase::InRenderTreeLayout.

* page/LayoutContext.cpp:
(WebCore::LayoutContext::layout):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (224224 => 224225)


--- trunk/Source/WebCore/ChangeLog	2017-10-31 08:06:07 UTC (rev 224224)
+++ trunk/Source/WebCore/ChangeLog	2017-10-31 08:39:38 UTC (rev 224225)
@@ -1,3 +1,15 @@
+2017-10-30  Ryosuke Niwa  <rn...@webkit.org>
+
+        Assert that no script is executed during LayoutPhase::InRenderTreeLayout
+        https://bugs.webkit.org/show_bug.cgi?id=179046
+
+        Reviewed by Antti Koivisto.
+
+        Added NoEventDispatchAssertion to LayoutPhase::InRenderTreeLayout.
+
+        * page/LayoutContext.cpp:
+        (WebCore::LayoutContext::layout):
+
 2017-10-31  Devin Rousso  <web...@devinrousso.com>
 
         Web Inspector: replace Canvas stack iteration with createScriptCallStack

Modified: trunk/Source/WebCore/page/LayoutContext.cpp (224224 => 224225)


--- trunk/Source/WebCore/page/LayoutContext.cpp	2017-10-31 08:06:07 UTC (rev 224224)
+++ trunk/Source/WebCore/page/LayoutContext.cpp	2017-10-31 08:39:38 UTC (rev 224225)
@@ -32,6 +32,7 @@
 #include "FrameView.h"
 #include "InspectorInstrumentation.h"
 #include "Logging.h"
+#include "NoEventDispatchAssertion.h"
 #include "RenderElement.h"
 #include "RenderView.h"
 #include "Settings.h"
@@ -210,7 +211,7 @@
     }
     {
         SetForScope<LayoutPhase> layoutPhase(m_layoutPhase, LayoutPhase::InRenderTreeLayout);
-    
+        NoEventDispatchAssertion noEventDispatchAssertion;
         SubtreeLayoutStateMaintainer subtreeLayoutStateMaintainer(subtreeLayoutRoot());
         RenderView::RepaintRegionAccumulator repaintRegionAccumulator(renderView());
 #ifndef NDEBUG
@@ -222,7 +223,6 @@
         applyTextSizingIfNeeded(*layoutRoot.get());
 #endif
         clearSubtreeLayoutRoot();
-
     }
     {
         SetForScope<LayoutPhase> layoutPhase(m_layoutPhase, LayoutPhase::InViewSizeAdjust);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to