InsertionPoint::attach should be consistent with Element.

• Component: DOM
• Status: RESOLVED FIXED

[Shinya Kawanaka]

This bug is preparation for Bug 78596.

InsertionPoint::attach will have code for <shadow> element later, but it is a bit messy now.
So let's clean up the code.

(1) Currently InsertionPoint attaches fallback elements before attaching distributed elements.
To be consistent, attaching distributed elements first is better, since Element attaches shadow tree first.

(2) Let's extract functions from InsertionPoint::attach() also.

[Shinya Kawanaka]

Created attachment 130278 [details]
Patch

[Hajime Morrita]

Comment on attachment 130278 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=130278&action=review

> Source/WebCore/ChangeLog:3
> +        Refactoring: Make InsertionPoint code clean.

Could you explain the change more concretely?
What the following explanation says isn't cleanup apparently.

> Source/WebCore/ChangeLog:10
> +        InsertionPoint used to attach fallback element before attaching distributed elements.

fallback elements?

> Source/WebCore/ChangeLog:11
> +        To be consistent with Element::attach behavior, attachign distributed elements first is

s/attachign/attaching/

> Source/WebCore/html/shadow/InsertionPoint.cpp:52
> +        distributeLightChildren(root->tree());

Could you reconsider the name? There is no term "light children" in the standard.

[Shinya Kawanaka]

(In reply to comment #2)
> (From update of attachment 130278 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=130278&action=review
> 
> > Source/WebCore/ChangeLog:3
> > +        Refactoring: Make InsertionPoint code clean.
> 
> Could you explain the change more concretely?
> What the following explanation says isn't cleanup apparently.

Done.

> 
> > Source/WebCore/ChangeLog:10
> > +        InsertionPoint used to attach fallback element before attaching distributed elements.
> 
> fallback elements?

Done.

> 
> > Source/WebCore/ChangeLog:11
> > +        To be consistent with Element::attach behavior, attachign distributed elements first is
> 
> s/attachign/attaching/

Done.

> 
> > Source/WebCore/html/shadow/InsertionPoint.cpp:52
> > +        distributeLightChildren(root->tree());
> 
> Could you reconsider the name? There is no term "light children" in the standard.

Renamed to distributeHostChildren.

[Shinya Kawanaka]

Created attachment 130280 [details]
Patch

[Hajime Morrita]

Comment on attachment 130280 [details]
Patch

looks good.

[Shinya Kawanaka]

Committed r109864: <http://trac.webkit.org/changeset/109864>