No --disable-raw option, and raw decoder not built if all other media types disabled

RESOLVED FIXED in mozilla2.0b7

Status

()

RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: cpearce, Assigned: khuey)

Tracking

Trunk
mozilla2.0b7
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
There does not seems to be a --disable-raw build option. In fact it looks like raw video can't be disabled, and its configure section doesn't ensure MOZ_MEDIA is defined, so raw video will only be enabled when some other media type is enabled and that sets MOZ_MEDIA. i.e. If we build with --disable-wave, --disable-webm, and --disable-ogg, despite the fact that MOZ_RAW will still be defined, and the pref media.raw.enabled will be true, raw support won't be built because MOZ_MEDIA isn't defined.

There should be a --disable-raw option, and if raw video is not disabled, its configure section should ensure MOZ_MEDIA is defined, like the other media types' configure sections.
(Reporter)

Updated

8 years ago
Blocks: 508082
Assignee: nobody → khuey
(Reporter)

Comment 2

8 years ago
Comment on attachment 469031 [details] [diff] [review]
Patch

Great, thanks!
Attachment #469031 - Flags: review?(chris) → review+
Comment on attachment 469031 [details] [diff] [review]
Patch

Trivial build only consistency fix.  We should take this for 2.0.
Attachment #469031 - Flags: approval2.0?
Comment on attachment 469031 [details] [diff] [review]
Patch

a=sdwilsh
Attachment #469031 - Flags: approval2.0? → approval2.0+
http://hg.mozilla.org/mozilla-central/rev/b03fab95dda0
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b6
You need to log in before you can comment on or make changes to this bug.