[debugger] Ensure debug listeners do not throw

[debugger] Ensure debug listeners do not throw

This exposes a couple of broken tests that used to silently throw within
the listener. Mark these as failing for now

BUG=v8:5330, v8:5581
Committed: https://crrev.com/16310b2e3933fa2fd5dd94036280eee2e0771ced
Cr-Commit-Position: refs/heads/master@{#40672}

[jgruber, 2016-10-28 08:54:47]

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

[commit-bot: I haz the power, 2016-10-28 08:54:56]

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, 2016-10-28 09:14:28]

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

[commit-bot: I haz the power, 2016-10-28 09:14:28]

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

[jgruber, 2016-10-28 09:58:36]

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

[commit-bot: I haz the power, 2016-10-28 09:58:44]

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

[jgruber, 2016-10-28 09:59:18]

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

[commit-bot: I haz the power, 2016-10-28 09:59:25]

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

[jgruber, 2016-10-28 10:02:11]

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

[jgruber, 2016-10-28 10:02:12]

We need to use CHECKs so that these tests also fail on release builds without
dcheck_always_on. This could cause crashes for embedders with listeners that may
throw. Do you think this might be an issue?

[commit-bot: I haz the power, 2016-10-28 10:30:52]

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

[commit-bot: I haz the power, 2016-10-28 10:30:53]

Dry run: This issue passed the CQ dry run.

[Yang, 2016-10-31 12:59:52]

On 2016/10/28 10:02:12, jgruber wrote:
> We need to use CHECKs so that these tests also fail on release builds without
> dcheck_always_on. This could cause crashes for embedders with listeners that
may
> throw. Do you think this might be an issue?

lgtm!

[jgruber, 2016-10-31 14:27:43]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2016-10-31 14:27:48]

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-10-31 14:56:12]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-17 22:18:07]

Description was changed from

==========
[debugger] Ensure debug listeners do not throw

This exposes a couple of broken tests that used to silently throw within
the listener. Mark these as failing for now

BUG=v8:5330, v8:5581
==========

to

==========
[debugger] Ensure debug listeners do not throw

This exposes a couple of broken tests that used to silently throw within
the listener. Mark these as failing for now

BUG=v8:5330, v8:5581

Committed: https://crrev.com/16310b2e3933fa2fd5dd94036280eee2e0771ced
Cr-Commit-Position: refs/heads/master@{#40672}
==========

[commit-bot: I haz the power, 2016-11-17 22:18:08]

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