Bug 1309299 - make navigator.requestMediaKeySystemAccess() reject with TypeError for empty keysystem string or config. r=gerald

This makes us match the spec and pass more cases of
http://www.w3c-test.org/encrypted-media/drm-mp4-syntax-mediakeysystemaccess.html
but we still need to adjust the test so that it's consistent, as pointed out
by bz.

MozReview-Commit-ID: KEfRk4QnJKR

--HG--
extra : source : 72af37aaf2a0a42c20e64b3b455f0cb79e2b7eb1

dom/media/eme/MediaKeySystemAccessManager.cpp

@@ -65,11 +65,6 @@ MediaKeySystemAccessManager::Request(DetailedPromise* aPromise,
                                      const nsAString& aKeySystem,
                                      const Sequence<MediaKeySystemConfiguration>& aConfigs)
 {
-  if (aKeySystem.IsEmpty() || aConfigs.IsEmpty()) {
-    aPromise->MaybeReject(NS_ERROR_DOM_INVALID_ACCESS_ERR,
-                          NS_LITERAL_CSTRING("Invalid keysystem type or invalid options sequence"));
-    return;
-  }
   Request(aPromise, aKeySystem, aConfigs, RequestType::Initial);
 }
 

testing/web-platform/meta/encrypted-media/Google/encrypted-media-requestmediakeysystemaccess.html.ini

@@ -0,0 +1,6 @@
+[encrypted-media-requestmediakeysystemaccess.html]
+  type: testharness
+  [Empty keySystem]
+    expected: FAIL
+  [Empty supportedConfigurations]
+    expected: FAIL

testing/web-platform/meta/encrypted-media/Google/encrypted-media-syntax.html.ini

@@ -21,3 +21,5 @@
   [Test MediaKeys setServerCertificate().]
     expected: TIMEOUT
 
+  [Test Navigator.requestMediaKeySystemAccess() exceptions.]
+    expected: TIMEOUT