[async-await] Remove RejectPromiseNoDebugEvent

[async-await] Remove RejectPromiseNoDebugEvent

Just desugar directly into the runtime call instead.

BUG=v8:5639
Review-Url: https://codereview.chromium.org/2633353002
Cr-Commit-Position: refs/heads/master@{#42492}
Committed: https://chromium.googlesource.com/v8/v8/+/27edf11e04d0b318eb7530a738c74bd7c1884d5a

[jgruber, 2017-01-17 14:57:35]

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

[commit-bot: I haz the power, 2017-01-17 14:57:47]

Dry run: 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-01-17 15:44:59]

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

[commit-bot: I haz the power, 2017-01-17 15:44:59]

Dry run: This issue passed the CQ dry run.

[jgruber, 2017-01-17 15:57:04]

jgruber@chromium.org changed reviewers:
+ gsathya@chromium.org, littledan@chromium.org

[gsathya, 2017-01-17 16:00:55]

On 2017/01/17 15:57:05, jgruber wrote:

With this change isolate->promise_reject() is only called with debug_event=true.
Can you update the TF_BUILTIN(InternalPromiseReject) to take just two args and
always default to true? and refactor its use in api.cc as well.

[jgruber, 2017-01-18 12:47:30]

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

[commit-bot: I haz the power, 2017-01-18 12:47:37]

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

[jgruber, 2017-01-18 12:49:02]

On 2017/01/17 16:00:55, gsathya wrote:
> On 2017/01/17 15:57:05, jgruber wrote:
> 
> With this change isolate->promise_reject() is only called with
debug_event=true.
> Can you update the TF_BUILTIN(InternalPromiseReject) to take just two args and
> always default to true? and refactor its use in api.cc as well.

Took me a while to grok what was going on here since this changed so recently.
Refactored to desugar into a call to internal_promise_reject.

[commit-bot: I haz the power, 2017-01-18 13:15:34]

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

[commit-bot: I haz the power, 2017-01-18 13:15:35]

Dry run: This issue passed the CQ dry run.

[gsathya, 2017-01-18 18:59:31]

lgtm

[jgruber, 2017-01-19 08:36:46]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-01-19 08:36:50]

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-01-19 08:39:59]

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

[commit-bot: I haz the power, 2017-01-19 08:39:59]

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

[jgruber, 2017-01-19 08:45:45]

jgruber@chromium.org changed reviewers:
+ vogelheim@chromium.org, yangguo@chromium.org

[jgruber, 2017-01-19 08:45:45]

+yang for src/js
+vogelheim for parsing/

PTAL

[Yang, 2017-01-19 08:47:47]

On 2017/01/19 08:45:45, jgruber wrote:
> +yang for src/js
> +vogelheim for parsing/
> 
> PTAL

lgtm.

[vogelheim, 2017-01-19 08:48:20]

lgtm for src/parsing/..

[jgruber, 2017-01-19 08:48:56]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-01-19 08:49:00]

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-01-19 08:50:42]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484815736627360,
"parent_rev": "4ffe0850dbb238dca4c1e2d14fb2984442a1fa24", "commit_rev":
"27edf11e04d0b318eb7530a738c74bd7c1884d5a"}

[commit-bot: I haz the power, 2017-01-19 08:50:47]

Description was changed from

==========
[async-await] Remove RejectPromiseNoDebugEvent

Just desugar directly into the runtime call instead.

BUG=v8:5639
==========

to

==========
[async-await] Remove RejectPromiseNoDebugEvent

Just desugar directly into the runtime call instead.

BUG=v8:5639

Review-Url: https://codereview.chromium.org/2633353002
Cr-Commit-Position: refs/heads/master@{#42492}
Committed:
https://chromium.googlesource.com/v8/v8/+/27edf11e04d0b318eb7530a738c74bd7c18...
==========

[commit-bot: I haz the power, 2017-01-19 08:50:48]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/27edf11e04d0b318eb7530a738c74bd7c18...