[heap] Reland "Prepare IncrementalMarking::VisitObject for concurrent marking."

[heap] Reland "Prepare IncrementalMarking::VisitObject for concurrent marking."

Currently the VisitObject function iterates the object and then colors
it black. This does not work well with concurrent marking. The function
should instead first try to mark the object black and iterate its body
only if the color transition succeeds.

BUG=chromium:694255
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_rel_ng

Review-Url: https://codereview.chromium.org/2863933002
Cr-Commit-Position: refs/heads/master@{#45139}
Committed: https://chromium.googlesource.com/v8/v8/+/02a7c020bd0ebc25dd1688795a730fb1ec1da4e3

[ulan, 2017-05-05 09:37:36]

Description was changed from

==========
[heap] Prepare IncrementalMarking::VisitObject for concurrent marking.

Currently the VisitObject function iterates the object and then colors
it black. This does not work well with concurrent marking. The function
should instead first try to mark the object black and iterate its body
only if the color transition succeeds.

BUG=chromium:694255
==========

to

==========
[heap] Reland "Prepare IncrementalMarking::VisitObject for concurrent marking."

Currently the VisitObject function iterates the object and then colors
it black. This does not work well with concurrent marking. The function
should instead first try to mark the object black and iterate its body
only if the color transition succeeds.

BUG=chromium:694255
==========

[ulan, 2017-05-05 09:38:23]

Description was changed from

==========
[heap] Reland "Prepare IncrementalMarking::VisitObject for concurrent marking."

Currently the VisitObject function iterates the object and then colors
it black. This does not work well with concurrent marking. The function
should instead first try to mark the object black and iterate its body
only if the color transition succeeds.

BUG=chromium:694255
==========

to

==========
[heap] Reland "Prepare IncrementalMarking::VisitObject for concurrent marking."

Currently the VisitObject function iterates the object and then colors
it black. This does not work well with concurrent marking. The function
should instead first try to mark the object black and iterate its body
only if the color transition succeeds.

BUG=chromium:694255
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_rel_ng
==========

[ulan, 2017-05-05 09:38:23]

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

[ulan, 2017-05-05 09:38:55]

Relanding without any changes from the original CL because the CL was revert due
to dependency with another CL.

[Michael Lippautz, 2017-05-05 09:39:24]

lgtm

[ulan, 2017-05-05 14:06:03]

The CQ bit was checked by ulan@chromium.org

[ulan, 2017-05-05 14:06:03]

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

[commit-bot: I haz the power, 2017-05-05 14:06:05]

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-05 14:56:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493993163173380,
"parent_rev": "b1d96fa1d410b2b7cfdff6131517dbf5cbfadc20", "commit_rev":
"02a7c020bd0ebc25dd1688795a730fb1ec1da4e3"}

[commit-bot: I haz the power, 2017-05-05 14:56:19]

Description was changed from

==========
[heap] Reland "Prepare IncrementalMarking::VisitObject for concurrent marking."

Currently the VisitObject function iterates the object and then colors
it black. This does not work well with concurrent marking. The function
should instead first try to mark the object black and iterate its body
only if the color transition succeeds.

BUG=chromium:694255
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_rel_ng
==========

to

==========
[heap] Reland "Prepare IncrementalMarking::VisitObject for concurrent marking."

Currently the VisitObject function iterates the object and then colors
it black. This does not work well with concurrent marking. The function
should instead first try to mark the object black and iterate its body
only if the color transition succeeds.

BUG=chromium:694255
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_rel_ng

Review-Url: https://codereview.chromium.org/2863933002
Cr-Commit-Position: refs/heads/master@{#45139}
Committed:
https://chromium.googlesource.com/v8/v8/+/02a7c020bd0ebc25dd1688795a730fb1ec1...
==========

[commit-bot: I haz the power, 2017-05-05 14:56:20]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/02a7c020bd0ebc25dd1688795a730fb1ec1...