DevTools: native breakpoint scopes break with blackboxing |
||
Issue descriptionThis includes: - interaction with RequestStepFrame and willExecuteScript; - nested scopes break all assumptions.
,
Feb 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/56bf7dbdaf6cb413390d56f2e595d1fbaecc9b30 commit 56bf7dbdaf6cb413390d56f2e595d1fbaecc9b30 Author: kozyatinskiy <kozyatinskiy@chromium.org> Date: Wed Feb 08 01:42:54 2017 [inspector] support for nested scheduled breaks In current implementation we don't support nested scheduled break at all. If one break was scheduled inside another and second one doesn't produce actual break (execution was in blackboxed code or no JavaScript was executed) then second one will clear first scheduled break even if any not blackboxed JavaScript will be executed later. Ambiguous break reason is added for the case when we have more then one scheduled reason. "auxData" in this case contains object with array of { reason: reason, auxData: auxData } objects for each reason in 'reasons' property. BUG= chromium:632405 Review-Url: https://codereview.chromium.org/2678313002 Cr-Commit-Position: refs/heads/master@{#43021} [modify] https://crrev.com/56bf7dbdaf6cb413390d56f2e595d1fbaecc9b30/src/inspector/js_protocol.json [modify] https://crrev.com/56bf7dbdaf6cb413390d56f2e595d1fbaecc9b30/src/inspector/v8-debugger-agent-impl.cc [modify] https://crrev.com/56bf7dbdaf6cb413390d56f2e595d1fbaecc9b30/src/inspector/v8-debugger-agent-impl.h [add] https://crrev.com/56bf7dbdaf6cb413390d56f2e595d1fbaecc9b30/test/inspector/debugger/framework-nested-scheduled-break-expected.txt [add] https://crrev.com/56bf7dbdaf6cb413390d56f2e595d1fbaecc9b30/test/inspector/debugger/framework-nested-scheduled-break.js [modify] https://crrev.com/56bf7dbdaf6cb413390d56f2e595d1fbaecc9b30/test/inspector/debugger/inspector-break-api.js [modify] https://crrev.com/56bf7dbdaf6cb413390d56f2e595d1fbaecc9b30/test/inspector/inspector-test.cc
,
Jun 5 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Jul 30 2016