 Chromium Code Reviews
 Chromium Code Reviews Issue 329533002:
  DevTools: Added test that LiveEdit doesn't crash on breakpoint  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master2
    
  
    Issue 329533002:
  DevTools: Added test that LiveEdit doesn't crash on breakpoint  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master2| Index: LayoutTests/inspector/sources/debugger/live-edit.html | 
| diff --git a/LayoutTests/inspector/sources/debugger/live-edit.html b/LayoutTests/inspector/sources/debugger/live-edit.html | 
| index 6e864fed2f34f93ee12a8e0943e8a03d55a65d18..1c66a19cae8622372a8eef5090ca2b45a8f259e5 100644 | 
| --- a/LayoutTests/inspector/sources/debugger/live-edit.html | 
| +++ b/LayoutTests/inspector/sources/debugger/live-edit.html | 
| @@ -119,6 +119,38 @@ function test() | 
| InspectorTest.assertEquals("5", lineNumber); | 
| next(); | 
| } | 
| + }, | 
| + | 
| + function testNoCrashWhenUpdateOnBreakpoint(next) | 
| 
vsevik
2014/06/10 15:29:56
nit: ..WhenLiveEditOnBreakpoint...
 | 
| + { | 
| + InspectorTest.showScriptSource("edit-me.js", didShowScriptSource); | 
| + | 
| + function didShowScriptSource(sourceFrame) | 
| + { | 
| + InspectorTest.addSniffer(sourceFrame, "_addBreakpointDecoration", breakpointAdded); | 
| + InspectorTest.setBreakpoint(sourceFrame, 3, "", true); | 
| + } | 
| + | 
| + function breakpointAdded() | 
| + { | 
| + InspectorTest.waitUntilPaused(pausedInF); | 
| + InspectorTest.evaluateInPageWithTimeout("f()"); | 
| + } | 
| + | 
| + function pausedInF(callFrames) | 
| + { | 
| + replaceInSource(panel.visibleView, "function f()", "function f()\n", didEditScriptSource); | 
| + } | 
| + | 
| + function didEditScriptSource() | 
| + { | 
| + InspectorTest.resumeExecution(resumed); | 
| + } | 
| + | 
| + function resumed() | 
| + { | 
| + next(); | 
| + } | 
| } | 
| ]); | 
| @@ -127,7 +159,6 @@ function test() | 
| InspectorTest.addSniffer(WebInspector.debuggerModel, "_didEditScriptSource", callback); | 
| InspectorTest.replaceInSource(sourceFrame, string, replacement); | 
| InspectorTest.commitSource(sourceFrame); | 
| - | 
| } | 
| }; |