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

Unified Diff: third_party/WebKit/LayoutTests/inspector/sources/debugger-ui/scripts-sorting.html

Issue 1563113002: DevTools: add a navigator menu with grouping options. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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/inspector/sources/debugger-ui/scripts-sorting.html
diff --git a/third_party/WebKit/LayoutTests/inspector/sources/debugger-ui/scripts-sorting.html b/third_party/WebKit/LayoutTests/inspector/sources/debugger-ui/scripts-sorting.html
index c74a435ec963ce803f68d70244ffeae6b373a12e..4d2b1c7676dac3d01450f5e63027a5dadcedf38f 100644
--- a/third_party/WebKit/LayoutTests/inspector/sources/debugger-ui/scripts-sorting.html
+++ b/third_party/WebKit/LayoutTests/inspector/sources/debugger-ui/scripts-sorting.html
@@ -7,11 +7,6 @@
<script>
function test()
{
- function dumpNavigator(sourcesNavigatorView)
- {
- InspectorTest.dumpNavigatorView(sourcesNavigatorView);
- }
-
function createNavigatorView(constructor)
{
var navigatorView = new constructor();
@@ -32,31 +27,18 @@ function test()
uiSourceCodes.push(uiSourceCode);
}
- function dumpNavigator(view)
- {
- ["frame", "frame/domain", "frame/domain/folder", "domain", "domain/folder"].forEach(dumpNavigatorInMode.bind(null, view));
- }
-
- function dumpNavigatorInMode(view, mode)
- {
- InspectorTest.addResult(view === sourcesNavigatorView ? "Sources:" : "Content Scripts:");
- WebInspector.moduleSetting("navigatorGrouping").set(mode);
- InspectorTest.addResult("-------- Setting mode: [" + mode + "]");
- InspectorTest.dumpNavigatorView(view);
- }
-
function dumpScriptsList()
{
- dumpNavigator(sourcesNavigatorView);
- dumpNavigator(contentScriptsNavigatorView);
+ InspectorTest.dumpNavigatorViewInAllModes(sourcesNavigatorView);
+ InspectorTest.dumpNavigatorViewInAllModes(contentScriptsNavigatorView);
for (var i = 0; i < uiSourceCodes.length; ++i) {
sourcesNavigatorView.revealUISourceCode(uiSourceCodes[i]);
contentScriptsNavigatorView.revealUISourceCode(uiSourceCodes[i]);
}
- dumpNavigator(sourcesNavigatorView);
- dumpNavigator(contentScriptsNavigatorView);
+ InspectorTest.dumpNavigatorViewInAllModes(sourcesNavigatorView);
+ InspectorTest.dumpNavigatorViewInAllModes(contentScriptsNavigatorView);
}
var scripts = [

Powered by Google App Engine
This is Rietveld 408576698