[heap] Process live weak cells directly in the marking visitor.

[heap] Process live weak cells directly in the marking visitor.

This reduces the pause time of weak cells processing during a full GC.

BUG=

Committed: https://crrev.com/0bfb773de2e165983deb020c35c7dc97ca164975
Cr-Commit-Position: refs/heads/master@{#30864}

[Hannes Payer (out of office), 2015-09-22 11:19:30]

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

[ulan, 2015-09-22 11:30:46]

lgtm if it is a bottleneck in a benchmark.

https://codereview.chromium.org/1363553002/diff/1/src/heap/objects-visiting-i...
File src/heap/objects-visiting-inl.h (right):

https://codereview.chromium.org/1363553002/diff/1/src/heap/objects-visiting-i...
src/heap/objects-visiting-inl.h:358: if (MarkCompactCollector::IsMarked(value))
{
Please add a comment explaining that this is done for performance and the
general case is processed in the "else" branch.

[Hannes Payer (out of office), 2015-09-22 13:22:57]

It is a bottleneck on many websites.

https://codereview.chromium.org/1363553002/diff/1/src/heap/objects-visiting-i...
File src/heap/objects-visiting-inl.h (right):

https://codereview.chromium.org/1363553002/diff/1/src/heap/objects-visiting-i...
src/heap/objects-visiting-inl.h:358: if (MarkCompactCollector::IsMarked(value))
{
On 2015/09/22 11:30:46, ulan wrote:
> Please add a comment explaining that this is done for performance and the
> general case is processed in the "else" branch.

Done.

[Hannes Payer (out of office), 2015-09-22 13:23:05]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2015-09-22 13:23:05]

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

[commit-bot: I haz the power, 2015-09-22 13:23:12]

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

[commit-bot: I haz the power, 2015-09-22 13:48:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-22 13:48:29]

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