[heap] Remove unecessary marking bit check in RecordWriteSlow.

[heap] Remove unecessary marking bit check in RecordWriteSlow.

BUG=

Committed: https://crrev.com/45cb28409d4f43b487c012befe7f355ca099ef09
Cr-Commit-Position: refs/heads/master@{#31766}

[Hannes Payer (out of office), 2015-11-03 22:51:15]

https://codereview.chromium.org/1430943004/diff/1/src/heap/incremental-markin...
File src/heap/incremental-marking.cc (left):

https://codereview.chromium.org/1430943004/diff/1/src/heap/incremental-markin...
src/heap/incremental-marking.cc:82: if (BaseRecordWrite(obj, slot, value) &&
slot != NULL) {
BaseRecordWrite already tells us if we have to record the slot, i.e. checks the
color of obj. No need to do it again.

[Hannes Payer (out of office), 2015-11-03 22:51:24]

Description was changed from

==========
[heap] Remove unecessary marking bit check in RecordWriteSlow.

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

to

==========
[heap] Remove unecessary marking bit check in RecordWriteSlow.

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

[Hannes Payer (out of office), 2015-11-03 22:51:24]

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

[Michael Lippautz, 2015-11-03 22:54:25]

lgtm

https://codereview.chromium.org/1430943004/diff/1/src/heap/incremental-markin...
File src/heap/incremental-marking.cc (left):

https://codereview.chromium.org/1430943004/diff/1/src/heap/incremental-markin...
src/heap/incremental-marking.cc:82: if (BaseRecordWrite(obj, slot, value) &&
slot != NULL) {
On 2015/11/03 22:51:14, Hannes Payer wrote:
> BaseRecordWrite already tells us if we have to record the slot, i.e. checks
the
> color of obj. No need to do it again.

Acknowledged.

[Hannes Payer (out of office), 2015-11-04 00:27:21]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2015-11-04 00:27:31]

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

[commit-bot: I haz the power, 2015-11-04 00:28:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-04 00:29:22]

Patchset 1 (id:??) landed as
https://crrev.com/45cb28409d4f43b487c012befe7f355ca099ef09
Cr-Commit-Position: refs/heads/master@{#31766}