Closed
Bug 600540
Opened 14 years ago
Closed 14 years ago
Disable raw video by default for Gecko 2
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla2.0b7
People
(Reporter: khuey, Assigned: khuey)
Details
Attachments
(1 file, 1 obsolete file)
1.57 KB,
patch
|
cpearce
:
review+
beltzner
:
approval2.0+
|
Details | Diff | Splinter Review |
Camera no longer blocks Fennec 4, so raw video has no consumers. We should turn it off; there's no point in shipping code that we're not using and that just increases our attack surface.
Assignee | ||
Comment 1•14 years ago
|
||
Assignee | ||
Comment 2•14 years ago
|
||
Need this too.
Attachment #479385 -
Attachment is obsolete: true
Attachment #479388 -
Flags: review?(chris)
Attachment #479385 -
Flags: review?(chris)
Comment 3•14 years ago
|
||
Comment on attachment 479388 [details] [diff] [review] Patch 2 No need to comment out the seek.yuv entry in content/media/test/manifest.js; its type should be unsupported with raw not enabled, so we shouldn't try to play it anyway.
Attachment #479388 -
Flags: review?(chris) → review+
Assignee | ||
Comment 4•14 years ago
|
||
Comment on attachment 479388 [details] [diff] [review] Patch 2 We should approve this for the reasons listed in comment 0.
Attachment #479388 -
Flags: approval2.0?
Comment 5•14 years ago
|
||
Comment on attachment 479388 [details] [diff] [review] Patch 2 a=beltzner
Attachment #479388 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Comment 6•14 years ago
|
||
Removed the commented test and ifdefed the mime type handler stuff just like the webm stuff is. http://hg.mozilla.org/mozilla-central/rev/a701d821ef36
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Assignee | ||
Comment 7•14 years ago
|
||
This didn't actually work because there is a MOZ_RAW=1 earlier in the file.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 8•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/5911688ed38a
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Updated•14 years ago
|
Target Milestone: mozilla2.0b8 → mozilla2.0b7
You need to log in
before you can comment on or make changes to this bug.
Description
•