third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-async-setcert-with-gc.html - Issue 2084053002: EME: Update tests so 'audioCapabilities' always provided

Side by Side Diff: third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-async-setcert-with-gc.html

Issue 2084053002: EME: Update tests so 'audioCapabilities' always provided (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: configuration functions Created 4 years, 6 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« no previous file with comments | « third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-async-creation-with-gc.html ('k') | third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-clear-key-invalid-license.html » ('j') | third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-utils.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <!DOCTYPE html>	1 <!DOCTYPE html>

2 <html>	2 <html>

3 <head>	3 <head>

4 <title>Test asynchronous setServerCertificate while running garbage coll ection</title>	4 <title>Test asynchronous setServerCertificate while running garbage coll ection</title>

5 <script src="encrypted-media-utils.js"></script>	5 <script src="encrypted-media-utils.js"></script>

6 <script src="../../resources/testharness.js"></script>	6 <script src="../../resources/testharness.js"></script>

7 <script src="../../resources/testharnessreport.js"></script>	7 <script src="../../resources/testharnessreport.js"></script>

8 </head>	8 </head>

9 <body>	9 <body>

10 <div id="log"></div>	10 <div id="log"></div>

11 <script>	11 <script>

12 // Run garbage collection continuously.	12 // Run garbage collection continuously.

13 setInterval(asyncGC, 0);	13 setInterval(asyncGC, 0);

14	14

15 promise_test(function(test)	15 promise_test(function(test)

16 {	16 {

17 return navigator.requestMediaKeySystemAccess('org.w3.clearkey', [{}]).then(function(access) {	17 return getSupportedInitDataType().then(function(initDataType) {

	18 return navigator.requestMediaKeySystemAccess('org.w3.clearke y', simpleConfigurationForInitDataType(initDataType));
	ddorwin 2016/06/23 22:52:38 As discussed, if we're not actually using initData As discussed, if we're not actually using initData in a test, we don't need to include it in the configuration. That allows us to keep this simple like it was before. If these tests are to be used for spec tests where not all user agents support WebM and/or Vorbis (your chosen simple configuration), such a promise-based mechanism will be necessary (see also my comments in the .js file.). However, since this is pretty isolated and does not address that, I think we should keep it simple. I think we should just call simpleConfiguration() her and the few other files that don't need initDataType and aren't already WebM-specific. jrummell 2016/06/24 00:12:29 Done. Show quoted text On 2016/06/23 22:52:38, ddorwin wrote: > As discussed, if we're not actually using initData in a test, we don't need to > include it in the configuration. That allows us to keep this simple like it was > before. > > If these tests are to be used for spec tests where not all user agents support > WebM and/or Vorbis (your chosen simple configuration), such a promise-based > mechanism will be necessary (see also my comments in the .js file.). However, > since this is pretty isolated and does not address that, I think we should keep > it simple. > > I think we should just call simpleConfiguration() her and the few other files > that don't need initDataType and aren't already WebM-specific. Done.
	19 }).then(function(access) {

18 return access.createMediaKeys();	20 return access.createMediaKeys();

19 }).then(function(mediaKeys) {	21 }).then(function(mediaKeys) {

20 var cert = new Uint8Array(200);	22 var cert = new Uint8Array(200);

21 return mediaKeys.setServerCertificate(cert);	23 return mediaKeys.setServerCertificate(cert);

22 }).then(function(result) {	24 }).then(function(result) {

23 assert_false(result);	25 assert_false(result);

24 });	26 });

25 }, 'Test asynchronous setServerCertificate while running garbage col lection.');	27 }, 'Test asynchronous setServerCertificate while running garbage col lection.');

26 </script>	28 </script>

27 </body>	29 </body>

28 </html>	30 </html>

OLD	NEW