[debugger] remove debugger statement support from FCG/CS.

[debugger] remove debugger statement support from FCG/CS.


R=mstarzinger@chromium.org

Review-Url: https://codereview.chromium.org/2650193002
Cr-Original-Commit-Position: refs/heads/master@{#42892}
Committed: https://chromium.googlesource.com/v8/v8/+/eef855a1dc956e9db03ec09abca1d732d379861b
Review-Url: https://codereview.chromium.org/2650193002
Cr-Commit-Position: refs/heads/master@{#43360}
Committed: https://chromium.googlesource.com/v8/v8/+/3f303da2920e21f631325e54904290ba20b86827

[Michael Starzinger, 2017-01-24 13:10:24]

LGTM. I love it!

https://codereview.chromium.org/2650193002/diff/1/src/full-codegen/full-codeg...
File src/full-codegen/full-codegen.cc (left):

https://codereview.chromium.org/2650193002/diff/1/src/full-codegen/full-codeg...
src/full-codegen/full-codegen.cc:1237:
PrepareForBailoutForId(stmt->DebugBreakId(), BailoutState::NO_REGISTERS);
suggestion: We can now also remove DebuggerStatement::DebugBreakId and the
underlying functionality for DebuggerStatement to have bailout ids at all. They
should no longer be needed.

[Yang, 2017-01-24 13:20:23]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2017-01-24 13:20:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/2650193002/#ps40001
(title: "remove unnecessary include")

[commit-bot: I haz the power, 2017-01-24 13:20:30]

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-24 13:27:58]

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

[commit-bot: I haz the power, 2017-01-24 13:27:59]

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, 2017-02-02 12:11:53]

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

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

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-02-02 12:18:10]

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

[commit-bot: I haz the power, 2017-02-02 12:18:11]

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, 2017-02-02 13:39:24]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2017-02-02 13:39:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/2650193002/#ps80001
(title: "Disable always-opt")

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

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-02 14:21:04]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1486042764422480,
"parent_rev": "d0edd08eb5e89c1e2c1bd336adfc3362eae18d35", "commit_rev":
"eef855a1dc956e9db03ec09abca1d732d379861b"}

[commit-bot: I haz the power, 2017-02-02 14:21:09]

Description was changed from

==========
[debugger] remove debugger statement support from FCG/CS.


R=mstarzinger@chromium.org
==========

to

==========
[debugger] remove debugger statement support from FCG/CS.


R=mstarzinger@chromium.org

Review-Url: https://codereview.chromium.org/2650193002
Cr-Commit-Position: refs/heads/master@{#42892}
Committed:
https://chromium.googlesource.com/v8/v8/+/eef855a1dc956e9db03ec09abca1d732d37...
==========

[commit-bot: I haz the power, 2017-02-02 14:21:10]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/v8/v8/+/eef855a1dc956e9db03ec09abca1d732d37...

[kozy, 2017-02-04 17:34:47]

A revert of this CL (patchset #5 id:80001) has been created in
https://codereview.chromium.org/2672823007/ by kozyatinskiy@chromium.org.

The reason for reverting is: Fails on chromium leak bot:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...
.

[Yang, 2017-02-21 10:50:53]

Description was changed from

==========
[debugger] remove debugger statement support from FCG/CS.


R=mstarzinger@chromium.org

Review-Url: https://codereview.chromium.org/2650193002
Cr-Commit-Position: refs/heads/master@{#42892}
Committed:
https://chromium.googlesource.com/v8/v8/+/eef855a1dc956e9db03ec09abca1d732d37...
==========

to

==========
[debugger] remove debugger statement support from FCG/CS.


R=mstarzinger@chromium.org

Review-Url: https://codereview.chromium.org/2650193002
Cr-Commit-Position: refs/heads/master@{#42892}
Committed:
https://chromium.googlesource.com/v8/v8/+/eef855a1dc956e9db03ec09abca1d732d37...
==========

[Yang, 2017-02-21 13:48:12]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2017-02-21 13:48:19]

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-21 13:49:49]

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

[commit-bot: I haz the power, 2017-02-21 13:49:50]

Try jobs failed on following builders:
  v8_linux64_asan_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  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_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/21396)
  v8_linux64_verify_csa_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_verify_csa_rel_n...)
  v8_linux_arm_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_mips64el_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_mipsel_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_linux_verify_csa_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_verify_csa_rel_ng/...)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/35237)
  v8_win64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel_ng/builds/22983)

[Yang, 2017-02-22 08:19:06]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2017-02-22 08:19:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/2650193002/#ps100001
(title: "rebase")

[commit-bot: I haz the power, 2017-02-22 08:19:16]

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-22 08:45:10]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1487751546849540,
"parent_rev": "02136e108884e58cc46697ad6de4515590ac26e9", "commit_rev":
"3f303da2920e21f631325e54904290ba20b86827"}

[commit-bot: I haz the power, 2017-02-22 08:45:16]

Description was changed from

==========
[debugger] remove debugger statement support from FCG/CS.


R=mstarzinger@chromium.org

Review-Url: https://codereview.chromium.org/2650193002
Cr-Commit-Position: refs/heads/master@{#42892}
Committed:
https://chromium.googlesource.com/v8/v8/+/eef855a1dc956e9db03ec09abca1d732d37...
==========

to

==========
[debugger] remove debugger statement support from FCG/CS.


R=mstarzinger@chromium.org

Review-Url: https://codereview.chromium.org/2650193002
Cr-Original-Commit-Position: refs/heads/master@{#42892}
Committed:
https://chromium.googlesource.com/v8/v8/+/eef855a1dc956e9db03ec09abca1d732d37...
Review-Url: https://codereview.chromium.org/2650193002
Cr-Commit-Position: refs/heads/master@{#43360}
Committed:
https://chromium.googlesource.com/v8/v8/+/3f303da2920e21f631325e54904290ba20b...
==========

[commit-bot: I haz the power, 2017-02-22 08:45:17]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/v8/v8/+/3f303da2920e21f631325e54904290ba20b...