Bug 1340163 - Fix a test failure related to ContentPrincipalInfo.origin usage, r=me CLOSED TREE
authorAndrea Marchesini <amarchesini@mozilla.com>
Mon, 20 Mar 2017 18:28:52 +0100
changeset 348490 0fe17d74ce9ea07f9ff0308b4c3619206e563fcb
parent 348489 1ecf3f3dde1e740e4cda4a5202fa9d6a444f1887
child 348491 474f08673e24ff21a161d4f3c6ce5b1623b528f8
push id31526
push userkwierso@gmail.com
push dateTue, 21 Mar 2017 01:20:02 +0000
treeherdermozilla-central@5fe5dcf1c10a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1340163
milestone55.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 1340163 - Fix a test failure related to ContentPrincipalInfo.origin usage, r=me CLOSED TREE
dom/media/tests/mochitest/test_enumerateDevices.html
--- a/dom/media/tests/mochitest/test_enumerateDevices.html
+++ b/dom/media/tests/mochitest/test_enumerateDevices.html
@@ -112,18 +112,17 @@ runTest(async () => {
   is(differentOriginDevices.length, devices.length);
   [...sameOriginDevices, ...differentOriginDevices].forEach(d => validateDevice(d));
 
   for (let device of sameOriginDevices) {
     ok(devices.find(d => d.deviceId == device.deviceId),
        "Same origin deviceId for " + device.label + " must match");
   }
   for (let device of differentOriginDevices) {
-    // TODO: s/todo/ok/ once bug 1340163 is fixed.
-    todo(!devices.find(d => d.deviceId == device.deviceId),
+    ok(!devices.find(d => d.deviceId == device.deviceId),
          "Different origin deviceId for " + device.label + " must be different");
   }
 
   // Check the special case of no devices found.
   await pushPrefs(["media.navigator.streams.fake", false],
                   ["media.audio_loopback_dev", "none"],
                   ["media.video_loopback_dev", "none"]);
   devices = await navigator.mediaDevices.enumerateDevices();