Changeset 228414 in webkit
- Timestamp:
- Feb 12, 2018 7:26:50 PM (6 years ago)
- Location:
- trunk/Source/WebCore
- Files:
-
- 5 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r228413 r228414 1 2018-02-12 Zalan Bujtas <zalan@apple.com> 2 3 [RenderTreeBuilder] Move RenderMenuList::takeChild() to RenderTreeBuilder 4 https://bugs.webkit.org/show_bug.cgi?id=182715 5 <rdar://problem/37477050> 6 7 Reviewed by Antti Koivisto. 8 9 No change in functionality. 10 11 * rendering/RenderMenuList.cpp: 12 (RenderMenuList::takeChild): Deleted. 13 * rendering/RenderMenuList.h: 14 * rendering/updating/RenderTreeBuilder.cpp: 15 (WebCore::RenderTreeBuilder::takeChild): 16 (WebCore::RenderTreeBuilder::takeChildFromRenderMenuList): 17 * rendering/updating/RenderTreeBuilder.h: 18 1 19 2018-02-12 Ross Kirsling <ross.kirsling@sony.com> 2 20 -
trunk/Source/WebCore/rendering/RenderMenuList.cpp
r228400 r228414 173 173 } 174 174 175 RenderPtr<RenderObject> RenderMenuList::takeChild(RenderTreeBuilder& builder, RenderObject& oldChild)176 {177 if (!m_innerBlock || &oldChild == m_innerBlock)178 return RenderFlexibleBox::takeChild(builder, oldChild);179 return builder.takeChild(*m_innerBlock, oldChild);180 }181 182 175 void RenderMenuList::styleDidChange(StyleDifference diff, const RenderStyle* oldStyle) 183 176 { -
trunk/Source/WebCore/rendering/RenderMenuList.h
r228337 r228414 71 71 72 72 void addChild(RenderTreeBuilder&, RenderPtr<RenderObject> newChild, RenderObject* beforeChild = 0) override; 73 RenderPtr<RenderObject> takeChild(RenderTreeBuilder&, RenderObject&) override;74 73 bool createsAnonymousWrapper() const override { return true; } 75 74 -
trunk/Source/WebCore/rendering/updating/RenderTreeBuilder.cpp
r228400 r228414 29 29 #include "RenderButton.h" 30 30 #include "RenderElement.h" 31 #include "RenderMenuList.h" 31 32 #include "RenderRuby.h" 32 33 #include "RenderRubyBase.h" … … 199 200 return rubyBuilder().takeChild(downcast<RenderRubyRun>(parent), child); 200 201 202 if (is<RenderMenuList>(parent)) 203 return takeChildFromRenderMenuList(downcast<RenderMenuList>(parent), child); 204 201 205 return parent.takeChild(*this, child); 202 206 } … … 511 515 } 512 516 513 } 517 RenderPtr<RenderObject> RenderTreeBuilder::takeChildFromRenderMenuList(RenderMenuList& parent, RenderObject& child) 518 { 519 auto* innerRenderer = parent.innerRenderer(); 520 if (!innerRenderer || &child == innerRenderer) 521 return parent.RenderBlock::takeChild(*this, child); 522 return takeChild(*innerRenderer, child); 523 } 524 525 } -
trunk/Source/WebCore/rendering/updating/RenderTreeBuilder.h
r228400 r228414 31 31 32 32 class RenderMathMLFenced; 33 class RenderMenuList; 33 34 class RenderRubyAsBlock; 34 35 class RenderRubyAsInline; … … 104 105 105 106 void removeAnonymousWrappersForInlineChildrenIfNeeded(RenderElement& parent); 107 RenderPtr<RenderObject> takeChildFromRenderMenuList(RenderMenuList& parent, RenderObject& child); 106 108 107 109 FirstLetter& firstLetterBuilder() { return *m_firstLetterBuilder; }
Note: See TracChangeset
for help on using the changeset viewer.