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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/misc/resource-timing-sizes-cors-preflight.html

Issue 2159163002: Set Resource Timing transferSize field in Workers (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@resource_timing_sizes_render_process
Patch Set: Rebase Created 4 years, 5 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/misc/resource-timing-sizes-cors-preflight.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/misc/resource-timing-sizes-cors-preflight.html b/third_party/WebKit/LayoutTests/http/tests/misc/resource-timing-sizes-cors-preflight.html
index 6525e2f87350adb68705343cdf936957b6daf84c..5347b7d8559cf15717bdc0f04dbade2dfa8d497c 100644
--- a/third_party/WebKit/LayoutTests/http/tests/misc/resource-timing-sizes-cors-preflight.html
+++ b/third_party/WebKit/LayoutTests/http/tests/misc/resource-timing-sizes-cors-preflight.html
@@ -3,61 +3,4 @@
<script src="/resources/testharnessreport.js"></script>
<script src="/resources/get-host-info.js?pipe=sub"></script>
<script src="/misc/resources/run-async-tasks-promise.js"></script>
-<script>
-// Because apache decrements the Keep-Alive max value on each request, the
-// transferSize will vary slightly between requests for the same resource.
-const fuzzFactor = 3; // bytes
-
-const hostInfo = get_host_info();
-const url = new URL('/misc/resources/cors-preflight.php',
- hostInfo['HTTP_REMOTE_ORIGIN']).href;
-
-// The header bytes are expected to be > |minHeaderSize| and
-// < |maxHeaderSize|. If they are outside this range the test will fail.
-const minHeaderSize = 100;
-const maxHeaderSize = 1024;
-
-function checkResourceSizes() {
- var lowerBound, upperBound;
- var entries = performance.getEntriesByName(url);
- assert_equals(entries.length, 3, 'Wrong number of entries');
- // Firefox 47 puts the preflight after the request in the timeline.
- // Sort by requestStart for compatibility.
- entries.sort((a, b) => b.requestStart - a.requestStart);
- var seenCount = 0;
- for (var entry of entries) {
- switch (seenCount) {
- case 0:
- assert_greater_than(entry.transferSize, 0,
- 'no preflight transferSize');
- lowerBound = entry.transferSize - fuzzFactor;
- upperBound = entry.transferSize + fuzzFactor;
- break;
-
- case 1:
- assert_between_exclusive(entry.transferSize, minHeaderSize,
- maxHeaderSize,
- 'preflight transferSize');
- break;
-
- case 2:
- assert_between_exclusive(entry.transferSize, lowerBound,
- upperBound,
- 'preflighted transferSize');
- break;
- }
- ++seenCount;
- }
-}
-
-promise_test(() => {
- var eatBody = response => response.arrayBuffer();
- var requirePreflight = {headers: {'X-Require-Preflight': '1'}};
- return fetch(url)
- .then(eatBody)
- .then(() => fetch(url, requirePreflight))
- .then(eatBody)
- .then(runAsyncTasks)
- .then(checkResourceSizes);
-}, 'PerformanceResourceTiming sizes Fetch with preflight test');
-</script>
+<script src="/misc/resources/resource-timing-sizes-cors-preflight.js"></script>

Powered by Google App Engine
This is Rietveld 408576698