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

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

Issue 2843013002: Revert of Don't lower priority for scripts inserted by doc.write (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/inspector/network/resource-priority-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/http/tests/inspector/network/resource-priority.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/inspector/network/resource-priority.html b/third_party/WebKit/LayoutTests/http/tests/inspector/network/resource-priority.html
index 025d43993f59e5a22356066484ee59205b3f5644..01023c092e279e70f7df22cc19d3caa481c34b8e 100644
--- a/third_party/WebKit/LayoutTests/http/tests/inspector/network/resource-priority.html
+++ b/third_party/WebKit/LayoutTests/http/tests/inspector/network/resource-priority.html
@@ -33,7 +33,7 @@
function sendImageRequest()
{
var img = document.createElement("img");
- img.src = "resources/abe.png?image";
+ img.src = "resources/abe.png";
document.body.appendChild(img);
}
@@ -41,7 +41,7 @@
{
var link = document.createElement("link");
link.rel = "stylesheet";
- link.href = "resources/style.css?style";
+ link.href = "resources/style.css";
document.head.appendChild(link);
}
@@ -49,8 +49,8 @@
{
var iframe = document.createElement("iframe");
document.body.appendChild(iframe);
- iframe.srcdoc = '<html><body><img src="resources/abe.png?precedingScript">'
- + '<script src="http://localhost:8000/inspector/network/resources/empty-script.js?precededByImage"></s'
+ iframe.srcdoc = '<html><body><img src="resources/abe.png?preceding">'
+ + '<script src="http://localhost:8000/inspector/network/resources/empty-script.js?preceded"></s'
+ 'cript>;</body></html>';
}
@@ -58,18 +58,9 @@
{
var iframe = document.createElement("iframe");
document.body.appendChild(iframe);
- iframe.srcdoc = '<html><body><link href="resources/abe.png?preloaded" rel=preload as=image>'
- + '<script src="http://localhost:8000/inspector/network/resources/empty-script.js?precededByPreloadedImage"></s'
- + 'cript><img src="resources/abe.png?followingPreload"></body></html>';
-}
-
-function sendScriptsFromDocumentWriteAfterImage()
-{
- var iframe = document.createElement("iframe");
- document.body.appendChild(iframe);
- iframe.srcdoc = '<html><body><img src="resources/abe.png?precedingDocWrite">'
- + '<script src="resources/docwrite.js"></s'
- + 'cript></body></html>';
+ iframe.srcdoc = '<html><body><link href="resources/abe.png?precedingPreload" rel=preload as=image>'
+ + '<script src="http://localhost:8000/inspector/network/resources/empty-script.js?precededByPreload"></s'
+ + 'cript><img src="resources/abe.png?precedingPreload"></body></html>';
}
function createIFrame()
@@ -82,41 +73,36 @@
function test()
{
var actions = [
- {"fn": "sendSyncScriptRequest", "requests": 1},
- {"fn": "sendAsyncScriptRequest", "requests": 1},
- {"fn": "sendScriptRequestPrecededByImage", "requests": 2},
- {"fn": "sendScriptRequestPrecededByPreloadedImage", "requests": 3},
- {"fn": "sendXHRSync", "requests": 1},
- {"fn": "sendXHRAsync", "requests": 1},
- {"fn": "sendImageRequest", "requests": 1},
- {"fn": "sendStyleRequest", "requests": 1},
- {"fn": "createIFrame", "requests": 1},
- {"fn": "sendScriptsFromDocumentWriteAfterImage", "requests": 5},
+ "sendSyncScriptRequest",
+ "sendAsyncScriptRequest",
+ "sendScriptRequestPrecededByImage",
+ "sendScriptRequestPrecededByPreloadedImage",
+ "sendXHRSync",
+ "sendXHRAsync",
+ "sendImageRequest",
+ "sendStyleRequest",
+ "createIFrame"
];
InspectorTest.networkManager.addEventListener(SDK.NetworkManager.Events.RequestStarted, onRequestStarted);
var nextAction = 0;
- var expectedRequestCount = 0;
- performNextAction();
+ performNextRequest();
- function performNextAction()
+ function performNextRequest()
{
if (nextAction >= actions.length) {
InspectorTest.networkManager.removeEventListener(SDK.NetworkManager.Events.RequestStarted, onRequestStarted);
InspectorTest.completeTest();
return;
}
- InspectorTest.addResult(actions[nextAction].fn);
- expectedRequestCount = actions[nextAction].requests;
- InspectorTest.evaluateInPage(actions[nextAction++].fn + "()");
+ InspectorTest.evaluateInPage(actions[nextAction++] + "()");
}
function onRequestStarted(event)
{
var request = event.data;
InspectorTest.addResult("Request: " + request.name() + " priority: " + request.initialPriority());
- expectedRequestCount--;
- if (expectedRequestCount < 1)
- performNextAction();
+ if (request.name().indexOf("preceded") == -1)
+ performNextRequest();
}
}
</script>
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/inspector/network/resource-priority-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698