Closed Bug 1905634 Opened 1 year ago Closed 1 year ago

Incorrect usage of preprocessor conditionals around media engine prefs

Categories

(Core :: Audio/Video: Playback, defect)

defect

Tracking

()

RESOLVED FIXED
129 Branch
Tracking Status
firefox129 --- fixed

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

Attachments

(1 file)

It looks to me like there's a mistake in the conditionals that landed in bug 1901084 around media-engine prefs. The symbol DEV_EDITION_OR_EARLY_BETA_OR_EARLIER that was created there is always #defined (either as true or false), which would be fine if used as the value of a boolean pref (like we do with other similarly-defined IS_... symbols). But using it in an #if defined(...) test will not have the intended effect, because whether it's true or false, it is defined in either case.

Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/84a6b4d37c01 Fix usage of conditionals around media engine prefs in StaticPrefList.yaml. r=alwu
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 129 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: