Move tracking of ActiveScriptWrappables to V8PerIsolateData.

Move tracking of ActiveScriptWrappables to V8PerIsolateData.

(Active)ScriptWrappables are now traced per isolate by ScriptWrappableVisitor.
Keep the set tracking the live ActiveScriptWrappables alongside the isolate
instead, moving it out of a static thread-specific persistent set. Releasing
that set upon finalization becomes simpler; a more natural home for this
per-isolate piece of state.

R=
BUG=

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

[sof, 2016-05-11 09:51:39]

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

[sof, 2016-05-11 09:51:40]

please take a look.

[jochen (gone - plz use gerrit), 2016-05-11 09:56:04]

jochen@chromium.org changed reviewers:
+ hlopko@chromium.org

[jochen (gone - plz use gerrit), 2016-05-11 09:56:05]

+hlopko

[Marcel Hlopko, 2016-05-11 09:57:39]

lgtm

[haraken, 2016-05-11 10:56:17]

LGTM

[sof, 2016-05-11 13:43:36]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-05-11 13:43:58]

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

[commit-bot: I haz the power, 2016-05-11 14:28:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-11 14:30:09]

Description was changed from

==========
Move tracking of ActiveScriptWrappables to V8PerIsolateData.

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

to

==========
Move tracking of ActiveScriptWrappables to V8PerIsolateData.

R=
BUG=

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

[commit-bot: I haz the power, 2016-05-11 14:30:10]

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

[esprehn, 2016-05-11 23:40:16]

Can you put more explanation in these patches? This patch has no BUG and no
context, why was it made? What does it improve? Please write descriptive
changes. :)

[sof, 2016-05-12 05:06:48]

Description was changed from

==========
Move tracking of ActiveScriptWrappables to V8PerIsolateData.

R=
BUG=

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

to

==========
Move tracking of ActiveScriptWrappables to V8PerIsolateData.

(Active)ScriptWrappables are now traced per isolate by ScriptWrappableVisitor.
Keep the set tracking the live ActiveScriptWrappables alongside the isolate
instead, moving it out of a static thread-specific persistent set. Releasing
that set upon finalization becomes simpler; a more natural home for this
per-isolate piece of state.

R=
BUG=

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

[sof, 2016-05-12 05:13:55]

Added a (review) description.

This CL sprang out of https://codereview.chromium.org/1962113003/ (which again
sprang out of observations around
https://codereview.chromium.org/1805843002/#msg36 )