Reland "[EventTiming] Round instead of ceil the |duration|"
This is a reland of 9ac1e68e80f46289d00e6a1858740b89e34eb01e
Original change's description:
> [EventTiming] Round instead of ceil the |duration|
>
> The purpose of this change is to expose entries whose |duration| is >=
> 100ms.
>
> Bug: 823744
> Change-Id: Ibb089009ba287769b1365a938cc7564278a86c35
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594818
> Reviewed-by: Timothy Dresser <tdresser@chromium.org>
> Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#656797}
Bug: 823744
Change-Id: Ia7ceffaf7dc84951b74fff507b024a3f895039db
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1596849
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#656927}
diff --git a/event-timing/event-timing-retrieve-firstInput.html b/event-timing/event-timing-retrieve-firstInput.html
index b1b020e..15729ca 100644
--- a/event-timing/event-timing-retrieve-firstInput.html
+++ b/event-timing/event-timing-retrieve-firstInput.html
@@ -23,7 +23,7 @@
const entry = performance.getEntriesByType('firstInput')[0];
assert_equals(entry.name, 'mousedown');
assert_equals(entry.entryType, 'firstInput');
- assert_greater_than(entry.duration, 50,
+ assert_greater_than_equal(entry.duration, 104,
"The first input was a long one.");
t.done();
}
diff --git a/event-timing/resources/event-timing-support.js b/event-timing/resources/event-timing-support.js
index f6e450d..53cd497 100644
--- a/event-timing/resources/event-timing-support.js
+++ b/event-timing/resources/event-timing-support.js
@@ -35,7 +35,9 @@
"The entry's processingStart should be greater than startTime.");
assert_greater_than_equal(entry.processingEnd, entry.processingStart,
"The entry's processingEnd must be at least as large as processingStart.");
- assert_greater_than_equal(entry.duration, entry.processingEnd - entry.startTime,
+ // |duration| is a number rounded to the nearest 8 ms, so add 4 to get a lower bound
+ // on the actual duration.
+ assert_greater_than_equal(entry.duration + 4, entry.processingEnd - entry.startTime,
"The entry's duration must be at least as large as processingEnd - startTime.");
if (is_first) {
let firstInputs = performance.getEntriesByType('firstInput');
