[heap] Report newly found wrappers after deserialization

[heap] Report newly found wrappers after deserialization

These wrappers wouldn't be found by the marker otherwise and are only
reported upon the next marking step or GC which potentially is already
too late; the embedder could've reclaimed those objects already.

BUG=chromium:717480
Review-Url: https://codereview.chromium.org/2860753003
Cr-Commit-Position: refs/heads/master@{#45094}
Committed: https://chromium.googlesource.com/v8/v8/+/2ec36b675e16693891986c10665b0e126defa10e

[Michael Lippautz, 2017-05-03 16:39:24]

Description was changed from

==========
[heap] Report newly found wrappers after deserialization

These wrappers wouldn't be found by the marker otherwise and are only
reported upon the next marking step or GC which potentially is already
too late; the embedder could've reclaimed those objects already.

BUG=
==========

to

==========
[heap] Report newly found wrappers after deserialization

These wrappers wouldn't be found by the marker otherwise and are only
reported upon the next marking step or GC which potentially is already
too late; the embedder could've reclaimed those objects already.

BUG=chromium:717480
==========

[Michael Lippautz, 2017-05-03 16:39:25]

mlippautz@chromium.org changed reviewers:
+ hpayer@chromium.org

[Michael Lippautz, 2017-05-03 16:40:35]

ptal; we already have DCHECKs that there are no cached wrappers left behind and
I am investigating for better spots to check that property.

[Michael Lippautz, 2017-05-03 16:40:45]

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

[commit-bot: I haz the power, 2017-05-03 16:40:47]

Dry run: 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, 2017-05-03 17:10:10]

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

[commit-bot: I haz the power, 2017-05-03 17:10:11]

Dry run: This issue passed the CQ dry run.

[Hannes Payer (out of office), 2017-05-04 12:46:27]

lgtm

[Michael Lippautz, 2017-05-04 13:08:35]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2017-05-04 13:08:39]

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, 2017-05-04 13:10:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493903315791750, "parent_rev":
"ca8b120f970e37714755663875bb45911654aac8", "commit_rev":
"2ec36b675e16693891986c10665b0e126defa10e"}

[commit-bot: I haz the power, 2017-05-04 13:10:39]

Description was changed from

==========
[heap] Report newly found wrappers after deserialization

These wrappers wouldn't be found by the marker otherwise and are only
reported upon the next marking step or GC which potentially is already
too late; the embedder could've reclaimed those objects already.

BUG=chromium:717480
==========

to

==========
[heap] Report newly found wrappers after deserialization

These wrappers wouldn't be found by the marker otherwise and are only
reported upon the next marking step or GC which potentially is already
too late; the embedder could've reclaimed those objects already.

BUG=chromium:717480

Review-Url: https://codereview.chromium.org/2860753003
Cr-Commit-Position: refs/heads/master@{#45094}
Committed:
https://chromium.googlesource.com/v8/v8/+/2ec36b675e16693891986c10665b0e126de...
==========

[commit-bot: I haz the power, 2017-05-04 13:10:40]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/2ec36b675e16693891986c10665b0e126de...