Reland "Prepare to enable in-object properties in subclasses on a case by case basis."

Reland "Prepare to enable in-object properties in subclasses on a case by case basis."

Original issue's description:
> Prepare to enable in-object properties in subclasses on a case by case basis.
>
> Minor cleanup in VisitorId selection.
>
> Committed: https://crrev.com/7c449a62edfc03aed84d94da323dcfe2b51a3600
> Cr-Commit-Position: refs/heads/master@{#32030}

This is a mostly clean reland.

Committed: https://crrev.com/71be4434305a7f2da543d5a4121de269efb4bdb4
Cr-Commit-Position: refs/heads/master@{#32148}

[Igor Sheludko, 2015-11-19 18:36:25]

Patchset #2 (id:20001) has been deleted

[Igor Sheludko, 2015-11-19 18:42:42]

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

[commit-bot: I haz the power, 2015-11-19 18:42:50]

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

[commit-bot: I haz the power, 2015-11-19 19:10:40]

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

[commit-bot: I haz the power, 2015-11-19 19:10:40]

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

[Igor Sheludko, 2015-11-20 07:12:51]

Patchset #2 (id:40001) has been deleted

[Igor Sheludko, 2015-11-20 07:12:57]

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

[commit-bot: I haz the power, 2015-11-20 07:13:04]

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

[commit-bot: I haz the power, 2015-11-20 07:50:26]

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

[commit-bot: I haz the power, 2015-11-20 07:50:27]

Dry run: This issue passed the CQ dry run.

[Igor Sheludko, 2015-11-20 11:28:36]

Patchset #2 (id:60001) has been deleted

[Igor Sheludko, 2015-11-20 11:28:48]

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

[commit-bot: I haz the power, 2015-11-20 11:28:51]

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

[Igor Sheludko, 2015-11-20 11:34:37]

Description was changed from

==========
Reland "Prepare to enable in-object properties in subclasses on a case by case
basis."

Original issue's description:
> Prepare to enable in-object properties in subclasses on a case by case basis.
>
> Minor cleanup in VisitorId selection.
>
> Committed: https://crrev.com/7c449a62edfc03aed84d94da323dcfe2b51a3600
> Cr-Commit-Position: refs/heads/master@{#32030}

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

to

==========
Reland "Prepare to enable in-object properties in subclasses on a case by case
basis."

Original issue's description:
> Prepare to enable in-object properties in subclasses on a case by case basis.
>
> Minor cleanup in VisitorId selection.
>
> Committed: https://crrev.com/7c449a62edfc03aed84d94da323dcfe2b51a3600
> Cr-Commit-Position: refs/heads/master@{#32030}

This is a mostly clean reland.
==========

[Igor Sheludko, 2015-11-20 11:34:50]

ishell@chromium.org changed reviewers:
+ verwaest@chromium.org

[Igor Sheludko, 2015-11-20 11:34:51]

PTAL

[ulan, 2015-11-20 11:43:04]

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

[ulan, 2015-11-20 11:43:05]

lgtm

[Toon Verwaest, 2015-11-20 11:52:08]

lgtm with nit

https://codereview.chromium.org/1459133002/diff/80001/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/1459133002/diff/80001/src/objects.cc#newcode1...
src/objects.cc:11976: case JS_PROMISE_TYPE:
JS_PROMISE_TYPE can be moved to the true part, since it used to be
JS_OBJECT_TYPE.

[Igor Sheludko, 2015-11-20 12:30:57]

https://codereview.chromium.org/1459133002/diff/80001/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/1459133002/diff/80001/src/objects.cc#newcode1...
src/objects.cc:11976: case JS_PROMISE_TYPE:
On 2015/11/20 11:52:08, Toon Verwaest wrote:
> JS_PROMISE_TYPE can be moved to the true part, since it used to be
> JS_OBJECT_TYPE.

Done.

[Igor Sheludko, 2015-11-20 13:15:28]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2015-11-20 13:15:29]

The patchset sent to the CQ was uploaded after l-g-t-m from ulan@chromium.org,
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/1459133002/#ps100001
(title: "Addressed comments")

[commit-bot: I haz the power, 2015-11-20 13:15:40]

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

[commit-bot: I haz the power, 2015-11-20 13:35:17]

Committed patchset #3 (id:100001)

[commit-bot: I haz the power, 2015-11-20 13:35:43]

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