[heap] Add the free remainder of a black page to the free list.

[heap] Add the free remainder of a black page to the free list.

BUG=chromium:615489
Committed: https://crrev.com/8c6fedd653b2f1506feafd098e17316b86089b44
Cr-Commit-Position: refs/heads/master@{#36587}

[ulan, 2016-05-30 09:22:12]

Description was changed from

==========
Regression test of live object counter bug with black allocation.

This also includes a hacky fix.

TODO: make proper fix before landing.

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

to

==========
[heap] Add the free remainder of a black page to free list.

BUG=chromium:615489
==========

[ulan, 2016-05-30 09:22:24]

Description was changed from

==========
[heap] Add the free remainder of a black page to free list.

BUG=chromium:615489
==========

to

==========
[heap] Add the free remainder of a black page to the free list.

BUG=chromium:615489
==========

[ulan, 2016-05-30 09:23:19]

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

[ulan, 2016-05-30 09:23:19]

ptal

[Hannes Payer (out of office), 2016-05-30 11:27:10]

Cool, thanks for fixing! LGTM

https://codereview.chromium.org/2020743002/diff/40001/test/cctest/heap/test-h...
File test/cctest/heap/test-heap.cc (right):

https://codereview.chromium.org/2020743002/diff/40001/test/cctest/heap/test-h...
test/cctest/heap/test-heap.cc:6769: TEST(MyTest) {
I know, it's your test. But let's rename it to something more descriptive ;)

[ulan, 2016-05-30 12:41:08]

https://codereview.chromium.org/2020743002/diff/40001/test/cctest/heap/test-h...
File test/cctest/heap/test-heap.cc (right):

https://codereview.chromium.org/2020743002/diff/40001/test/cctest/heap/test-h...
test/cctest/heap/test-heap.cc:6769: TEST(MyTest) {
On 2016/05/30 11:27:10, Hannes Payer wrote:
> I know, it's your test. But let's rename it to something more descriptive ;)

good catch :)

[ulan, 2016-05-30 12:41:18]

The CQ bit was checked by ulan@chromium.org

[ulan, 2016-05-30 12:41:19]

The patchset sent to the CQ was uploaded after l-g-t-m from hpayer@chromium.org
Link to the patchset: https://codereview.chromium.org/2020743002/#ps60001
(title: "Rename test")

[commit-bot: I haz the power, 2016-05-30 12:41:23]

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

[commit-bot: I haz the power, 2016-05-30 13:09:33]

Description was changed from

==========
[heap] Add the free remainder of a black page to the free list.

BUG=chromium:615489
==========

to

==========
[heap] Add the free remainder of a black page to the free list.

BUG=chromium:615489
==========

[commit-bot: I haz the power, 2016-05-30 13:09:34]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-30 13:10:17]

Description was changed from

==========
[heap] Add the free remainder of a black page to the free list.

BUG=chromium:615489
==========

to

==========
[heap] Add the free remainder of a black page to the free list.

BUG=chromium:615489

Committed: https://crrev.com/8c6fedd653b2f1506feafd098e17316b86089b44
Cr-Commit-Position: refs/heads/master@{#36587}
==========

[commit-bot: I haz the power, 2016-05-30 13:10:18]

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