Record slots in large objects.

Record slots in large objects.

BUG=

Committed: https://crrev.com/43f3303838e6343c1b81da06124f34ca531c0382
Cr-Commit-Position: refs/heads/master@{#30299}

[Hannes Payer (out of office), 2015-08-20 11:08:28]

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

[Michael Lippautz, 2015-08-20 11:14:39]

lgtm % nits

https://codereview.chromium.org/1296713007/diff/40001/test/cctest/cctest.h
File test/cctest/cctest.h (right):

https://codereview.chromium.org/1296713007/diff/40001/test/cctest/cctest.h#ne...
test/cctest/cctest.h:559: bool complete = true) {
nit: naming

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

https://codereview.chromium.org/1296713007/diff/40001/test/cctest/test-heap.c...
test/cctest/test-heap.cc:4661: FLAG_manual_evacuation_candidates_selection =
true;
nit: Can we set this before VM initialization?

[Hannes Payer (out of office), 2015-08-20 11:39:17]

https://codereview.chromium.org/1296713007/diff/40001/test/cctest/cctest.h
File test/cctest/cctest.h (right):

https://codereview.chromium.org/1296713007/diff/40001/test/cctest/cctest.h#ne...
test/cctest/cctest.h:559: bool complete = true) {
On 2015/08/20 11:14:39, Michael Lippautz wrote:
> nit: naming

Done.

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

https://codereview.chromium.org/1296713007/diff/40001/test/cctest/test-heap.c...
test/cctest/test-heap.cc:4661: FLAG_manual_evacuation_candidates_selection =
true;
On 2015/08/20 11:14:39, Michael Lippautz wrote:
> nit: Can we set this before VM initialization?

Done.

[Michael Lippautz, 2015-08-20 12:13:26]

On 2015/08/20 11:39:17, Hannes Payer wrote:
> https://codereview.chromium.org/1296713007/diff/40001/test/cctest/cctest.h
> File test/cctest/cctest.h (right):
> 
>
https://codereview.chromium.org/1296713007/diff/40001/test/cctest/cctest.h#ne...
> test/cctest/cctest.h:559: bool complete = true) {
> On 2015/08/20 11:14:39, Michael Lippautz wrote:
> > nit: naming
> 
> Done.
> 
> https://codereview.chromium.org/1296713007/diff/40001/test/cctest/test-heap.cc
> File test/cctest/test-heap.cc (right):
> 
>
https://codereview.chromium.org/1296713007/diff/40001/test/cctest/test-heap.c...
> test/cctest/test-heap.cc:4661: FLAG_manual_evacuation_candidates_selection =
> true;
> On 2015/08/20 11:14:39, Michael Lippautz wrote:
> > nit: Can we set this before VM initialization?
> 
> Done.

still lgtm but this seems risky

[Hannes Payer (out of office), 2015-08-21 10:41:55]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2015-08-21 10:42:08]

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

[commit-bot: I haz the power, 2015-08-21 11:41:50]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-08-21 11:42:01]

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