Closed
Bug 1367585
Opened 7 years ago
Closed 7 years ago
TPS tab tests fail
Categories
(Firefox :: Sync, enhancement)
Firefox
Sync
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 hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
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
Comment 4•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2fa3616b2435
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in
before you can comment on or make changes to this bug.
Description
•