Remove unwanted copyToVector() uses.

Remove unwanted copyToVector() uses.

If a collection is being cleared while creating an iterable
view of its current contents, swap in an empty collection
rather than copy out the current contents into a temporary
vector (by way of copyToVector().)

R=tkent
BUG=

Committed: https://crrev.com/b0423799d4c45060338e6d7ac6db3a284f238aac
Cr-Commit-Position: refs/heads/master@{#395547}

[sof, 2016-05-24 07:10:14]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, tkent@chromium.org

[sof, 2016-05-24 07:10:15]

please take a look.

[tkent, 2016-05-24 07:13:51]

lgtm

[sof, 2016-05-24 07:16:32]

https://codereview.chromium.org/2004343002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/IntersectionObserverController.cpp
(left):

https://codereview.chromium.org/2004343002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/IntersectionObserverController.cpp:12:
typedef HeapVector<Member<IntersectionObserver>> IntersectionObserverVector;
now unused, forgot to remove this in ps#1.

[sof, 2016-05-24 07:16:42]

Description was changed from

==========
Remove unwanted copyToVector() uses.

If a collection is being cleared while creating an iterable
view of its current contents, swap in an empty collection
rather than copy out the current contents into a temporary
vector (by way of copyToVector().)

R=
BUG=
==========

to

==========
Remove unwanted copyToVector() uses.

If a collection is being cleared while creating an iterable
view of its current contents, swap in an empty collection
rather than copy out the current contents into a temporary
vector (by way of copyToVector().)

R=tkent
BUG=
==========

[sof, 2016-05-24 07:16:45]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2016-05-24 07:16:46]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2004343002/#ps20001
(title: "remove unused typedef")

[commit-bot: I haz the power, 2016-05-24 07:17:14]

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

[haraken, 2016-05-24 07:21:51]

LGTM

[commit-bot: I haz the power, 2016-05-24 08:45:21]

Description was changed from

==========
Remove unwanted copyToVector() uses.

If a collection is being cleared while creating an iterable
view of its current contents, swap in an empty collection
rather than copy out the current contents into a temporary
vector (by way of copyToVector().)

R=tkent
BUG=
==========

to

==========
Remove unwanted copyToVector() uses.

If a collection is being cleared while creating an iterable
view of its current contents, swap in an empty collection
rather than copy out the current contents into a temporary
vector (by way of copyToVector().)

R=tkent
BUG=
==========

[commit-bot: I haz the power, 2016-05-24 08:45:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-24 08:47:13]

Description was changed from

==========
Remove unwanted copyToVector() uses.

If a collection is being cleared while creating an iterable
view of its current contents, swap in an empty collection
rather than copy out the current contents into a temporary
vector (by way of copyToVector().)

R=tkent
BUG=
==========

to

==========
Remove unwanted copyToVector() uses.

If a collection is being cleared while creating an iterable
view of its current contents, swap in an empty collection
rather than copy out the current contents into a temporary
vector (by way of copyToVector().)

R=tkent
BUG=

Committed: https://crrev.com/b0423799d4c45060338e6d7ac6db3a284f238aac
Cr-Commit-Position: refs/heads/master@{#395547}
==========

[commit-bot: I haz the power, 2016-05-24 08:47:14]

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