[heap] Sort declarations for MemoryChunk.

[heap] Sort declarations for MemoryChunk.

R=hpayer@chromium.org
BUG=

Committed: https://crrev.com/41b8e10e1ac0dac50384bc9ef1d141fae7e5c67b
Cr-Commit-Position: refs/heads/master@{#30850}

[Michael Lippautz, 2015-09-21 09:39:32]

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

[Michael Lippautz, 2015-09-21 09:39:32]

Sorting MemoryChunk (+ prev and next pointers are now AtomicValue's)

[Hannes Payer (out of office), 2015-09-21 12:58:49]

LGTM. It would be nice to check which methods do not have to be public, but
currently are.

https://codereview.chromium.org/1360553003/diff/1/src/heap/spaces.h
File src/heap/spaces.h (right):

https://codereview.chromium.org/1360553003/diff/1/src/heap/spaces.h#newcode635
src/heap/spaces.h:635: //
---------------------------------------------------------------------
The // ----- line doesn't fit.

[Michael Lippautz, 2015-09-21 13:38:19]

Moved one method to private section as all the others are unfortunately in use
by other components. Will take a closer look later.

https://codereview.chromium.org/1360553003/diff/1/src/heap/spaces.h
File src/heap/spaces.h (right):

https://codereview.chromium.org/1360553003/diff/1/src/heap/spaces.h#newcode635
src/heap/spaces.h:635: //
---------------------------------------------------------------------
On 2015/09/21 12:58:48, Hannes Payer wrote:
> The // ----- line doesn't fit.

Done.

[Michael Lippautz, 2015-09-21 13:38:23]

The CQ bit was checked by mlippautz@chromium.org

[Michael Lippautz, 2015-09-21 13:38:24]

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/1360553003/#ps20001
(title: "Addressed comment")

[commit-bot: I haz the power, 2015-09-21 13:38:26]

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

[commit-bot: I haz the power, 2015-09-21 14:03:48]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-21 14:04:07]

Patchset 2 (id:??) landed as
https://crrev.com/41b8e10e1ac0dac50384bc9ef1d141fae7e5c67b
Cr-Commit-Position: refs/heads/master@{#30850}