Chromium Code Reviews Chromium Code Reviews
Issue 209103002:
Add lifetime tests for MediaKeySession (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-reference.html |
| diff --git a/LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-reference.html b/LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-reference.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..f294be752e43d956f50d902c3943859a9da75932 |
| --- /dev/null |
| +++ b/LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-reference.html |
| @@ -0,0 +1,77 @@ |
| +<!DOCTYPE html> |
|
ddorwin
2014/03/27 17:20:36
Once the Decryptor lifetimes are fixed, we'll want
|
| +<html> |
| + <head> |
| + <title>Test MediaKeySession lifetime</title> |
| + <script src="../w3c-media-utils.js"></script> |
| + <script src="../../resources/testharness.js"></script> |
| + <script src="../../resources/testharnessreport.js"></script> |
| + <script src="../../resources/gc.js"></script> |
| + </head> |
| + <body> |
| + <div id="log"></div> |
| + <script> |
| + setup({ timeout: 60000 }); // Timeout for all tests to run. |
|
ddorwin
2014/03/27 17:20:36
We shouldn't need these anymore:
https://src.chrom
jrummell
2014/03/27 18:42:56
Done.
|
| + |
| + // Since MediaKeySession (but not MediaKeys) are ActiveDOMObjects, |
| + // we can determine when they are garbage collected. |
| + // MediaKeySessions remain as long as: |
| + // JavaScript has a reference to it |
| + // OR (MediaKeys is around AND the session has not received a close() event) |
| + test(function() |
| + { |
| + var initData = new Uint8Array([0, 1, 2, 3, 4, 5, 6, 7]); |
| + var startingActiveDOMObjectCount = window.internals.activeDOMObjectCount(document); |
| + |
| + function numActiveDOMObjectsCreated() |
| + { |
| + return window.internals.activeDOMObjectCount(document) - startingActiveDOMObjectCount; |
| + } |
| + |
| + var mediaKeys = new MediaKeys("org.w3.clearkey"); |
| + assert_equals(mediaKeys.keySystem, 'org.w3.clearkey'); |
| + |
| + // Verify MediaKeys are not ActiveDOMObjects. |
| + assert_equals(numActiveDOMObjectsCreated(), 0, "MediaKeys created an ActiveDOMObject"); |
| + |
| + // Create 3 sessions. |
| + var mediaKeySession1 = mediaKeys.createSession("video/webm", initData); |
| + assert_equals(numActiveDOMObjectsCreated(), 1); |
| + var mediaKeySession2 = mediaKeys.createSession("video/webm", initData); |
| + assert_equals(numActiveDOMObjectsCreated(), 2); |
| + var mediaKeySession3 = mediaKeys.createSession("video/webm", initData); |
| + assert_equals(numActiveDOMObjectsCreated(), 3); |
| + |
| + // Run gc(). All sessions should remain as we have a reference |
| + // to each one. |
| + gc(); |
| + assert_equals(numActiveDOMObjectsCreated(), 3); |
| + |
| + // Now drop references to 2 of the sessions. Even though we |
| + // don't have a reference, MediaKeys is still around (and the |
| + // sessions aren't closed), so the objects won't be collected. |
| + mediaKeySession2 = null; |
| + mediaKeySession3 = null; |
| + gc(); |
| + assert_equals(numActiveDOMObjectsCreated(), 3); |
| + |
| + // Now drop the reference to MediaKeys. It and the 2 |
| + // unreferenced sessions should be collected. Since |
| + // MediaKeySessions remain alive as long as MediaKeys is around, |
| + // it is possible that gc() checks one or both MediaKeySession |
| + // objects first, and doesn't collect them since MediaKeys |
| + // hasn't been collected yet. Thus run gc() twice, to ensure |
| + // that the unreferenced MediaKeySession objects get collected. |
| + mediaKeys = null; |
| + gc(); |
| + gc(); |
| + assert_equals(numActiveDOMObjectsCreated(), 1); |
| + |
| + // Drop the reference to the first session. It should get |
| + // collected now since MediaKeys is gone. |
| + mediaKeySession1 = null; |
| + gc(); |
| + assert_equals(numActiveDOMObjectsCreated(), 0); |
| + }, "MediaKeySession lifetime without release()", { timeout: 60000 }); |
| + </script> |
| + </body> |
| +</html> |
