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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/inspector/network/network-memory-cached-resource.html

Issue 2203613003: Split header modification out of willSendRequest (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Filter the header instead Created 4 years, 4 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/network/network-memory-cached-resource.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/network/network-memory-cached-resource.html b/third_party/WebKit/LayoutTests/http/tests/inspector/network/network-memory-cached-resource.html
index 9e9d3670e01f2cdaf3d6338c818090fade2be1e0..53e112a378b4c3450b3182b97b00f617aa73666e 100644
--- a/third_party/WebKit/LayoutTests/http/tests/inspector/network/network-memory-cached-resource.html
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector/network/network-memory-cached-resource.html
@@ -34,6 +34,11 @@ function test()
function step2()
{
+ InspectorTest.NetworkAgent.setCacheDisabled(false, step3);
+ }
+
+ function step3()
+ {
InspectorTest.evaluateInPage("attachIframe()");
}
@@ -42,19 +47,9 @@ function test()
if (!finished && findResource(/abe\.png/, 200, false) && findResource(/abe\.png/, 200, true)) {
finished = true;
InspectorTest.addResult("Memory-cached resource found.");
- step3();
+ InspectorTest.completeTest();
}
}
-
- function step3()
- {
- InspectorTest.NetworkAgent.setCacheDisabled(false, step4);
- }
-
- function step4(msg)
- {
- InspectorTest.completeTest();
- }
}
</script>
</head>

Powered by Google App Engine
This is Rietveld 408576698