Bug 1315737 - Test that allowed_domains is no longer needed for screensharing. draft
authorJan-Ivar Bruaroey <jib@mozilla.com>
Mon, 07 Nov 2016 13:51:58 -0500
changeset 434929 3a6996922b6b8bc469800f0585da903f5d1140c6
parent 434926 c83c11ef636df75014ab23be5d9a830e1832b99f
child 536160 461d4adce15a25befada9412fd66bcb2921c7287
push id34874
push userjbruaroey@mozilla.com
push dateMon, 07 Nov 2016 18:52:22 +0000
bugs1315737
milestone52.0a1
Bug 1315737 - Test that allowed_domains is no longer needed for screensharing. MozReview-Commit-ID: FNMuqq9fzKu
dom/media/tests/mochitest/test_getUserMedia_basicScreenshare.html
--- a/dom/media/tests/mochitest/test_getUserMedia_basicScreenshare.html
+++ b/dom/media/tests/mochitest/test_getUserMedia_basicScreenshare.html
@@ -6,20 +6,16 @@
 <body>
 <pre id="test">
 <script type="application/javascript">
   createHTML({
     title: "getUserMedia Basic Screenshare Test",
     bug: "1211656"
   });
 
-  var mustFailWith = (msg, reason, f) =>
-    f().then(() => ok(false, msg + " must fail"),
-             e => is(e.name, reason, msg + " must fail: " + e.message));
-
   var pushPrefs = (...p) => new Promise(r => SpecialPowers.pushPrefEnv({set: p}, r));
 
   /**
    * Run a test to verify that we can complete a start and stop media playback
    * cycle for a screenshare LocalMediaStream on a video HTMLMediaElement.
    */
   runTest(function () {
     const isWinXP = navigator.userAgent.indexOf("Windows NT 5.1") != -1;
@@ -58,25 +54,19 @@
         height: 200,
         frameRate: {
           min: '5',
           max: '10'
         }
       }
     ];
     return Promise.resolve()
+      // Screensharing must work even without "mochi.test," in allowed_domains
       .then(() => pushPrefs(["media.getusermedia.screensharing.allowed_domains",
                              "mozilla.github.io,*.bugzilla.mozilla.org"]))
-      .then(() => mustFailWith("Screensharing if absent in allowed_domains",
-                               "NotAllowedError",
-                               () => navigator.mediaDevices.getUserMedia({
-                                 video: videoConstraints[0], fake: false
-                               })))
-      .then(() => pushPrefs(["media.getusermedia.screensharing.allowed_domains",
-                             "mozilla.github.io,mochi.test,*.bugzilla.mozilla.org"]))
       .then(() => getUserMedia(constraints).then(stream => {
         var playback = new LocalMediaStreamPlayback(testVideo, stream);
         return playback.playMediaWithDeprecatedStreamStop(false);
       }))
       .then(() => getUserMedia({video: videoConstraints[0], fake: false}))
       .then(stream => {
         var playback = new LocalMediaStreamPlayback(testVideo, stream);
         playback.startMedia(false);