Bug 1313758 - set media.getusermedia.browser.enabled to default false; r=jib draft
authorMunro Mengjue Chiang <mchiang@mozilla.com>
Wed, 30 Nov 2016 15:03:48 +0800
changeset 445760 1637779d55f4d8deddb7b1a516db9e1fd3242de6
parent 445717 a69583d2dbc6fdc18f63761a89cf539c356668be
child 538607 f852cd205f1c31ec8b7ed0634e337570668fc40f
push id37603
push usermchiang@mozilla.com
push dateWed, 30 Nov 2016 07:04:18 +0000
reviewersjib
bugs1313758
milestone53.0a1
Bug 1313758 - set media.getusermedia.browser.enabled to default false; r=jib MozReview-Commit-ID: 54YTn2p4nnX
dom/media/tests/mochitest/test_getUserMedia_basicTabshare.html
modules/libpref/init/all.js
--- a/dom/media/tests/mochitest/test_getUserMedia_basicTabshare.html
+++ b/dom/media/tests/mochitest/test_getUserMedia_basicTabshare.html
@@ -18,18 +18,20 @@
    */
   runTest(function () {
     const isWinXP = navigator.userAgent.indexOf("Windows NT 5.1") != -1;
     if (IsMacOSX10_6orOlder() || isWinXP) {
         ok(true, "Screensharing disabled for OSX10.6 and WinXP");
         return;
     }
     var testVideo = createMediaElement('video', 'testVideo');
+    var pushPrefs = (...p) => SpecialPowers.pushPrefEnv({set: p});
 
     return Promise.resolve()
+      .then(() => pushPrefs(["media.getusermedia.browser.enabled", true]))
       .then(() => getUserMedia({
         video: { mediaSource: "browser",
                  scrollWithPage: true },
         fake: false
       }))
       .then(stream => {
         var playback = new LocalMediaStreamPlayback(testVideo, stream);
         return playback.playMediaWithDeprecatedStreamStop(false);
--- a/modules/libpref/init/all.js
+++ b/modules/libpref/init/all.js
@@ -462,17 +462,17 @@ pref("media.peerconnection.video.enabled
 pref("media.navigator.video.max_fs", 12288); // Enough for 2048x1536
 pref("media.navigator.video.max_fr", 60);
 pref("media.navigator.video.h264.level", 31); // 0x42E01f - level 3.1
 pref("media.navigator.video.h264.max_br", 0);
 pref("media.navigator.video.h264.max_mbps", 0);
 pref("media.peerconnection.video.h264_enabled", false);
 pref("media.peerconnection.video.vp9_enabled", true);
 pref("media.getusermedia.aec", 1);
-pref("media.getusermedia.browser.enabled", true);
+pref("media.getusermedia.browser.enabled", false);
 #endif
 // Gonk typically captures at QVGA, and so min resolution is QQVGA or
 // 160x120; 100Kbps is plenty for that.
 // Desktop is typically VGA capture or more; and qm_select will not drop resolution
 // below 1/2 in each dimension (or so), so QVGA (320x200) is the lowest here usually.
 pref("media.peerconnection.video.min_bitrate", 0);
 pref("media.peerconnection.video.start_bitrate", 0);
 pref("media.peerconnection.video.max_bitrate", 0);