No need to set setFlowThreadState on RenderText in createTextRenderer.
authorzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 9 Nov 2016 22:25:07 +0000 (22:25 +0000)
committerzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 9 Nov 2016 22:25:07 +0000 (22:25 +0000)
https://bugs.webkit.org/show_bug.cgi?id=164559

Reviewed by Antti Koivisto.

       setFlowThreadState in create*Renderer ensures that by the time we issue the initial call to
       styleWillChange/styleDidChange through initializeStyle, the state is already set.
       However since RenderText does not have its own style, it's sufficient to have the flow state set
       through the normal RenderElement::insertChildInternal code path.

No change in functionality.

* style/RenderTreeUpdater.cpp:
(WebCore::createTextRenderer):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@208487 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/style/RenderTreeUpdater.cpp

index 9e8bd73..cdb88af 100644 (file)
@@ -1,3 +1,20 @@
+2016-11-09  Zalan Bujtas  <zalan@apple.com>
+
+        No need to set setFlowThreadState on RenderText in createTextRenderer.
+        https://bugs.webkit.org/show_bug.cgi?id=164559
+
+        Reviewed by Antti Koivisto.
+
+       setFlowThreadState in create*Renderer ensures that by the time we issue the initial call to
+       styleWillChange/styleDidChange through initializeStyle, the state is already set.
+       However since RenderText does not have its own style, it's sufficient to have the flow state set
+       through the normal RenderElement::insertChildInternal code path.
+
+        No change in functionality.
+
+        * style/RenderTreeUpdater.cpp:
+        (WebCore::createTextRenderer):
+
 2016-11-09  Brady Eidson  <beidson@apple.com>
 
         IndexedDB 2.0: Encapsulate cursor iteration parameters for easy future expansion.
index 3a39e23..782a6ab 100644 (file)
@@ -429,10 +429,6 @@ static void createTextRenderer(Text& textNode, RenderTreePosition& renderTreePos
     if (!renderTreePosition.canInsert(*newRenderer))
         return;
 
-    // Make sure the RenderObject already knows it is going to be added to a RenderFlowThread before we set the style
-    // for the first time. Otherwise code using inRenderFlowThread() in the styleWillChange and styleDidChange will fail.
-    newRenderer->setFlowThreadState(renderTreePosition.parent().flowThreadState());
-
     textNode.setRenderer(newRenderer.get());
     renderTreePosition.insert(*newRenderer.leakPtr());
 }