Accepting request 573267 from home:fstrba:branches:mozilla:Factory

Fix bmo#1430274, ALSA sound not working in Firefox 58 due to sandboxing

OBS-URL: https://build.opensuse.org/request/show/573267
OBS-URL: https://build.opensuse.org/package/show/mozilla:Factory/MozillaFirefox?expand=0&rev=632

MozillaFirefox.changes

@@ -1,3 +1,10 @@
+-------------------------------------------------------------------
+Tue Feb  6 07:03:42 UTC 2018 - fstrba@suse.com
+
+- Added patch:
+  * mozilla-alsa-sandbox.patch: Fix bmo#1430274, ALSA sound (still
+    or again?) not working in Firefox 58 due to sandboxing.
+
 -------------------------------------------------------------------
 Mon Jan 29 22:32:21 UTC 2018 - wr@rosenauer.org
 

MozillaFirefox.spec

@@ -155,6 +155,7 @@ Patch6:         mozilla-reduce-files-per-UnifiedBindings.patch
 Patch7:         mozilla-aarch64-startup-crash.patch
 Patch8:         mozilla-enable-csd.patch
 Patch9:         mozilla-bmo256180.patch
+Patch10:        mozilla-alsa-sandbox.patch
 # Firefox/browser
 Patch101:       firefox-kde.patch
 Patch102:       firefox-no-default-ualocale.patch
@@ -267,6 +268,7 @@ cd $RPM_BUILD_DIR/mozilla
 %patch7 -p1
 %patch8 -p1
 %patch9 -p1
+%patch10 -p1
 # Firefox
 %patch101 -p1
 %patch102 -p1

mozilla-alsa-sandbox.patch

@@ -0,0 +1,12 @@
+--- mozilla/security/sandbox/linux/moz.build	2018-02-06 07:58:27.127145777 +0100
++++ mozilla/security/sandbox/linux/moz.build	2018-02-06 08:01:22.213706297 +0100
+@@ -77,6 +77,9 @@
+         'SandboxOpenedFiles.cpp',
+     ]
+ 
++if CONFIG['MOZ_ALSA']:
++    DEFINES['MOZ_ALSA'] = True
++
+ # This copy of SafeSPrintf doesn't need to avoid the Chromium logging
+ # dependency like the one in libxul does, but this way the behavior is
+ # consistent.  See also the comment in SandboxLogging.h.