Remove unnecessary recalc of host subtree on distribute.

Remove unnecessary recalc of host subtree on distribute.

Nodes which need style recalculations after being distributed to new
insertion points, new positions within insertion points, or simply not
distributed anymore, are being reattached in
InsertionPoint::setDistributionNodes().

R=hayato@chromium.org,kochi@chromium.org,esprehn@chromium.org
TEST=fast/dom/shadow,fast/css
BUG=584177
Committed: https://crrev.com/6969673edc422ff98080a1e6ff1fa798425954c6
Cr-Commit-Position: refs/heads/master@{#374629}

[rune, 2016-02-05 13:56:20]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-05 13:56:34]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1677463002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1677463002/1

[commit-bot: I haz the power, 2016-02-05 15:09:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-05 15:10:00]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[rune, 2016-02-08 12:02:53]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-08 12:04:29]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1677463002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1677463002/40001

[rune, 2016-02-08 13:02:46]

Regarding the changed test-expectations:

The dumped text now contains the characters you see output for a normal browser
run. I have not been able to exactly pin-point where we behave differently when
we force an unnecessary subtree recalc on the host before re-distribution, but
when we do force the recalc, we end up setting m_textBox to null in [1]. The bug
[2] indicates this is a combination of ::first-letter and RTL which splits
differently at punctuations.

The fact that the ::first-letter is invisible, yet the rest of ::first-line
isn't, is because we support setting all properties on ::first-line, including
visibility, yet we don't apply it. However, it's inherited down to
::first-letter which supports visibility.

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...
[2] https://crbug.com/446502

[rune, 2016-02-08 13:03:09]

So, ptal.

[commit-bot: I haz the power, 2016-02-08 15:40:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-08 15:40:21]

Dry run: This issue passed the CQ dry run.

[esprehn, 2016-02-10 01:20:58]

lgtm seems legit.

[esprehn, 2016-02-10 01:21:00]

lgtm

[kochi, 2016-02-10 02:24:30]

lgtm

[rune, 2016-02-10 07:19:55]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-02-10 07:20:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1677463002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1677463002/40001

[commit-bot: I haz the power, 2016-02-10 08:27:04]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-10 08:28:10]

Description was changed from

==========
Remove unnecessary recalc of host subtree on distribute.

Nodes which need style recalculations after being distributed to new
insertion points, new positions within insertion points, or simply not
distributed anymore, are being reattached in
InsertionPoint::setDistributionNodes().

R=hayato@chromium.org,kochi@chromium.org,esprehn@chromium.org
TEST=fast/dom/shadow,fast/css
BUG=584177
==========

to

==========
Remove unnecessary recalc of host subtree on distribute.

Nodes which need style recalculations after being distributed to new
insertion points, new positions within insertion points, or simply not
distributed anymore, are being reattached in
InsertionPoint::setDistributionNodes().

R=hayato@chromium.org,kochi@chromium.org,esprehn@chromium.org
TEST=fast/dom/shadow,fast/css
BUG=584177

Committed: https://crrev.com/6969673edc422ff98080a1e6ff1fa798425954c6
Cr-Commit-Position: refs/heads/master@{#374629}
==========

[commit-bot: I haz the power, 2016-02-10 08:28:14]

Patchset 3 (id:??) landed as
https://crrev.com/6969673edc422ff98080a1e6ff1fa798425954c6
Cr-Commit-Position: refs/heads/master@{#374629}