Skip to content
This repository has been archived by the owner on Nov 7, 2018. It is now read-only.

Commit

Permalink
Bug 1465603 [wpt PR 11257] - Change Resource Timing default buffer si…
Browse files Browse the repository at this point in the history
…ze to 250., a=testonly

Automatic update from web-platform-testsChange Resource Timing default buffer size to 250.

Following a proposed spec change[1], this CL changes the number of entries
buffered by default before the Resource Timing buffer is full.

[1] w3c/resource-timing#155

Bug: 847689
Change-Id: Id93bfea902b0cf049abdac12a98cc4fba7ff1dd3
Reviewed-on: https://chromium-review.googlesource.com/1078107
Reviewed-by: Nicolás Peña Moreno <npm@chromium.org>
Commit-Queue: Yoav Weiss <yoav@yoav.ws>
Cr-Commit-Position: refs/heads/master@{#563133}

--

wpt-commits: f696c249c6d41c32bdc04d0bec688ee00228a0cf
wpt-pr: 11257
  • Loading branch information
yoavweiss authored and jgraham committed Jun 10, 2018
1 parent 7ef2ace commit b02e6e8
Show file tree
Hide file tree
Showing 2 changed files with 41 additions and 0 deletions.
10 changes: 10 additions & 0 deletions testing/web-platform/meta/MANIFEST.json
Original file line number Diff line number Diff line change
Expand Up @@ -365549,6 +365549,12 @@
{}
]
],
"resource-timing/resource_timing_buffer_full_eventually.html": [
[
"/resource-timing/resource_timing_buffer_full_eventually.html",
{}
]
],
"resource-timing/resource_timing_buffer_full_when_populate_entries.html": [
[
"/resource-timing/resource_timing_buffer_full_when_populate_entries.html",
Expand Down Expand Up @@ -600560,6 +600566,10 @@
"ccf716aec242f3ec6d3754e4dd43456954d45d3c",
"testharness"
],
"resource-timing/resource_timing_buffer_full_eventually.html": [
"5c0a85e6e61a11fca1682b30b6b68f317f9c343e",
"testharness"
],
"resource-timing/resource_timing_buffer_full_when_populate_entries.html": [
"af0727706a4160a49aabe131b2f3a4bbca8268e0",
"testharness"
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
<!DOCTYPE HTML>
<html>
<head>
<meta charset="utf-8" />
<link rel="help" href="http://www.w3.org/TR/resource-timing/#performanceresourcetiming"/>
<title>This test validates that resource timing implementations have a finite number of entries in their buffer.</title>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
</head>
<body>
<script>
const t = async_test("Finite resource timing entries buffer size");
performance.onresourcetimingbufferfull = t.step_func_done(function() {
});
window.onload = t.step_func(
function() {
// Scripts appended in JS to ensure setResourceTimingBufferSize is called before.
let counter = performance.getEntriesByType("resource").length;
function appendScript() {
const src = "resources/empty.js?" + counter;
const script = document.createElement('script');
script.type = 'text/javascript';
script.onload = function() { ++counter; appendScript()};
script.src = src;
document.body.appendChild(script);
}
appendScript();
});
</script>
</body>
</html>

0 comments on commit b02e6e8

Please sign in to comment.