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-release.html |
| diff --git a/LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-release.html b/LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-release.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..b81d5c0473a166a87bfbdd1d746f8c45e6a4b940 |
| --- /dev/null |
| +++ b/LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-release.html |
| @@ -0,0 +1,77 @@ |
| +<!DOCTYPE html> |
| +<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. |
| + |
| + // 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) |
| + async_test(function(test) |
| + { |
| + var startingActiveDOMObjectCount = window.internals.activeDOMObjectCount(document); |
| + |
| + function numActiveDOMObjectsCreated() |
| + { |
| + return window.internals.activeDOMObjectCount(document) - startingActiveDOMObjectCount; |
| + } |
| + |
| + // Create 2 sessions. |
| + var mediaKeys = new MediaKeys("org.w3.clearkey"); |
| + var mediaKeySession1 = mediaKeys.createSession('video/webm', new Uint8Array([1, 2, 3])); |
| + assert_equals(numActiveDOMObjectsCreated(), 1); |
| + var mediaKeySession2 = mediaKeys.createSession('video/webm', new Uint8Array([4, 5]));; |
|
xhwang
2014/03/27 17:07:00
nit: can we do the same as the other file, i.e.
v
jrummell
2014/03/27 18:42:56
Done.
|
| + assert_equals(numActiveDOMObjectsCreated(), 2); |
| + var sessionsCreated = 2; |
|
ddorwin
2014/03/27 17:20:36
nit: activeSessions or openSessions? They aren't n
jrummell
2014/03/27 18:42:56
Done.
|
| + |
| + // Release the sessions. Once the close() event is received, |
| + // only the JS reference to them keeps them around. |
|
ddorwin
2014/03/27 17:20:36
It would be nice to have another test (might make
jrummell
2014/03/27 18:42:56
Good idea. Added another test.
|
| + mediaKeySession1.release(); |
| + waitForEventAndRunStep("close", mediaKeySession1, onClose, test); |
| + mediaKeySession2.release(); |
| + waitForEventAndRunStep("close", mediaKeySession2, onClose, test); |
| + |
| + function onClose(event) |
| + { |
| + --sessionsCreated; |
| + if (sessionsCreated > 0) |
| + return; |
| + |
| + // Delay to give time for close to complete since |
| + // event.target is a reference to the MediaKeySession. |
| + setTimeout(finish, 1); |
| + } |
| + |
| + function finish() |
| + { |
| + // Since both sessions have been closed, dropping the |
| + // reference to them from JS will result in the session |
| + // being garbage-collected. |
| + consoleWrite("Finish"); |
|
ddorwin
2014/03/27 17:20:36
nit: Do we need this? Is there a message that woul
jrummell
2014/03/27 18:42:56
Changed. This is just to ensure that it only ran a
|
| + assert_not_equals(mediaKeys, null); |
| + assert_equals(numActiveDOMObjectsCreated(), 2); |
| + |
| + mediaKeySession1 = null; |
| + gc(); |
| + assert_equals(numActiveDOMObjectsCreated(), 1, "mediaKeySession1 not collected"); |
| + |
| + mediaKeySession2 = null; |
| + gc(); |
| + assert_equals(numActiveDOMObjectsCreated(), 0, "mediaKeySession2 not collected"); |
| + |
| + test.done(); |
| + } |
| + }, "MediaKeySession lifetime after release()", { timeout: 60000 }); |
| + </script> |
| + </body> |
| +</html> |
