[debug] remove JSON debug protocol related tests.

[debug] remove JSON debug protocol related tests.

In most cases we can use the debug event listener as alternative.
Multithreaded tests are obsolete.

R=jgruber@chromium.org
BUG=v8:5510
Committed: https://crrev.com/81a64aa5488602e4643f1ecc3beeddb8828a853c
Cr-Commit-Position: refs/heads/master@{#41256}

[Yang, 2016-11-24 09:22:43]

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

[commit-bot: I haz the power, 2016-11-24 09:22:48]

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-11-24 09:28:58]

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

[commit-bot: I haz the power, 2016-11-24 09:28:59]

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...)

[Yang, 2016-11-24 09:37:32]

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

[commit-bot: I haz the power, 2016-11-24 09:37:41]

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

[jgruber, 2016-11-24 09:53:08]

lgtm

https://codereview.chromium.org/2529883002/diff/20001/include/v8-debug.h
File include/v8-debug.h (right):

https://codereview.chromium.org/2529883002/diff/20001/include/v8-debug.h#newc...
include/v8-debug.h:173: V8_DEPRECATED("No longer supported",
👍

https://codereview.chromium.org/2529883002/diff/20001/test/inspector/runtime/...
File test/inspector/runtime/evaluate-empty-stack-expected.txt (right):

https://codereview.chromium.org/2529883002/diff/20001/test/inspector/runtime/...
test/inspector/runtime/evaluate-empty-stack-expected.txt:7: value : snape kills
dumbledore
Someone seems to be having fun here :)

https://codereview.chromium.org/2529883002/diff/40001/test/cctest/test-debug.cc
File test/cctest/test-debug.cc (left):

https://codereview.chromium.org/2529883002/diff/40001/test/cctest/test-debug....
test/cctest/test-debug.cc:6694: after_compile_message_count = 0;
Why does this test work if after_compile_event_count is not reset?

[Yang, 2016-11-24 09:56:17]

https://codereview.chromium.org/2529883002/diff/20001/test/inspector/runtime/...
File test/inspector/runtime/evaluate-empty-stack-expected.txt (right):

https://codereview.chromium.org/2529883002/diff/20001/test/inspector/runtime/...
test/inspector/runtime/evaluate-empty-stack-expected.txt:7: value : snape kills
dumbledore
On 2016/11/24 09:53:08, jgruber wrote:
> Someone seems to be having fun here :)

I hope I'm not spoiling it for anyone :)

https://codereview.chromium.org/2529883002/diff/40001/test/cctest/test-debug.cc
File test/cctest/test-debug.cc (left):

https://codereview.chromium.org/2529883002/diff/40001/test/cctest/test-debug....
test/cctest/test-debug.cc:6694: after_compile_message_count = 0;
On 2016/11/24 09:53:08, jgruber wrote:
> Why does this test work if after_compile_event_count is not reset?

cctests are supposed to be called one by one, to provide a clean state.
after_compile_event_count is set to 0 from start, so we don't need to reset it.
This is a pattern we use elsewhere, so I just applied it here :).

[commit-bot: I haz the power, 2016-11-24 10:02:57]

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

[commit-bot: I haz the power, 2016-11-24 10:02:59]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-11-24 10:36:21]

lgtm

[Yang, 2016-11-24 10:41:15]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2016-11-24 10:41:23]

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-11-24 10:43:01]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1479984075157370,
"parent_rev": "faa1fa8bbd34603f27064b03d01ae13f0e33b9ee", "commit_rev":
"c4c98fdec82ea1f5eb8f6d7880cd65f49508ac10"}

[commit-bot: I haz the power, 2016-11-24 10:43:07]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-24 10:43:28]

Description was changed from

==========
[debug] remove JSON debug protocol related tests.

In most cases we can use the debug event listener as alternative.
Multithreaded tests are obsolete.

R=jgruber@chromium.org
BUG=v8:5510
==========

to

==========
[debug] remove JSON debug protocol related tests.

In most cases we can use the debug event listener as alternative.
Multithreaded tests are obsolete.

R=jgruber@chromium.org
BUG=v8:5510

Committed: https://crrev.com/81a64aa5488602e4643f1ecc3beeddb8828a853c
Cr-Commit-Position: refs/heads/master@{#41256}
==========

[commit-bot: I haz the power, 2016-11-24 10:43:30]

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