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

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

Issue 2795193003: DevTools: carefully cleanup script UISourceCodes (Closed)
Patch Set: address comments Created 3 years, 8 months 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/debugger-test.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/debugger-test.js b/third_party/WebKit/LayoutTests/http/tests/inspector/debugger-test.js
index d36a55e937c51ac735ea41dcf12574174acabc61..e3f5b2e29365a29084d9942e6d9a810a5081d899 100644
--- a/third_party/WebKit/LayoutTests/http/tests/inspector/debugger-test.js
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector/debugger-test.js
@@ -574,13 +574,8 @@ InspectorTest.setQuiet = function(quiet)
InspectorTest.queryScripts = function(filter)
{
- var scripts = [];
- for (var scriptId in InspectorTest.debuggerModel._scripts) {
- var script = InspectorTest.debuggerModel._scripts[scriptId];
- if (!filter || filter(script))
- scripts.push(script);
- }
- return scripts;
+ var scripts = InspectorTest.debuggerModel.scripts();
+ return filter ? scripts.filter(filter) : scripts;
};
InspectorTest.createScriptMock = function(url, startLine, startColumn, isContentScript, source, target, preRegisterCallback)

Powered by Google App Engine
This is Rietveld 408576698