Closed Bug 1256766 Opened 8 years ago Closed 8 years ago

Make sure youtube flash rewrite tests work when pref is turned off

Categories

(Core :: DOM: Core & HTML, defect)

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla48
Tracking Flags:
Tracking Status
firefox48 --- fixed

People

(Reporter: qdot, Assigned: qdot)

References

Details

(Whiteboard: btpp-active)

Attachments

(1 file)

Patch 1 (v1) - Make sure all objects load afer pref set for youtube rewrite test
4.64 KB, patch
bzbarsky
: review+
Details | Diff | Splinter Review 
The tests for bug 769117 assume the pref is always on, which may not be the case. The test should account for the pref being off and turn it on when needed.
Are you still working on this, qDot?
Flags: needinfo?(kyle)
Hoping to get to it soon, just been a lot of things piling up.
Flags: needinfo?(kyle)
Whiteboard: btpp-active
Update of test from bug 769117. Just need to delay loading of static tags until after pref is set, so moving those into an iframe.
Attachment #8740202 - Flags: review?(bzbarsky)
Comment on attachment 8740202 [details] [diff] [review]
Patch 1 (v1) - Make sure all objects load afer pref set for youtube rewrite test

r=me
Attachment #8740202 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/mozilla-central/rev/345e0864f665
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox48: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.