[heap] Use size_t throughout MemoryAllocator

[heap] Use size_t throughout MemoryAllocator

BUG=chromium:652721
R=hpayer@chromium.org

Committed: https://crrev.com/8b682e1a5ad1fcd15f989b17001b0dbc832cc98f
Cr-Commit-Position: refs/heads/master@{#39986}

[Michael Lippautz, 2016-10-04 19:22:04]

Description was changed from

==========
[heap] Use size_t throughout MemoryAllocator

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

to

==========
[heap] Use size_t throughout MemoryAllocator

BUG=chromium:652721
==========

[Michael Lippautz, 2016-10-04 19:43:55]

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

[Michael Lippautz, 2016-10-04 19:43:56]

ptal

[Michael Lippautz, 2016-10-04 19:44:50]

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

https://codereview.chromium.org/2395563002/diff/40001/src/heap/spaces.h#newco...
src/heap/spaces.h:1240: static int CodePageGuardStartOffset();
Just moved up. size_t'ing these constants is an exercise for another CL.

[Michael Lippautz, 2016-10-04 19:44:59]

Description was changed from

==========
[heap] Use size_t throughout MemoryAllocator

BUG=chromium:652721
==========

to

==========
[heap] Use size_t throughout MemoryAllocator

BUG=chromium:652721
R=hpayer@chromium.org
==========

[Hannes Payer (out of office), 2016-10-05 07:38:01]

lgtm

[Michael Lippautz, 2016-10-05 07:39:10]

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

[commit-bot: I haz the power, 2016-10-05 07:39:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-05 08:06:57]

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

[commit-bot: I haz the power, 2016-10-05 08:06:58]

Dry run: This issue passed the CQ dry run.

[Michael Lippautz, 2016-10-05 08:07:46]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2016-10-05 08:07:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-05 08:13:07]

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

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

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/25623)

[Michael Lippautz, 2016-10-05 08:15:11]

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

[Michael Lippautz, 2016-10-05 08:15:12]

+Yang for regepx owner

[Yang, 2016-10-05 10:37:01]

On 2016/10/05 08:15:12, Michael Lippautz wrote:
> +Yang for regepx owner

lgtm.

[Michael Lippautz, 2016-10-05 10:38:14]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2016-10-05 10:38:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

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

Description was changed from

==========
[heap] Use size_t throughout MemoryAllocator

BUG=chromium:652721
R=hpayer@chromium.org
==========

to

==========
[heap] Use size_t throughout MemoryAllocator

BUG=chromium:652721
R=hpayer@chromium.org
==========

[commit-bot: I haz the power, 2016-10-05 10:40:19]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-05 10:40:41]

Description was changed from

==========
[heap] Use size_t throughout MemoryAllocator

BUG=chromium:652721
R=hpayer@chromium.org
==========

to

==========
[heap] Use size_t throughout MemoryAllocator

BUG=chromium:652721
R=hpayer@chromium.org

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

[commit-bot: I haz the power, 2016-10-05 10:40:42]

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