[promises] cleanup default promise handlers

[promises] cleanup default promise handlers

Use private symbols to mark default promise handler, instead of calling out to default 
handlers defined in JS. We check for this symbol in PromiseHandle and perform the 
appropriate behavior as the default handlers.

Catch prediction logic is updated to account for a symbol.

BUG=v8:5343
Review-Url: https://codereview.chromium.org/2695593002
Cr-Commit-Position: refs/heads/master@{#43135}
Committed: https://chromium.googlesource.com/v8/v8/+/31bc17f006f11e7699ff1f3ba86e3d580a707652

[gsathya, 2017-02-13 05:55:37]

Description was changed from

==========
[promises] cleanup default promise handlers

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

to

==========
[promises] cleanup default promise handlers

Use private symbols to mark default promise handler, instead of calling out to
default handlers defined in JS. We check for this symbol in PromiseHandle and
perform the appropriate behavior as the default handlers.

BUG=v8:5343
==========

[gsathya, 2017-02-13 05:56:49]

Description was changed from

==========
[promises] cleanup default promise handlers

Use private symbols to mark default promise handler, instead of calling out to
default handlers defined in JS. We check for this symbol in PromiseHandle and
perform the appropriate behavior as the default handlers.

BUG=v8:5343
==========

to

==========
[promises] cleanup default promise handlers

Use private symbols to mark default promise handler, instead of calling out to
default 
handlers defined in JS. We check for this symbol in PromiseHandle and perform
the 
appropriate behavior as the default handlers.

Catch prediction logic is updated to account for a symbol.

BUG=v8:5343
==========

[gsathya, 2017-02-13 06:00:59]

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

[commit-bot: I haz the power, 2017-02-13 06:01:05]

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

[gsathya, 2017-02-13 06:03:42]

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

[commit-bot: I haz the power, 2017-02-13 06:03:52]

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

[gsathya, 2017-02-13 06:06:03]

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

[commit-bot: I haz the power, 2017-02-13 06:06:12]

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

[gsathya, 2017-02-13 06:06:30]

gsathya@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Benedikt Meurer, 2017-02-13 06:09:29]

Nice, LGTM!

[commit-bot: I haz the power, 2017-02-13 06:28:42]

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

[commit-bot: I haz the power, 2017-02-13 06:28:43]

Dry run: This issue passed the CQ dry run.

[gsathya, 2017-02-13 06:29:29]

The CQ bit was checked by gsathya@chromium.org

[commit-bot: I haz the power, 2017-02-13 06:29:32]

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, 2017-02-13 06:31:12]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1486967369534410,
"parent_rev": "299f834058739f53eb1283516b95da48fd13b58d", "commit_rev":
"31bc17f006f11e7699ff1f3ba86e3d580a707652"}

[commit-bot: I haz the power, 2017-02-13 06:31:17]

Description was changed from

==========
[promises] cleanup default promise handlers

Use private symbols to mark default promise handler, instead of calling out to
default 
handlers defined in JS. We check for this symbol in PromiseHandle and perform
the 
appropriate behavior as the default handlers.

Catch prediction logic is updated to account for a symbol.

BUG=v8:5343
==========

to

==========
[promises] cleanup default promise handlers

Use private symbols to mark default promise handler, instead of calling out to
default 
handlers defined in JS. We check for this symbol in PromiseHandle and perform
the 
appropriate behavior as the default handlers.

Catch prediction logic is updated to account for a symbol.

BUG=v8:5343

Review-Url: https://codereview.chromium.org/2695593002
Cr-Commit-Position: refs/heads/master@{#43135}
Committed:
https://chromium.googlesource.com/v8/v8/+/31bc17f006f11e7699ff1f3ba86e3d580a7...
==========

[commit-bot: I haz the power, 2017-02-13 06:31:18]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/v8/v8/+/31bc17f006f11e7699ff1f3ba86e3d580a7...