Closed Bug 809972 Opened 9 years ago Closed 9 years ago
.js: gracefully handle missing ns IAudio Manager on desktop builds
Bug 791642 broke the desktop builds. settings.js now tries to access Ci.nsIAudioManager no matter what. We should handle this gracefully to not break desktop builds.
Nom'ing for basecamp blocking because desktop build breakage = sad panda.
blocking-basecamp: --- → ?
Assignee: nobody → philipp
Attachment #679765 - Flags: review?(fabrice)
Dupe of bug 809577, slightly different fix. I tend to prefer this one though.
Attachment #679765 - Flags: review?(fabrice) → review+
Note that I had already landed bug 809577 on inbound when it was duped to this one, so the patch here will need to be rebased. https://hg.mozilla.org/integration/mozilla-inbound/rev/6f9dd07d9d64
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
I'd like to ask for approval to land this on aurora as well. This fix makes it possible to change the locale (among other settings) in the desktop build. It's important to our localization effort and testing.
Comment on attachment 679777 [details] [diff] [review] v2 r=fabrice Requesting approval-mozilla-aurora in case blocking-basecamp is denied. [Approval Request Comment] Bug caused by (feature/regressing bug #): 791642 User impact if declined: see comment 10 Testing completed (on m-c, etc.): Already tested on m-c / b2g nightlies. Risk to taking this patch (and alternatives if risky): virtually none String or UUID changes made by this patch: none
Attachment #679777 - Flags: approval-mozilla-aurora?
l10n blocker, approving for Aurora 18.
blocking-basecamp: ? → +
Priority: -- → P1
Target Milestone: --- → B2G C1 (to 19nov)
Attachment #679777 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.