[DevTools] Added inline breakpoints

third_party/WebKit/LayoutTests/http/tests/inspector/debugger-test.js

 function scheduleTestFunction()
 {
     setTimeout(testFunction, 0);
 }
 
 var initialize_DebuggerTest = function() {
 
 InspectorTest.preloadPanel("sources");
 
 InspectorTest.startDebuggerTest = function(callback, quiet)
@@ skipping 611 matching lines @@
     var threadsPane = self.runtime.sharedInstance(Sources.ThreadsSidebarPane);
     var listItem = threadsPane._listItemForTarget(target);
     threadsPane._onListItemClick(listItem);
 }
 
 InspectorTest.evaluateOnCurrentCallFrame = function(code)
 {
     return new Promise(succ => InspectorTest.debuggerModel.evaluateOnSelectedCallFrame(code, "console", false, true, false, false, InspectorTest.safeWrap(succ)));
 }
 
-InspectorTest.waitJavaScriptSourceFrameBreakpoints = function(sourceFrame)
+InspectorTest.prepareJavaScriptSourceFrame = function(sourceFrame)

[lushnikov, 2016/11/29 21:24:29]

prepareSourceFrameForBreakpointTest =

[kozy, 2016/11/29 22:30:10]

Done.

 {
-    return new Promise(resolve => InspectorTest.addSniffer(sourceFrame.__proto__, "_breakpointDecorationsUpdatedForTest", resolveIfAllBreakpointsResolved.bind(null, resolve)));
+    var symbol = Symbol('waitedDecorations');
+    sourceFrame[symbol] = 0;
+    InspectorTest.addSniffer(sourceFrame.__proto__, "_willAddInlineDecorationsForTest", () => sourceFrame[symbol]++, true);
+    InspectorTest.addSniffer(sourceFrame.__proto__, "_didAddInlineDecorationsForTest", (updateWasScheduled) => {
+        sourceFrame[symbol]--;
+        if (!updateWasScheduled)
+            sourceFrame._breakpointDecorationsUpdatedForTest();
+    }, true);
+    sourceFrame._waitingForPossibleLocationsForTest = () => !!sourceFrame[symbol];
+}
 
-    function resolveIfAllBreakpointsResolved(resolve)
+InspectorTest.waitJavaScriptSourceFrameBreakpoints = function(sourceFrame, inline)
+{
+    return waitUpdate().then(checkIfReady);

[lushnikov, 2016/11/29 21:24:29]

let's that sourceFrame is prepared!

if (..) {
   log
   InspectorTest.completeTest();
   return;
}

[kozy, 2016/11/29 22:30:10]

Done.

+    function waitUpdate()
     {
+        return new Promise(resolve => InspectorTest.addSniffer(sourceFrame.__proto__, "_breakpointDecorationsUpdatedForTest", resolve));
+    }
+    function checkIfReady()
+    {
+        if (sourceFrame._waitingForPossibleLocationsForTest())
+            return waitUpdate().then(checkIfReady);

[lushnikov, 2016/11/29 21:24:29]

this will potentially result in an infinite promise chain

[kozy, 2016/11/29 22:30:10]

Yes, but all previous promises will be collected after resolving since there is
no other reference to promise object and there are no any memory leaks.

         for (var breakpoint of Bindings.breakpointManager._allBreakpoints()) {
-            if (breakpoint._fakePrimaryLocation && breakpoint.enabled()) {
-                InspectorTest.addSniffer(sourceFrame.__proto__, "_breakpointDecorationsUpdatedForTest", resolveIfAllBreakpointsResolved.bind(null, resolve));
-                return;
-            }
+            if (breakpoint._fakePrimaryLocation && breakpoint.enabled())
+                return waitUpdate().then(checkIfReady);
         }
-        resolve();
+        return Promise.resolve();
     }
 }
 
 InspectorTest.dumpJavaScriptSourceFrameBreakpoints = function(sourceFrame)
 {
     var textEditor = sourceFrame._textEditor;
     for (var lineNumber = 0; lineNumber < textEditor.linesCount; ++lineNumber) {
         if (!textEditor.hasLineClass(lineNumber, "cm-breakpoint"))
             continue;
         var disabled = textEditor.hasLineClass(lineNumber, "cm-breakpoint-disabled");
         var conditional = textEditor.hasLineClass(lineNumber, "cm-breakpoint-conditional")
         InspectorTest.addResult("breakpoint at " + lineNumber + (disabled ? " disabled" : "") + (conditional ? " conditional" : ""));
     }
+    var bookmarks = textEditor.bookmarks(textEditor.fullRange(), Sources.JavaScriptSourceFrame.BreakpointDecoration._bookmarkSymbol);
+    bookmarks = bookmarks.filter(bookmark => !!bookmark.position());
+    bookmarks.sort((bookmark1, bookmark2) => bookmark1.position().startColumn - bookmark2.position().startColumn);
+    for (var bookmark of bookmarks) {
+        var position = bookmark.position();
+        var element = bookmark[Sources.JavaScriptSourceFrame.BreakpointDecoration._elementSymbolForTest];
+        var disabled = element.classList.contains("cm-inline-disabled");
+        var conditional = element.classList.contains("cm-inline-conditional");
+        InspectorTest.addResult("  inline breakpoint at (" + position.startLine + ", " + position.startColumn + ")" + (disabled ? " disabled" : "") + (conditional ? " conditional" : ""));
+    }
+}
+
+InspectorTest.clickJavaScriptSourceFrameBreakpoint = function(sourceFrame, lineNumber, index)
+{
+    var textEditor = sourceFrame._textEditor;
+    var lineLength = textEditor.line(lineNumber).length;
+    var lineRange = new Common.TextRange(lineNumber, 0, lineNumber, lineLength);
+    var bookmarks = textEditor.bookmarks(lineRange, Sources.JavaScriptSourceFrame.BreakpointDecoration._bookmarkSymbol);
+    bookmarks.sort((bookmark1, bookmark2) => bookmark1.position().startColumn - bookmark2.position().startColumn);
+    bookmarks[index][Sources.JavaScriptSourceFrame.BreakpointDecoration._elementSymbolForTest].click();
 }
 
 };