[heap] Fix skip list for deserialized code objects.

[heap] Fix skip list for deserialized code objects.

R=hpayer@chromium.org

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

[Yang, 2016-03-18 07:27:54]

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

[commit-bot: I haz the power, 2016-03-18 07:28:00]

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

[commit-bot: I haz the power, 2016-03-18 07:44:56]

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

[commit-bot: I haz the power, 2016-03-18 07:44:57]

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

[Yang, 2016-03-18 10:24:00]

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

[commit-bot: I haz the power, 2016-03-18 10:24:13]

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

[Yang, 2016-03-18 10:25:03]

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

[commit-bot: I haz the power, 2016-03-18 10:25:11]

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

[commit-bot: I haz the power, 2016-03-18 10:47:42]

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

[commit-bot: I haz the power, 2016-03-18 10:47:43]

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

[Yang, 2016-03-18 12:19:59]

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

[commit-bot: I haz the power, 2016-03-18 12:20:02]

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

[commit-bot: I haz the power, 2016-03-18 12:44:43]

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

[commit-bot: I haz the power, 2016-03-18 12:44:44]

Dry run: This issue passed the CQ dry run.

[Hannes Payer (out of office), 2016-03-18 12:54:09]

LGTM, one comment nit

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

https://codereview.chromium.org/1816463002/diff/60001/src/heap/spaces.h#newco...
src/heap/spaces.h:2051: // failure object if not.
Add a warning there that if you pass in ignore_skip_list you have to make sure
that the skip list entry is created later on.

[Yang, 2016-03-18 14:34:26]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-03-18 14:34:26]

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/1816463002/#ps80001
(title: "added comment")

[commit-bot: I haz the power, 2016-03-18 14:34:34]

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

[commit-bot: I haz the power, 2016-03-18 14:57:17]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-03-18 14:58:02]

Description was changed from

==========
[heap] Fix skip list for deserialized code objects.

R=hpayer@chromium.org
==========

to

==========
[heap] Fix skip list for deserialized code objects.

R=hpayer@chromium.org

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

[commit-bot: I haz the power, 2016-03-18 14:58:03]

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