Closed
Bug 1159756
Opened 9 years ago
Closed 9 years ago
Stop forcing the media.mediasource.enabled preference to true in the test harness
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
Details
Attachments
(2 files)
2.26 KB,
patch
|
jgraham
:
review+
|
Details | Diff | Splinter Review |
36.28 KB,
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
This makes it impossible to properly test that we don't ship support when we don't mean to, for example. The tests that rely on this preference should set it as needed.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8599641 -
Flags: review?(james)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8599642 -
Flags: review?(kinetik)
Updated•9 years ago
|
Attachment #8599642 -
Flags: review?(kinetik) → review+
Comment 3•9 years ago
|
||
Comment on attachment 8599641 [details] [diff] [review] part 1. Fix web platform test harness to work correctly when trying to set a boolean pref to false Review of attachment 8599641 [details] [diff] [review]: ----------------------------------------------------------------- Oops. Thanks for the patch, but please don't land here; I'll merge it upstream and then update the harness in m-i.
Attachment #8599641 -
Flags: review?(james) → review+
Assignee | ||
Comment 6•9 years ago
|
||
Followup is because apparently MediaSource got enabled on b2g yesterday.
Comment 7•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/fe44cfb27e75 https://hg.mozilla.org/mozilla-central/rev/2b79e2614294
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in
before you can comment on or make changes to this bug.
Description
•