LoAF: UI events should be outside renderStart
- Mark renderStart after UI events
- Still count the UI event time as part of blockingDuration
Bug: 1481484
Change-Id: I6b4579c09f3bdd0828cc7e4eee10545db6fe86ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5046790
Reviewed-by: Michal Mocny <mmocny@chromium.org>
Commit-Queue: Noam Rosenthal <nrosenthal@chromium.org>
Reviewed-by: Yoav Weiss <yoavweiss@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1227914}
diff --git a/long-animation-frame/tentative/loaf-ui-event-render-start.html b/long-animation-frame/tentative/loaf-ui-event-render-start.html
new file mode 100644
index 0000000..f8bcf34
--- /dev/null
+++ b/long-animation-frame/tentative/loaf-ui-event-render-start.html
@@ -0,0 +1,46 @@
+<!DOCTYPE HTML>
+<meta charset=utf-8>
+<title>Long Animation Frame Timing: UI events and renderStart</title>
+<meta name="timeout" content="long">
+<script src=/resources/testdriver.js></script>
+<script src=/resources/testdriver-actions.js></script>
+<script src=/resources/testdriver-vendor.js></script>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="resources/utils.js"></script>
+
+<body>
+<h1>Long Animation Frame: UI events and renderStart</h1>
+<div id="log"></div>
+<script>
+
+promise_test(async t => {
+ const BUSY_DURATION_1 = 150;
+ const BUSY_DURATION_2 = 250;
+ const eventPromise = new Promise(resolve => document.body.addEventListener("pointermove", () => {
+ busy_wait(BUSY_DURATION_1);
+ requestAnimationFrame(() => busy_wait(BUSY_DURATION_2));
+ resolve();
+ }, {passive: true}));
+ const actions = new test_driver.Actions();
+ await actions.pointerMove(10, 10, {origin: document.body})
+ .pointerDown()
+ .pointerMove(3, 3)
+ .pointerUp()
+ .send();
+
+ const scriptPredicate = s => s.name === "BODY.onpointermove";
+
+ const loaf = await new Promise(resolve =>
+ new PerformanceObserver(entries => {
+ const entry = entries.getEntries().find(
+ e => e.scripts.some(scriptPredicate));
+ resolve(entry);
+ }).observe({type: "long-animation-frame", buffered: true}));
+
+ const script = loaf.scripts.find(scriptPredicate)
+ assert_greater_than_equal(loaf.renderStart, script.startTime + script.duration);
+ assert_greater_than_equal(loaf.blockingDuration, BUSY_DURATION_1 + BUSY_DURATION_2 - 50);
+}, "UI events should always be before renderStart but still affect blockingDuration")
+</script>
+</body>
