Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(55)

Unified Diff: third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload.html

Issue 2522753002: DevTools: [Persistence] properly copy breakpoints to filesystem during reload (Closed)
Patch Set: fix test Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload.html b/third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload.html
new file mode 100644
index 0000000000000000000000000000000000000000..b2634ae7b63496281225e7ea7f724bd1b045b4a1
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector/persistence/persistence-move-breakpoints-on-reload.html
@@ -0,0 +1,74 @@
+<html>
+<head>
+<script src='../inspector-test.js'></script>
+<script src='../debugger-test.js'></script>
+<script src='../workspace-test.js'></script>
+<script src='../isolated-filesystem-test.js'></script>
+<script src='./persistence-test.js'></script>
+<script>
+
+function addFooJS() {
+ var script = document.createElement('script');
+ script.src = './resources/foo.js';
+ document.body.appendChild(script);
+}
+
+function test()
+{
+ var fs = new InspectorTest.TestFileSystem('file:///var/www');
+ fs.addFileMapping('http://127.0.0.1:8000', '/');
+ InspectorTest.addFooJSFile(fs);
+
+ InspectorTest.runTestSuite([
+ function addFileSystem(next)
+ {
+ fs.reportCreated(next);
+ },
+
+ function addNetworkFooJS(next)
+ {
+ InspectorTest.evaluateInPage('addFooJS()');
+ InspectorTest.waitForBinding('foo.js').then(next);
+ },
+
+ function setBreakpointInNetworkUISourceCode(next)
+ {
+ InspectorTest.waitForUISourceCode('foo.js', Workspace.projectTypes.Network)
+ .then(sourceCode => InspectorTest.showUISourceCodePromise(sourceCode))
+ .then(onSourceFrame);
+
+ function onSourceFrame(sourceFrame)
+ {
+ InspectorTest.setBreakpoint(sourceFrame, 0, '', true);
+ InspectorTest.waitBreakpointSidebarPane().then(dumpBreakpointSidebarPane).then(next);
+ }
+ },
+
+ function reloadPageAndDumpBreakpoints(next)
+ {
+ Promise.all([
+ InspectorTest.waitBreakpointSidebarPane(),
+ InspectorTest.reloadPagePromise()
+ ]).then(dumpBreakpointSidebarPane).then(next);
+ },
+ ]);
+
+ function dumpBreakpointSidebarPane()
+ {
+ var paneElement = self.runtime.sharedInstance(Sources.JavaScriptBreakpointsSidebarPane).contentElement;
+ var empty = paneElement.querySelector('.gray-info-message');
+ if (empty)
+ return InspectorTest.textContentWithLineBreaks(empty);
+ var entries = Array.from(paneElement.querySelectorAll('.breakpoint-entry'));
+ for (var entry of entries) {
+ var uiLocation = entry[Sources.JavaScriptBreakpointsSidebarPane._locationSymbol];
+ InspectorTest.addResult(" " + uiLocation.uiSourceCode.url() +":" + uiLocation.lineNumber);
+ }
+ }
+};
+</script>
+</head>
+<body onload='runTest()'>
+<p>Verify that breakpoints are moved appropriately in case of page reload.</p>
+</body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698