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

Unified Diff: third_party/WebKit/LayoutTests/fast/performance/longtasktiming.html

Issue 2610063006: Create TaskAttributionTiming PerformanceEntry for attribution in PerformanceLongTaskTiming. Move cu… (Closed)
Patch Set: update layout test expectation Created 3 years, 11 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/fast/performance/longtasktiming.html
diff --git a/third_party/WebKit/LayoutTests/fast/performance/longtasktiming.html b/third_party/WebKit/LayoutTests/fast/performance/longtasktiming.html
index cd5f0453b1d4b345660afa0f71766cb906ec8616..f2db95a75435cf5bd9c9ba0bfbe31eb5bc902eed 100644
--- a/third_party/WebKit/LayoutTests/fast/performance/longtasktiming.html
+++ b/third_party/WebKit/LayoutTests/fast/performance/longtasktiming.html
@@ -7,17 +7,27 @@ async_test(function (t) {
var observer = new PerformanceObserver(
t.step_func(function (entryList) {
var entries = entryList.getEntries();
- /* TODO(panicker): Update after fixing http://crbug.com/651461 */
- for (var i = 0; i < entries.length; i++) {
- assert_equals(entries[i].entryType, "longtask",
- "entryType expected to be: longtask");
- assert_equals(entries[i].name, "same-origin-self",
- "name expected to be: same-origin-self");
- assert_greater_than(entries[i].duration, 50,
- "duration expected to be greater than 50ms threshold");
- assert_equals(entries[i].startTime, Math.floor(entries[i].startTime),
- "startTime expected to have 1 miilisecond granularity");
- }
+ assert_equals(entries.length, 1,
+ "Exactly one entry is expected.");
+ var longtask = entries[0];
+ assert_equals(longtask.entryType, "longtask",
+ "entryType expected to be: longtask");
+ assert_equals(longtask.name, "same-origin-self",
+ "name expected to be: same-origin-self");
+ assert_greater_than(longtask.duration, 50,
+ "duration expected to be greater than 50ms threshold");
+ assert_equals(longtask.startTime, Math.floor(longtask.startTime),
+ "startTime expected to have 1 miillisecond granularity");
+
+ // Assert the TaskAttributionTiming entry in attribution.
+ assert_equals(longtask.attribution.length, 1,
+ "Exactly one attribution entry is expected");
+ var attribution = longtask.attribution[0];
+ assert_equals(attribution.entryType, "taskattribution");
+ assert_equals(attribution.name, "frame");
+ assert_equals(attribution.duration, 0);
+ assert_equals(attribution.startTime, 0);
+
observer.disconnect();
t.done();
})

Powered by Google App Engine
This is Rietveld 408576698