Reviewers: dgozman
CL:
https://codereview.chromium.org/2548583002/Message:
Dmitry, please take a look.
Description:
[DevTools] Ship inline breakpoints
Enabled inline breakpoints without experiment.
+ small UX improvement.
BUG=chromium:566801
R=dgo...@chromium.orgAffected files (+4, -6 lines):
M third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload-expected.txt
M third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html
M third_party/WebKit/Source/devtools/front_end/main/Main.js
M third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
M third_party/WebKit/Source/devtools/front_end/text_editor/cmdevtools.css
Index: third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload-expected.txt
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload-expected.txt b/third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload-expected.txt
index 6d3084b01246cf240f2b9e7fefeed8d963fb6143..bc0d50abc88fe5492da6500ef11bad9b9fccd462 100644
--- a/third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload-expected.txt
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload-expected.txt
@@ -6,7 +6,7 @@ Running: addFileSystem
Running: addNetworkFooJS
Running: setBreakpointInNetworkUISourceCode
-
http://127.0.0.1:8000/inspector/persistence/resources/foo.js:0+
http://127.0.0.1:8000/inspector/persistence/resources/foo.js:2 Running: reloadPageAndDumpBreakpoints
Page reloaded.
Index: third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html
diff --git a/third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html b/third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html
index 93329e48bed3a3fb434790836025e8dbfded7421..92a0e63c7fc77bab3424e46b111d987a081d1e7d 100644
--- a/third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html
+++ b/third_party/WebKit/LayoutTests/inspector/sources/debugger/source-frame-inline-breakpoint-decorations.html
@@ -21,7 +21,6 @@ function test()
.then(() => InspectorTest.dumpJavaScriptSourceFrameBreakpoints(sourceFrame));
}
- Runtime.experiments.enableForTest("inlineBreakpoints");
Bindings.breakpointManager._storage._breakpoints = {};
InspectorTest.runDebuggerTestSuite([
function testAddRemoveBreakpoint(next)
Index: third_party/WebKit/Source/devtools/front_end/main/Main.js
diff --git a/third_party/WebKit/Source/devtools/front_end/main/Main.js b/third_party/WebKit/Source/devtools/front_end/main/Main.js
index b313e24603938e75ddc198b6c3e263c6d23b5777..b8a510aa01461248cb20a5da8bb23500ba6ca77c 100644
--- a/third_party/WebKit/Source/devtools/front_end/main/Main.js
+++ b/third_party/WebKit/Source/devtools/front_end/main/Main.js
@@ -111,7 +111,6 @@ Main.Main = class {
Runtime.experiments.register('resolveVariableNames', 'Resolve variable names');
Runtime.experiments.register('timelineShowAllEvents', 'Show all events on Timeline', true);
Runtime.experiments.register('timelineShowAllProcesses', 'Show all processes on Timeline', true);
- Runtime.experiments.register('inlineBreakpoints', 'Show inline breakpoints');
Runtime.experiments.register('securityPanel', 'Security panel');
Runtime.experiments.register('sourceDiff', 'Source diff');
Runtime.experiments.register('terminalInDrawer', 'Terminal in drawer', true);
Index: third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js b/third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
index 628f2ab47b7b24218a0dbb9c3052418c9ed78755..f0fb003b1244338978013565618bf8158d3486d8 100644
--- a/third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
+++ b/third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
@@ -916,7 +916,7 @@ Sources.JavaScriptSourceFrame = class extends Sources.UISourceCodeFrame {
}
breakpoint[Sources.JavaScriptSourceFrame.BreakpointDecoration._decorationSymbol] = decoration;
this._updateBreakpointDecoration(decoration);
- if (!lineDecorations.length && Runtime.experiments.isEnabled('inlineBreakpoints')) {
+ if (!lineDecorations.length) {
this._willAddInlineDecorationsForTest();
this._breakpointManager
.possibleBreakpoints(
@@ -1307,7 +1307,7 @@ Sources.JavaScriptSourceFrame.BreakpointDecoration = class {
}
show() {
- if (this.bookmark || !Runtime.experiments.isEnabled('inlineBreakpoints'))
+ if (this.bookmark)
return;
var location = this.handle.resolve();
if (!location)
Index: third_party/WebKit/Source/devtools/front_end/text_editor/cmdevtools.css
diff --git a/third_party/WebKit/Source/devtools/front_end/text_editor/cmdevtools.css b/third_party/WebKit/Source/devtools/front_end/text_editor/cmdevtools.css
index 18590e195e5baeae45d475609d64fabd93cc1bd1..969daea5acdbf78529e86030d050ab322b6e296d 100644
--- a/third_party/WebKit/Source/devtools/front_end/text_editor/cmdevtools.css
+++ b/third_party/WebKit/Source/devtools/front_end/text_editor/cmdevtools.css
@@ -121,7 +121,7 @@
.cm-inline-breakpoint {
position:relative;
- top: 1px;
+ top: 2px;
cursor: pointer;
}