Chromium Code Reviews Chromium Code Reviews
Issue 1663723002:
[DevTools] Add sourceMap support for blackboxing (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@call-set-blackboxed-ranges-on-script-parsed| Index: third_party/WebKit/LayoutTests/inspector/sources/debugger-frameworks/frameworks-sourcemap.html |
| diff --git a/third_party/WebKit/LayoutTests/inspector/sources/debugger-frameworks/frameworks-step-into-skips-setTimeout.html b/third_party/WebKit/LayoutTests/inspector/sources/debugger-frameworks/frameworks-sourcemap.html |
| similarity index 57% |
| copy from third_party/WebKit/LayoutTests/inspector/sources/debugger-frameworks/frameworks-step-into-skips-setTimeout.html |
| copy to third_party/WebKit/LayoutTests/inspector/sources/debugger-frameworks/frameworks-sourcemap.html |
| index de0edb7e1449ce1b5a85f7573bf582555b4475ac..d3d42782d0b2a3533712e77d894589a0c5d64b54 100644 |
| --- a/third_party/WebKit/LayoutTests/inspector/sources/debugger-frameworks/frameworks-step-into-skips-setTimeout.html |
| +++ b/third_party/WebKit/LayoutTests/inspector/sources/debugger-frameworks/frameworks-sourcemap.html |
| @@ -2,27 +2,17 @@ |
| <head> |
| <script src="../../../http/tests/inspector/inspector-test.js"></script> |
| <script src="../../../http/tests/inspector/debugger-test.js"></script> |
| -<script src="../debugger/resources/framework.js"></script> |
| +<script src="../debugger/resources/framework-with-sourcemap.js"></script> |
| <script> |
| - |
| -var counter = 0; |
| - |
| function testFunction() |
| { |
| - Framework.scheduleUntilDone(callback, 0); |
| -} |
| - |
| -function callback() |
| -{ |
| - ++counter; |
| - if (counter === 1) |
| - stop(); |
| - return counter === 2; |
| + debugger; |
| + return foo(callback); |
| } |
| -function stop() |
| +function callback(i) |
| { |
| - debugger; |
| + return i; |
| } |
| function test() |
| @@ -40,10 +30,9 @@ function test() |
| function step2() |
| { |
| var actions = [ |
| - "Print", // debugger; |
| - "StepOut", "Print", |
| - "StepInto", "Print", |
| - "StepInto", "Print", // Should NOT stop on setTimeout() inside framework |
| + "Print", // "debugger" in testFunction() |
| + "StepInto", "StepInto", "Print", // entered callback(i) |
|
dgozman
2016/02/03 18:24:10
Can we have more extensive test? Examine async cal
kozy
2016/02/09 23:10:09
This test is checked stepping behavior with blackb
|
| + "StepOut", "Print" |
| ]; |
| InspectorTest.waitUntilPausedAndPerformSteppingActions(actions, step3); |
| } |
| @@ -53,14 +42,9 @@ function test() |
| InspectorTest.completeDebuggerTest(); |
| } |
| } |
| - |
| </script> |
| </head> |
| - |
| <body onload="runTest()"> |
| -<input type='button' onclick='testFunction()' value='Test'/> |
| -<p> |
| -Tests that stepping into blackboxed framework will not pause on setTimeout() inside the framework. |
| -</p> |
| +<p>Tests framework blackboxing feature with sourcemaps.</p> |
| </body> |
| </html> |
