Add RegisterExternallyReferencedObject to PersistenValueMap

Add RegisterExternallyReferencedObject to PersistenValueMap

PersistentValueMap is used to hold per-world wrappers in the blink. Currently,
when we trace wrappers, we visit wrappers in all worlds via this PersistentValueMap. This cl introduces convenient (and faster) way of registering these external references.

BUG=468240
LOG=no

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

[Marcel Hlopko, 2016-04-14 13:18:57]

Description was changed from

==========
Add RegisterExternallyReferencedObject to PersistenValueMap

PersistentValueMap is used to hold per-world wrappers in the blink. Currently,
when we trace wrappers, we visit wrappers in all worlds. This cl introduces
convenient (and faster) way of registering these external references.

BUG=468240
LOG=no
==========

to

==========
Add RegisterExternallyReferencedObject to PersistenValueMap

PersistentValueMap is used to hold per-world wrappers in the blink. Currently,
when we trace wrappers, we visit wrappers in all worlds via this
PersistentValueMap. This cl introduces convenient (and faster) way of
registering these external references.

BUG=468240
LOG=no
==========

[Marcel Hlopko, 2016-04-14 13:18:57]

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

[Marcel Hlopko, 2016-04-14 13:27:05]

ptal

[jochen (gone - plz use gerrit), 2016-04-14 14:28:09]

https://codereview.chromium.org/1883043003/diff/40001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/1883043003/diff/40001/include/v8.h#newcode6632
include/v8.h:6632: static void
RegisterExternallyReferencedObject(internal::Object** object,
this should stay private. Can you make the map a friend of V8 instead?

[Marcel Hlopko, 2016-04-14 14:50:42]

https://codereview.chromium.org/1883043003/diff/40001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/1883043003/diff/40001/include/v8.h#newcode6632
include/v8.h:6632: static void
RegisterExternallyReferencedObject(internal::Object** object,
Done.

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

lgtm

[Marcel Hlopko, 2016-04-15 11:56:41]

The CQ bit was checked by hlopko@chromium.org

[commit-bot: I haz the power, 2016-04-15 11:56:47]

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

[commit-bot: I haz the power, 2016-04-15 12:22:24]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-04-15 12:23:58]

Description was changed from

==========
Add RegisterExternallyReferencedObject to PersistenValueMap

PersistentValueMap is used to hold per-world wrappers in the blink. Currently,
when we trace wrappers, we visit wrappers in all worlds via this
PersistentValueMap. This cl introduces convenient (and faster) way of
registering these external references.

BUG=468240
LOG=no
==========

to

==========
Add RegisterExternallyReferencedObject to PersistenValueMap

PersistentValueMap is used to hold per-world wrappers in the blink. Currently,
when we trace wrappers, we visit wrappers in all worlds via this
PersistentValueMap. This cl introduces convenient (and faster) way of
registering these external references.

BUG=468240
LOG=no

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

[commit-bot: I haz the power, 2016-04-15 12:23:59]

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