From 5061c392f09f73f62b8d06c42a1d1413cecca96d Mon Sep 17 00:00:00 2001 From: Chun-Min Chang Date: Mon, 15 May 2017 13:20:50 +0800 Subject: [PATCH] Bug 1331763 - part2: Test cases; r=cpearce MozReview-Commit-ID: 6dXLYq4unxa --HG-- extra : rebase_source : dc3335935881bfcf5c3f9e70a48a35a7acb6eb72 --- .../test/test_eme_stream_capture_blocked_case1.html | 5 ----- .../test/test_eme_stream_capture_blocked_case2.html | 10 +++------- .../test/test_eme_stream_capture_blocked_case3.html | 5 ----- 3 files changed, 3 insertions(+), 17 deletions(-) diff --git a/dom/media/test/test_eme_stream_capture_blocked_case1.html b/dom/media/test/test_eme_stream_capture_blocked_case1.html index 51dc0aea128e..f8d724f16822 100644 --- a/dom/media/test/test_eme_stream_capture_blocked_case1.html +++ b/dom/media/test/test_eme_stream_capture_blocked_case1.html @@ -14,11 +14,6 @@ var manager = new MediaTestManager; function startTest(test, token) { - // Three cases: - // 1. setting MediaKeys on an element captured by MediaElementSource should fail, and - // 2. creating a MediaElementSource on a media element with a MediaKeys should fail, and - // 3. capturing a media element with mozCaptureStream that has a MediaKeys should fail. - // Case 1. setting MediaKeys on an element captured by MediaElementSource should fail. var p1 = new EMEPromise; var case1token = token + "_case1"; diff --git a/dom/media/test/test_eme_stream_capture_blocked_case2.html b/dom/media/test/test_eme_stream_capture_blocked_case2.html index 2e6b826299f9..a3cb48ad59bc 100644 --- a/dom/media/test/test_eme_stream_capture_blocked_case2.html +++ b/dom/media/test/test_eme_stream_capture_blocked_case2.html @@ -14,12 +14,8 @@ var manager = new MediaTestManager; function startTest(test, token) { - // Three cases: - // 1. setting MediaKeys on an element captured by MediaElementSource should fail, and - // 2. creating a MediaElementSource on a media element with a MediaKeys should fail, and - // 3. capturing a media element with mozCaptureStream that has a MediaKeys should fail. - - // Case 2. creating a MediaElementSource on a media element with a MediaKeys should fail. + // Case 2. creating a MediaElementSource on a media element should always succeed + // (no matter whether it's restricted content or not), and var p1 = new EMEPromise; var case2token = token + "_case2"; let v2 = document.createElement("video"); @@ -33,7 +29,7 @@ function startTest(test, token) } catch (e) { threw = true; } - ok(threw, "Should throw an error creating a MediaElementSource on an EME video."); + ok(!threw, "Should always work when creating a MediaElementSource."); p1.resolve(); }); diff --git a/dom/media/test/test_eme_stream_capture_blocked_case3.html b/dom/media/test/test_eme_stream_capture_blocked_case3.html index b9e8633564ad..548f6656e5fc 100644 --- a/dom/media/test/test_eme_stream_capture_blocked_case3.html +++ b/dom/media/test/test_eme_stream_capture_blocked_case3.html @@ -14,11 +14,6 @@ var manager = new MediaTestManager; function startTest(test, token) { - // Three cases: - // 1. setting MediaKeys on an element captured by MediaElementSource should fail, and - // 2. creating a MediaElementSource on a media element with a MediaKeys should fail, and - // 3. capturing a media element with mozCaptureStream that has a MediaKeys should fail. - // Case 3. capturing a media element with mozCaptureStream that has a MediaKeys should fail. var p1 = new EMEPromise; var case3token = token + "_case3";