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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/inspector/security/interstitial-sidebar.html

Issue 2522733002: Do not prompt for reload when security panel is opened on an interstitial (Closed)
Patch Set: remove isInterstitialShowing setter 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/security/interstitial-sidebar.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/security/interstitial-sidebar.html b/third_party/WebKit/LayoutTests/http/tests/inspector/security/interstitial-sidebar.html
index 916d422d080c5706a2961b42ebc3661167d32013..fa121cb01bb44256f9f2886079c92a42f3af1fbc 100644
--- a/third_party/WebKit/LayoutTests/http/tests/inspector/security/interstitial-sidebar.html
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector/security/interstitial-sidebar.html
@@ -19,6 +19,10 @@ function test()
// Test that the sidebar is hidden when an interstitial is shown. https://crbug.com/559150
InspectorTest.mainTarget.model(SDK.ResourceTreeModel).dispatchEventToListeners(SDK.ResourceTreeModel.Events.InterstitialShown);
+ // Simulate a request finishing after the interstitial is shown, to make sure that doesn't show up in the sidebar.
+ var request3 = new SDK.NetworkRequest(InspectorTest.mainTarget, 0, "https://bar.test/foo.jpg", "https://bar.test", 0, 0, null);
+ request3.setSecurityState(Protocol.Security.SecurityState.Unknown);
+ InspectorTest.dispatchRequestFinished(request3);
InspectorTest.addResult("After interstitial is shown:");
InspectorTest.dumpDeepInnerHTML(Security.SecurityPanel._instance()._sidebarTree.element);

Powered by Google App Engine
This is Rietveld 408576698