Use the instance type to determine if an object is a promise.

Use the instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine if it is a promise rather than test whether the property promise_state_symbol is present.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel

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

[Franzi, 2016-06-24 09:47:35]

Description was changed from

==========
Use instance type to determine if object is a promise.

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

to

==========
Use instance type to determine if object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather then looking for the property promise_state_symbol. 

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

[Franzi, 2016-06-24 09:49:27]

Description was changed from

==========
Use instance type to determine if object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather then looking for the property promise_state_symbol. 

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

to

==========
Use instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather than test whether the property promise_state_symbol is
present. 

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

[Franzi, 2016-06-24 09:50:19]

Description was changed from

==========
Use instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather than test whether the property promise_state_symbol is
present. 

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

to

==========
Use the instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather than test whether the property promise_state_symbol is
present. 

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

[Franzi, 2016-06-24 09:50:20]

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

[Franzi, 2016-06-24 10:26:32]

Hi Toon, 

I think we can simplify IsPromise() as well. Please take a look. 

Thanks, 

Franzi

[Toon Verwaest, 2016-06-24 11:55:22]

Nice :) lgtm

[Franzi, 2016-06-27 08:41:57]

Description was changed from

==========
Use the instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather than test whether the property promise_state_symbol is
present. 

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

to

==========
Use the instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather than test whether the property promise_state_symbol is
present. 

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
==========

[Franzi, 2016-06-27 08:42:02]

The CQ bit was checked by franzih@chromium.org

[Franzi, 2016-06-27 08:42:03]

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

[commit-bot: I haz the power, 2016-06-27 08:42:13]

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

[commit-bot: I haz the power, 2016-06-27 08:42:14]

This CL has an open dependency (Issue 2090333002 Patch 180001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[Franzi, 2016-06-27 09:33:33]

The CQ bit was checked by franzih@chromium.org

[commit-bot: I haz the power, 2016-06-27 09:33:44]

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-06-27 09:35:14]

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

[commit-bot: I haz the power, 2016-06-27 09:35:15]

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

[Franzi, 2016-06-27 09:53:58]

The CQ bit was checked by franzih@chromium.org

[Franzi, 2016-06-27 09:53:58]

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

[commit-bot: I haz the power, 2016-06-27 09:54:02]

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-06-27 11:05:22]

Description was changed from

==========
Use the instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather than test whether the property promise_state_symbol is
present. 

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
==========

to

==========
Use the instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather than test whether the property promise_state_symbol is
present. 

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-06-27 11:05:24]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-06-27 11:08:08]

Description was changed from

==========
Use the instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather than test whether the property promise_state_symbol is
present. 

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
==========

to

==========
Use the instance type to determine if an object is a promise.

We can check if the instance type of an object is JS_PROMISE_TYPE to determine
if it is a promise rather than test whether the property promise_state_symbol is
present.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel

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

[commit-bot: I haz the power, 2016-06-27 11:08:09]

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