[debug] mark more unused debug API as deprecated.

[debug] mark more unused debug API as deprecated.

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

[jgruber, 2016-11-24 15:27:04]

lgtm

https://codereview.chromium.org/2531543002/diff/1/test/debugger/test-api.js
File test/debugger/test-api.js (right):

https://codereview.chromium.org/2531543002/diff/1/test/debugger/test-api.js#n...
test/debugger/test-api.js:35: AfterCompile: 4,
We can probably also remove BeforeCompile. And looking at AsyncTaskEvent, I just
noticed the three tests that use it appear to be currently broken.

[Yang, 2016-11-25 08:31:35]

https://codereview.chromium.org/2531543002/diff/1/test/debugger/test-api.js
File test/debugger/test-api.js (right):

https://codereview.chromium.org/2531543002/diff/1/test/debugger/test-api.js#n...
test/debugger/test-api.js:35: AfterCompile: 4,
On 2016/11/24 15:27:03, jgruber wrote:
> We can probably also remove BeforeCompile. And looking at AsyncTaskEvent, I
just
> noticed the three tests that use it appear to be currently broken.

Done.

[Yang, 2016-11-25 08:31:40]

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

[commit-bot: I haz the power, 2016-11-25 08:31: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-11-25 08:43:43]

lgtm

[commit-bot: I haz the power, 2016-11-25 09:01:00]

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

[commit-bot: I haz the power, 2016-11-25 09:01:01]

Dry run: This issue passed the CQ dry run.

[Yang, 2016-11-25 09:09:07]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2016-11-25 09:09:13]

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-25 09:10:52]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480064947558390,
"parent_rev": "9dc196da41ee65acac2090b16d1eba8a7cd47fc3", "commit_rev":
"ddf46b6b21aa76c395c91fc3fd6e4f50c8013e3f"}

[commit-bot: I haz the power, 2016-11-25 09:10:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-25 09:11:07]

Description was changed from

==========
[debug] mark more unused debug API as deprecated.

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

to

==========
[debug] mark more unused debug API as deprecated.

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

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

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

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

[jgruber, 2016-12-01 08:23:39]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2542873002/ by jgruber@chromium.org.

The reason for reverting is: Legacy debugging API is still in use by Node
tests..

[jgruber, 2016-12-01 08:51:08]

On 2016/12/01 08:23:39, jgruber wrote:
> A revert of this CL (patchset #2 id:20001) has been created in
> https://codereview.chromium.org/2542873002/ by mailto:jgruber@chromium.org.
> 
> The reason for reverting is: Legacy debugging API is still in use by Node
> tests..

Manual revert in https://codereview.chromium.org/2537313005/