Add tests for GlobalHandles::IterateObjectGroups.

Add tests for GlobalHandles::IterateObjectGroups.

BUG=none
TEST=cctest/test-global-handles

Committed: http://code.google.com/p/v8/source/detail?r=14249

[marja, 2013-04-11 08:42:23]

ptal

[Michael Starzinger, 2013-04-12 09:12:32]

Looking good, just a few nits.

https://codereview.chromium.org/13952005/diff/2001/test/cctest/test-global-ha...
File test/cctest/test-global-handles.cc (right):

https://codereview.chromium.org/13952005/diff/2001/test/cctest/test-global-ha...
test/cctest/test-global-handles.cc:1: // Copyright 2012 the V8 project authors.
All rights reserved.
Should say 2013 here.

https://codereview.chromium.org/13952005/diff/2001/test/cctest/test-global-ha...
test/cctest/test-global-handles.cc:37: static void InitializeVM() {
Could we switch this to use CcTest::InitializeVM and CcTest::env instead?

https://codereview.chromium.org/13952005/diff/2001/test/cctest/test-global-ha...
test/cctest/test-global-handles.cc:119:
global_handles->Create(HEAP->AllocateFixedArray(1)->ToObjectChecked());
nit: Indentation is off.

[marja, 2013-04-12 09:33:04]

thanks for having a look!

https://codereview.chromium.org/13952005/diff/2001/test/cctest/test-global-ha...
File test/cctest/test-global-handles.cc (right):

https://codereview.chromium.org/13952005/diff/2001/test/cctest/test-global-ha...
test/cctest/test-global-handles.cc:1: // Copyright 2012 the V8 project authors.
All rights reserved.
On 2013/04/12 09:12:32, Michael Starzinger wrote:
> Should say 2013 here.

Done.

https://codereview.chromium.org/13952005/diff/2001/test/cctest/test-global-ha...
test/cctest/test-global-handles.cc:37: static void InitializeVM() {
On 2013/04/12 09:12:32, Michael Starzinger wrote:
> Could we switch this to use CcTest::InitializeVM and CcTest::env instead?

Done.

https://codereview.chromium.org/13952005/diff/2001/test/cctest/test-global-ha...
test/cctest/test-global-handles.cc:119:
global_handles->Create(HEAP->AllocateFixedArray(1)->ToObjectChecked());
On 2013/04/12 09:12:32, Michael Starzinger wrote:
> nit: Indentation is off.

Done.

[Michael Starzinger, 2013-04-12 09:39:34]

LGTM. I'll land this.

[Michael Starzinger, 2013-04-12 09:55:04]

Committed patchset #3 manually as r14249 (presubmit successful).