Closed Bug 1255784 Opened 8 years ago Closed 8 years ago

u2f tests broken in terms of their pref handling

Categories

(Core :: DOM: Device Interfaces, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: bzbarsky, Assigned: baku)

References

Details

Attachments

(1 file)

They're racy in that they assume that nothing enumerates the window before they've set their prefs, for example.  This is blocking other tests (e.g. bug 1255692) from landing because minor timing changes will make the u2f tests fail.

They're also using SpecialPowers to directly set their prefs and leaving them in random states (not to mention not really being e10s-friendly) after the test, instead of using pushPrefEnv like tests are supposed to.
Flags: needinfo?(jjones)
Flags: needinfo?(dkeeler)
Flags: needinfo?(amarchesini)
Blocks: 1255692
See bug 1255692 comment 5 for the sorts of problems this causes.  Basically, right now these tests are preventing any worker tests from being added to the tree.
Attached patch u2f.patchSplinter Review
Assignee: nobody → amarchesini
Flags: needinfo?(jjones)
Flags: needinfo?(dkeeler)
Flags: needinfo?(amarchesini)
Attachment #8729542 - Flags: review?(bzbarsky)
Comment on attachment 8729542 [details] [diff] [review]
u2f.patch

Please have someone from bug 1231681 review this; I have too many other things on my plate right now, sorry.
Attachment #8729542 - Flags: review?(bzbarsky)
Attachment #8729542 - Flags: review?(jjones)
If this can't land in a very short time and we don't have 100% confidence that it will fix the issues observed in the dependencies, but 1231681 needs to be backed out ASAP.
Flags: needinfo?(jjones)
Comment on attachment 8729542 [details] [diff] [review]
u2f.patch

Review of attachment 8729542 [details] [diff] [review]:
-----------------------------------------------------------------

Everything looks like the tests are still good. I'm sorry I didn't know about the Push/Pop pref handling a month ago!
Attachment #8729542 - Flags: review?(jjones) → review+
Flags: needinfo?(jjones)
Thanks for the quick fix guys!
Yes, thank you!
Component: DOM: Security → DOM: Device Interfaces
https://hg.mozilla.org/mozilla-central/rev/8836b439d0b3
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: