[heap] Bundle platform-specific constants for CodeRange

[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object (it might be
  invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller doesn't need to be
  aware of it

R=ulan@chromium.org

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

[Michael Lippautz, 2016-05-19 15:05:22]

Patchset #2 (id:20001) has been deleted

[Michael Lippautz, 2016-05-19 15:05:23]

Patchset #1 (id:1) has been deleted

[Michael Lippautz, 2016-05-19 15:08:20]

Patchset #2 (id:60001) has been deleted

[Michael Lippautz, 2016-05-19 15:08:23]

Patchset #1 (id:40001) has been deleted

[Michael Lippautz, 2016-05-19 15:09:19]

Description was changed from

==========
[heap] Bundle platform-specific constants for CodeRange

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

to

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object
 (it might be invalid, but it's there).

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

[Michael Lippautz, 2016-05-19 15:10:02]

Description was changed from

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object
 (it might be invalid, but it's there).

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

to

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object
  (it might be invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller
  doesn't need to be aware of it

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

[Michael Lippautz, 2016-05-19 15:10:16]

Description was changed from

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object
  (it might be invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller
  doesn't need to be aware of it

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

to

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object (it might be
  invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller doesn't need to be
  aware of it

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

[Michael Lippautz, 2016-05-19 15:11:02]

Description was changed from

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object (it might be
  invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller doesn't need to be
  aware of it

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

to

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object (it might be
  invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller doesn't need to be
  aware of it

R=ulan@chromium.org
==========

[Michael Lippautz, 2016-05-19 15:11:02]

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

[Michael Lippautz, 2016-05-19 15:11:11]

ptal

[ulan, 2016-05-19 15:19:18]

lgtm

[Michael Lippautz, 2016-05-19 15:20:18]

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

[commit-bot: I haz the power, 2016-05-19 15:20:25]

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

[commit-bot: I haz the power, 2016-05-19 15:28:59]

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

[commit-bot: I haz the power, 2016-05-19 15:29:00]

Dry run: Try jobs failed on following builders:
  v8_win64_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel_ng/builds/7544)

[Michael Lippautz, 2016-05-19 15:45:13]

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

[commit-bot: I haz the power, 2016-05-19 15:45:20]

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

[commit-bot: I haz the power, 2016-05-19 16:13:07]

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

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

Dry run: Try jobs failed on following builders:
  v8_win_nosnap_shared_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_rel_ng...)
  v8_win_nosnap_shared_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_rel_ng...)

[Michael Lippautz, 2016-05-19 16:54:54]

Patchset #3 (id:120001) has been deleted

[Michael Lippautz, 2016-05-19 16:55:09]

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

[commit-bot: I haz the power, 2016-05-19 16:55:19]

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

[commit-bot: I haz the power, 2016-05-19 17:18:17]

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

[commit-bot: I haz the power, 2016-05-19 17:18:18]

Dry run: Try jobs failed on following builders:
  v8_linux64_asan_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  v8_linux64_asan_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng_trig...)

[Michael Lippautz, 2016-05-19 17:20:32]

Patchset #4 (id:160001) has been deleted

[Michael Lippautz, 2016-05-19 17:20:45]

The CQ bit was checked by mlippautz@chromium.org

[Michael Lippautz, 2016-05-19 17:20:45]

The patchset sent to the CQ was uploaded after l-g-t-m from ulan@chromium.org
Link to the patchset: https://codereview.chromium.org/1991253002/#ps180001
(title: " ")

[commit-bot: I haz the power, 2016-05-19 17:20:57]

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

[commit-bot: I haz the power, 2016-05-19 17:54:05]

Description was changed from

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object (it might be
  invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller doesn't need to be
  aware of it

R=ulan@chromium.org
==========

to

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object (it might be
  invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller doesn't need to be
  aware of it

R=ulan@chromium.org
==========

[commit-bot: I haz the power, 2016-05-19 17:54:06]

Committed patchset #4 (id:180001)

[commit-bot: I haz the power, 2016-05-19 17:54:52]

Description was changed from

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object (it might be
  invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller doesn't need to be
  aware of it

R=ulan@chromium.org
==========

to

==========
[heap] Bundle platform-specific constants for CodeRange

- Removing null checks as we always have the CodeRange object (it might be
  invalid, but it's there)
- Account for reserved area (Win64) in SetUp so we the caller doesn't need to be
  aware of it

R=ulan@chromium.org

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

[commit-bot: I haz the power, 2016-05-19 17:54:53]

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