Bug 1519415 - Don't use https://example.com in test_getUserMedia_permission to avoid timeouts on Android. r=jib
authorJohann Hofmann <jhofmann@mozilla.com>
Tue, 15 Jan 2019 20:17:47 +0000
changeset 514029 9ff32c3351ac657b82370b774825f35ebdb05136
parent 514028 6e9eaee838be152b9680da2a1ea11b9418713b15
child 514030 e2b32bfc7522e311d487fbc1193c3e9348ab647c
push id1953
push userffxbld-merge
push dateMon, 11 Mar 2019 12:10:20 +0000
treeherdermozilla-release@9c35dcbaa899 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjib
bugs1519415
milestone66.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1519415 - Don't use https://example.com in test_getUserMedia_permission to avoid timeouts on Android. r=jib For some reason that I could not figure out, usage of any https host in these getUserMedia mochitests on Android causes a network proxy error and will, in this particular case, lead to the iframe not loading the test page. This causes our test to timeout. I don't think testing https here is particularly important, and so I'd rather save some effort for myself and just remove it. Differential Revision: https://phabricator.services.mozilla.com/D16538
dom/media/tests/mochitest/test_getUserMedia_permission.html
--- a/dom/media/tests/mochitest/test_getUserMedia_permission.html
+++ b/dom/media/tests/mochitest/test_getUserMedia_permission.html
@@ -50,17 +50,17 @@ runTest(async () => {
       setTimeout(() => { throw e; });
     }
   }
 
   const source = `<html\><script\>(${sourceFn.toString()})()</script\></html\>`;
 
   // Test gUM in sandboxed vs. regular iframe.
 
-  for (const origin of ["http://mochi.test:8888", "https://example.com", "http://test1.mochi.test:8888"]) {
+  for (const origin of ["http://mochi.test:8888", "http://test1.mochi.test:8888"]) {
     const src = origin + path;
     is(await iframeGum({ src, sandbox: "allow-scripts" }),
        "NotAllowedError", "gUM fails in sandboxed iframe " + origin);
     is(await iframeGum({ src, sandbox: "allow-scripts allow-same-origin " + origin }),
        "success", "gUM works in regular iframe");
   }
 
   // Test gUM in sandboxed vs regular srcdoc iframe