From 5c00acca9828f6d35ecec7b55b2bcb19e1a73fd9 Mon Sep 17 00:00:00 2001 From: Patrick Shaughnessy Date: Wed, 29 Jan 2020 14:59:43 -0500 Subject: w3c/performance-timeline#159 --- components/script/dom/performance.rs | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) (limited to 'components/script/dom') diff --git a/components/script/dom/performance.rs b/components/script/dom/performance.rs index 52fbf46860e..82eec0e4996 100644 --- a/components/script/dom/performance.rs +++ b/components/script/dom/performance.rs @@ -218,17 +218,12 @@ impl Performance { let mut obs_entries = observer.entries(); obs_entries.append(&mut new_entries); observer.set_entries(obs_entries); + } - // W3C spec as of Jan 24 2020 does not say that we necessarily - // queue a notification task here, but WPT tests such as - // performance-timeline/multiple-buffered-flag-observers.any.js - // assume we do, and we get intermittent race condition - // test results if we don't. - if !self.pending_notification_observers_task.get() { - self.pending_notification_observers_task.set(true); - let task_source = self.global().performance_timeline_task_source(); - task_source.queue_notification(&self.global()); - } + if !self.pending_notification_observers_task.get() { + self.pending_notification_observers_task.set(true); + let task_source = self.global().performance_timeline_task_source(); + task_source.queue_notification(&self.global()); } } let mut observers = self.observers.borrow_mut(); -- cgit v1.2.3