Closed Bug 1367585 Opened 7 years ago Closed 7 years ago

TPS tab tests fail

Categories

(Firefox :: Sync, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 55
Tracking Status
firefox55 --- fixed

People

(Reporter: tcsc, Assigned: tcsc)

Details

Attachments

(1 file)

It seems like load events are dispatched synchronously for data uris, and so we never get the event unless we add it before the new tab.

I'm going to assume this is a result of the performance work we've been doing lately, and it's easy enough to fix.
Comment on attachment 8871037 [details]
Bug 1367585 - Add the 'load' event listener before adding the new tab in sync's TPS tests

https://reviewboard.mozilla.org/r/142592/#review146204
Attachment #8871037 - Flags: review?(markh) → review+
Pushed by tchiovoloni@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2fa3616b2435
Add the 'load' event listener before adding the new tab in sync's TPS tests r=markh
https://hg.mozilla.org/mozilla-central/rev/2fa3616b2435
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: