Changeset 228345 in webkit

Timestamp:

Feb 9, 2018 4:45:25 PM (6 years ago)

Author:

Alan Bujtas

Message:

[RenderTreeBuilder] Move RenderRubyRun::takeChild mutation to a RenderTreeBuilder
​https://bugs.webkit.org/show_bug.cgi?id=182655
<rdar://problem/37406801>

Reviewed by Simon Fraser.

No change in functionality.

• rendering/RenderRubyRun.cpp:

(WebCore::RenderRubyRun::takeChild):

• rendering/updating/RenderTreeBuilder.cpp:

(WebCore::RenderTreeBuilder::takeChildFromRenderRubyRun):

• rendering/updating/RenderTreeBuilder.h:
• rendering/updating/RenderTreeBuilderRuby.cpp:

(WebCore::RenderTreeBuilder::Ruby::takeChild):

• rendering/updating/RenderTreeBuilderRuby.h:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderRubyRun.cpp (modified) (1 diff)
• rendering/updating/RenderTreeBuilder.cpp (modified) (1 diff)
• rendering/updating/RenderTreeBuilder.h (modified) (1 diff)
• rendering/updating/RenderTreeBuilderRuby.cpp (modified) (1 diff)
• rendering/updating/RenderTreeBuilderRuby.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2018-02-09  Zalan Bujtas  <zalan@apple.com>
+
+        [RenderTreeBuilder] Move RenderRubyRun::takeChild mutation to a RenderTreeBuilder
+        https://bugs.webkit.org/show_bug.cgi?id=182655
+        <rdar://problem/37406801>
+
+        Reviewed by Simon Fraser.
+
+        No change in functionality.
+
+        * rendering/RenderRubyRun.cpp:
+        (WebCore::RenderRubyRun::takeChild):
+        * rendering/updating/RenderTreeBuilder.cpp:
+        (WebCore::RenderTreeBuilder::takeChildFromRenderRubyRun):
+        * rendering/updating/RenderTreeBuilder.h:
+        * rendering/updating/RenderTreeBuilderRuby.cpp:
+        (WebCore::RenderTreeBuilder::Ruby::takeChild):
+        * rendering/updating/RenderTreeBuilderRuby.h:
+
 2018-02-09  Andy Estes  <aestes@apple.com>
 

trunk/Source/WebCore/rendering/RenderRubyRun.cpp

@@ -100 +100 @@
 RenderPtr<RenderObject> RenderRubyRun::takeChild(RenderTreeBuilder& builder, RenderObject& child)
 {
-    // If the child is a ruby text, then merge the ruby base with the base of
-    // the right sibling run, if possible.
-    if (!beingDestroyed() && !renderTreeBeingDestroyed() && child.isRubyText()) {
-        RenderRubyBase* base = rubyBase();
-        RenderObject* rightNeighbour = nextSibling();
-        if (base && is<RenderRubyRun>(rightNeighbour)) {
-            // Ruby run without a base can happen only at the first run.
-            RenderRubyRun& rightRun = downcast<RenderRubyRun>(*rightNeighbour);
-            if (rightRun.hasRubyBase()) {
-                RenderRubyBase* rightBase = rightRun.rubyBase();
-                // Collect all children in a single base, then swap the bases.
-                builder.moveRubyChildren(*rightBase, *base);
-                moveChildTo(builder, &rightRun, base, RenderBoxModelObject::NormalizeAfterInsertion::No);
-                rightRun.moveChildTo(builder, this, rightBase, RenderBoxModelObject::NormalizeAfterInsertion::No);
-                // The now empty ruby base will be removed below.
-                ASSERT(!rubyBase()->firstChild());
-            }
-        }
-    }
-
-    auto takenChild = RenderBlockFlow::takeChild(builder, child);
-
-    if (!beingDestroyed() && !renderTreeBeingDestroyed()) {
-        // Check if our base (if any) is now empty. If so, destroy it.
-        RenderBlock* base = rubyBase();
-        if (base && !base->firstChild()) {
-            auto takenBase = RenderBlockFlow::takeChild(builder, *base);
-            base->deleteLines();
-        }
-    }
-
-    return takenChild;
+    return builder.takeChildFromRenderRubyRun(*this, child);
 }
 

trunk/Source/WebCore/rendering/updating/RenderTreeBuilder.cpp

@@ -481 +481 @@
 }
 
+RenderPtr<RenderObject> RenderTreeBuilder::takeChildFromRenderRubyRun(RenderRubyRun& parent, RenderObject& child)
+{
+    return rubyBuilder().takeChild(parent, child);
+}
+
 void RenderTreeBuilder::updateAfterDescendants(RenderElement& renderer)
 {

trunk/Source/WebCore/rendering/updating/RenderTreeBuilder.h

@@ -76 +76 @@
     RenderPtr<RenderObject> takeChildFromRenderRubyAsInline(RenderRubyAsInline& parent, RenderObject& child);
     RenderPtr<RenderObject> takeChildFromRenderRubyAsBlock(RenderRubyAsBlock& parent, RenderObject& child);
+    RenderPtr<RenderObject> takeChildFromRenderRubyRun(RenderRubyRun& parent, RenderObject& child);
 
     bool childRequiresTable(const RenderElement& parent, const RenderObject& child);

trunk/Source/WebCore/rendering/updating/RenderTreeBuilderRuby.cpp

@@ -421 +421 @@
 }
 
-}
+RenderPtr<RenderObject> RenderTreeBuilder::Ruby::takeChild(RenderRubyRun& parent, RenderObject& child)
+{
+    // If the child is a ruby text, then merge the ruby base with the base of
+    // the right sibling run, if possible.
+    if (!parent.beingDestroyed() && !parent.renderTreeBeingDestroyed() && child.isRubyText()) {
+        RenderRubyBase* base = parent.rubyBase();
+        RenderObject* rightNeighbour = parent.nextSibling();
+        if (base && is<RenderRubyRun>(rightNeighbour)) {
+            // Ruby run without a base can happen only at the first run.
+            RenderRubyRun& rightRun = downcast<RenderRubyRun>(*rightNeighbour);
+            if (rightRun.hasRubyBase()) {
+                RenderRubyBase* rightBase = rightRun.rubyBase();
+                // Collect all children in a single base, then swap the bases.
+                m_builder.moveRubyChildren(*rightBase, *base);
+                parent.moveChildTo(m_builder, &rightRun, base, RenderBoxModelObject::NormalizeAfterInsertion::No);
+                rightRun.moveChildTo(m_builder, &parent, rightBase, RenderBoxModelObject::NormalizeAfterInsertion::No);
+                // The now empty ruby base will be removed below.
+                ASSERT(!parent.rubyBase()->firstChild());
+            }
+        }
+    }
+
+    auto takenChild = parent.RenderBlockFlow::takeChild(m_builder, child);
+
+    if (!parent.beingDestroyed() && !parent.renderTreeBeingDestroyed()) {
+        // Check if our base (if any) is now empty. If so, destroy it.
+        RenderBlock* base = parent.rubyBase();
+        if (base && !base->firstChild()) {
+            auto takenBase = parent.RenderBlockFlow::takeChild(m_builder, *base);
+            base->deleteLines();
+        }
+    }
+    return takenChild;
+}
+
+}

trunk/Source/WebCore/rendering/updating/RenderTreeBuilderRuby.h

@@ -45 +45 @@
     RenderPtr<RenderObject> takeChild(RenderRubyAsInline& parent, RenderObject& child);
     RenderPtr<RenderObject> takeChild(RenderRubyAsBlock& parent, RenderObject& child);
+    RenderPtr<RenderObject> takeChild(RenderRubyRun& parent, RenderObject& child);
+
+    void moveChildren(RenderRubyBase& from, RenderRubyBase& to);
 
     RenderElement& findOrCreateParentForChild(RenderRubyAsBlock& parent, const RenderObject& child, RenderObject*& beforeChild);
     RenderElement& findOrCreateParentForChild(RenderRubyAsInline& parent, const RenderObject& child, RenderObject*& beforeChild);
-
-    void moveChildren(RenderRubyBase& from, RenderRubyBase& to);
 
 private: