[Shadow] Move Distribution requirements from ShadowRoot

[Shinya Kawanaka]

We would like to make ShadowRoot as small as possible.
Currently ShadowRoot has several ContentDistribution stuffs which are used only for ShadowRoot having <content>, <shadow> or nested ShadowRoot.
We woud like to move them somewhere.

[Shinya Kawanaka]

Apple folks seem to reduce memory footprint aggressively now.

[Shinya Kawanaka]

Created attachment 176417 [details]
Patch

[Shinya Kawanaka]

Let's move DistributionData into ContentDistribution.h

[Shinya Kawanaka]

Created attachment 176436 [details]
Patch

[Build Bot]

Comment on attachment 176436 [details]
Patch

Attachment 176436 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/15016356

[kov's GTK+ EWS bot]

Comment on attachment 176436 [details]
Patch

Attachment 176436 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/15027358

[Build Bot]

Comment on attachment 176436 [details]
Patch

Attachment 176436 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/15031177

[Shinya Kawanaka]

Created attachment 176450 [details]
Patch

[Hajime Morrita]

Comment on attachment 176450 [details]
Patch

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

> Source/WebCore/ChangeLog:10
> +        Since <input> or <textarea> do not have such stuffs, the members consume a few amount of memory.

s/stuffs/stuff/

> Source/WebCore/dom/ShadowRoot.cpp:352
> +    return distributionData()->numberOfShadowElementChildren() > 0;

hasXXX could be a method on ShadowRootContentDistributionData

> Source/WebCore/dom/ShadowRoot.cpp:370
> +    return distributionData()->numberOfContentElementChildren() > 0;

Ditto.

> Source/WebCore/dom/ShadowRoot.cpp:389
> +    return distributionData()->numberOfElementShadowChildren() > 0;

Ditto.

[Dimitri Glazkov (Google)]

Comment on attachment 176450 [details]
Patch

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

>> Source/WebCore/ChangeLog:10
>> +        Since <input> or <textarea> do not have such stuffs, the members consume a few amount of memory.
> 
> s/stuffs/stuff/

s/stuffs/requirements/ </bikeshed>

[Shinya Kawanaka]

Created attachment 176617 [details]
Patch for landing

[WebKit Review Bot]

Comment on attachment 176617 [details]
Patch for landing

Clearing flags on attachment: 176617

Committed r136081: <http://trac.webkit.org/changeset/136081>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.