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

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

Issue 2347483002: [DevTools] Remove SidebarTreeElement from SecurityPanel. (Closed)
Patch Set: Created 4 years, 3 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/security-test.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/security-test.js b/third_party/WebKit/LayoutTests/http/tests/inspector/security-test.js
index bed4d4dcd2937eb182bb8a86c9d33a71744b08b9..7131d1d551a139dc2c8de9ad53b48d5e7c4832f6 100644
--- a/third_party/WebKit/LayoutTests/http/tests/inspector/security-test.js
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector/security-test.js
@@ -5,12 +5,13 @@ InspectorTest.preloadPanel("security");
InspectorTest.dumpSecurityPanelSidebarOrigins = function() {
for (var key in WebInspector.SecurityPanelSidebarTree.OriginGroupName) {
var originGroupName = WebInspector.SecurityPanelSidebarTree.OriginGroupName[key];
- var originTitles = WebInspector.SecurityPanel._instance()._sidebarTree._originGroups.get(originGroupName).childrenListElement.getElementsByClassName("title");
- if (originTitles.length > 0) {
- InspectorTest.addResult("Group: " + originGroupName);
- for (var originTitle of originTitles)
- InspectorTest.dumpDeepInnerHTML(originTitle);
- }
+ var originGroup = WebInspector.SecurityPanel._instance()._sidebarTree._originGroups.get(originGroupName);
+ if (originGroup.hidden)
+ continue;
+ InspectorTest.addResult("Group: " + originGroupName);
+ var originTitles = originGroup.childrenListElement.getElementsByClassName("title");
+ for (var originTitle of originTitles)
+ InspectorTest.dumpDeepInnerHTML(originTitle);
}
}

Powered by Google App Engine
This is Rietveld 408576698