[heap] Remove separate constant for newspace page allocatable size

[heap] Remove separate constant for newspace page allocatable size

* New and old space pages have the same allocatable memory size
* Enforce declaration order in NewSpacePage

BUG=chromium:581412
LOG=N

Committed: https://crrev.com/ef6585d8282d7437034de7a97695e2787ae29daa
Cr-Commit-Position: refs/heads/master@{#34851}

[Michael Lippautz, 2016-03-17 10:13:07]

Description was changed from

==========
[heap] Align size of allocatable memory in new space pages with old space.

BUG=chromium:581412
LOG=N
==========

to

==========
[heap] Remove separate constant for newspace page allocatable size

* New and old space pages have the same allocatable memory size
* Enforce declaration order in NewSpacePage



BUG=chromium:581412
LOG=N
==========

[Michael Lippautz, 2016-03-17 10:13:13]

Description was changed from

==========
[heap] Remove separate constant for newspace page allocatable size

* New and old space pages have the same allocatable memory size
* Enforce declaration order in NewSpacePage



BUG=chromium:581412
LOG=N
==========

to

==========
[heap] Remove separate constant for newspace page allocatable size

* New and old space pages have the same allocatable memory size
* Enforce declaration order in NewSpacePage

BUG=chromium:581412
LOG=N
==========

[Michael Lippautz, 2016-03-17 10:13:42]

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

[Michael Lippautz, 2016-03-17 10:13:43]

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

https://codereview.chromium.org/1809863003/diff/1/src/heap/spaces.h#oldcode2207
src/heap/spaces.h:2207: static const int kAreaSize = Page::kAllocatableMemory;
Now available on MemoryChunk since the allocatable area is the same.

https://codereview.chromium.org/1809863003/diff/1/src/heap/spaces.h#oldcode2234
src/heap/spaces.h:2234: Address address() { return
reinterpret_cast<Address>(this); }
dead code

[Michael Lippautz, 2016-03-17 10:28:17]

The CQ bit was checked by mlippautz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-17 10:28:23]

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

[ulan, 2016-03-17 10:59:05]

lgtm

[commit-bot: I haz the power, 2016-03-17 11:00:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-17 11:00:32]

Dry run: This issue passed the CQ dry run.

[Michael Lippautz, 2016-03-17 11:01:52]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2016-03-17 11:02:03]

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

[commit-bot: I haz the power, 2016-03-17 11:03:45]

Description was changed from

==========
[heap] Remove separate constant for newspace page allocatable size

* New and old space pages have the same allocatable memory size
* Enforce declaration order in NewSpacePage

BUG=chromium:581412
LOG=N
==========

to

==========
[heap] Remove separate constant for newspace page allocatable size

* New and old space pages have the same allocatable memory size
* Enforce declaration order in NewSpacePage

BUG=chromium:581412
LOG=N
==========

[commit-bot: I haz the power, 2016-03-17 11:03:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-17 11:04:19]

Description was changed from

==========
[heap] Remove separate constant for newspace page allocatable size

* New and old space pages have the same allocatable memory size
* Enforce declaration order in NewSpacePage

BUG=chromium:581412
LOG=N
==========

to

==========
[heap] Remove separate constant for newspace page allocatable size

* New and old space pages have the same allocatable memory size
* Enforce declaration order in NewSpacePage

BUG=chromium:581412
LOG=N

Committed: https://crrev.com/ef6585d8282d7437034de7a97695e2787ae29daa
Cr-Commit-Position: refs/heads/master@{#34851}
==========

[commit-bot: I haz the power, 2016-03-17 11:04:20]

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