Do not call updateDistribution() in Node::assingedSlotForBinding()

Do not call updateDistribution() in Node::assingedSlotForBinding()

updateDistribution() is no longer required here since slotname-to-slot hashmap is
dynamically updated at the timing of DOM mutations.

BUG=671311
Committed: https://crrev.com/5277d3159e5a5ca75fb1306167d6356b99c97e32
Cr-Commit-Position: refs/heads/master@{#437505}

[hayato, 2016-12-09 07:57:17]

The CQ bit was checked by hayato@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-09 07:57:28]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hayato, 2016-12-09 08:11:05]

Description was changed from

==========
Do not call updateDistribution() in Node::assingedNodeForBinding()

updateDistribution() is no longer required here since slotname-to-slot hashmap
is
dynamically updated at the timing of DOM mutations.

BUG= 671311
==========

to

==========
Do not call updateDistribution() in Node::assingedNodeForBinding()

updateDistribution() is no longer required here since slotname-to-slot hashmap
is
dynamically updated at the timing of DOM mutations.

BUG=671311
==========

[hayato, 2016-12-09 08:11:17]

hayato@chromium.org changed reviewers:
+ kochi@chromium.org

[hayato, 2016-12-09 08:11:17]

PTAL

[hayato, 2016-12-09 08:11:38]

Description was changed from

==========
Do not call updateDistribution() in Node::assingedNodeForBinding()

updateDistribution() is no longer required here since slotname-to-slot hashmap
is
dynamically updated at the timing of DOM mutations.

BUG=671311
==========

to

==========
Do not call updateDistribution() in Node::assingedSlotForBinding()

updateDistribution() is no longer required here since slotname-to-slot hashmap
is
dynamically updated at the timing of DOM mutations.

BUG=671311
==========

[kochi, 2016-12-09 08:29:04]

lgtm

https://codereview.chromium.org/2559373003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/dom/Node.cpp (right):

https://codereview.chromium.org/2559373003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/dom/Node.cpp:2303: // assignedSlot doesn't needs
to call updateDistribution().
s/needs/need/

https://codereview.chromium.org/2559373003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/dom/Node.cpp:2311: // assignedSlot doesn't needs
to call updateDistribution().
Ditto.

[hayato, 2016-12-09 08:31:13]

fixed

[hayato, 2016-12-09 08:31:13]

The CQ bit was checked by hayato@chromium.org

[hayato, 2016-12-09 08:31:14]

The patchset sent to the CQ was uploaded after l-g-t-m from kochi@chromium.org
Link to the patchset: https://codereview.chromium.org/2559373003/#ps20001
(title: "fixed")

[commit-bot: I haz the power, 2016-12-09 08:31:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-09 09:45:05]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481272273656430,
"parent_rev": "85451da397b36fbcb497a526f6c70afb1a6577ce", "commit_rev":
"d596a10b9056addf0c44a48e800fa7f01a749ec6"}

[commit-bot: I haz the power, 2016-12-09 09:45:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-09 09:48:20]

Description was changed from

==========
Do not call updateDistribution() in Node::assingedSlotForBinding()

updateDistribution() is no longer required here since slotname-to-slot hashmap
is
dynamically updated at the timing of DOM mutations.

BUG=671311
==========

to

==========
Do not call updateDistribution() in Node::assingedSlotForBinding()

updateDistribution() is no longer required here since slotname-to-slot hashmap
is
dynamically updated at the timing of DOM mutations.

BUG=671311

Committed: https://crrev.com/5277d3159e5a5ca75fb1306167d6356b99c97e32
Cr-Commit-Position: refs/heads/master@{#437505}
==========

[commit-bot: I haz the power, 2016-12-09 09:48:20]

Patchset 2 (id:??) landed as
https://crrev.com/5277d3159e5a5ca75fb1306167d6356b99c97e32
Cr-Commit-Position: refs/heads/master@{#437505}