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..777919f6f5127a084fd45e14b31c32c52fc1c5ca |
| --- /dev/null |
| +++ b/LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-reference.html |
| @@ -0,0 +1,75 @@ |
| +<!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> |
| + <video id="testVideo"></video> |
| + <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. |
|
xhwang
2014/03/26 20:58:32
I'll be nicer to have a more detailed document lev
jrummell
2014/03/26 22:19:03
Done.
|
| + test(function() |
| + { |
| + var video = document.getElementById("testVideo"); |
|
xhwang
2014/03/26 20:58:32
Since you are using MediaKeys only, no need to use
jrummell
2014/03/26 22:19:03
Done.
|
| + var initData = new Uint8Array(['n', 'o', 't', 'h', 'i', 'n', 'g']); |
|
xhwang
2014/03/26 20:58:32
This is probably not what you intended. I tried it
jrummell
2014/03/26 22:19:03
Done.
|
| + var startingObjects = window.internals.activeDOMObjectCount(document); |
|
xhwang
2014/03/26 20:58:32
startingActiveDOMObjectsCreated?
jrummell
2014/03/26 22:19:03
Done.
|
| + |
| + function numObjectsCreated() |
|
xhwang
2014/03/26 20:58:32
numActiveDOMObjectsCreated?
jrummell
2014/03/26 22:19:03
Done.
|
| + { |
| + return window.internals.activeDOMObjectCount(document) - startingObjects; |
| + } |
| + |
| + var mediaKeys = new MediaKeys("org.w3.clearkey"); |
| + assert_equals(mediaKeys.keySystem, 'org.w3.clearkey'); |
| + |
| + // Run gc(), should not affect count since MediaKeys are not |
| + // ActiveDOMObjects. |
|
xhwang
2014/03/26 20:58:32
This is a bit confusing... gc() here should not af
jrummell
2014/03/26 22:19:03
Done.
|
| + gc(); |
| + assert_equals(numObjectsCreated(), 0, "MediaKeys created an ActiveDOMObject"); |
| + |
| + // Create 3 sessions. |
| + var mediaKeySession1 = mediaKeys.createSession("video/webm", initData); |
| + assert_equals(numObjectsCreated(), 1); |
| + var mediaKeySession2 = mediaKeys.createSession("video/webm", initData); |
| + assert_equals(numObjectsCreated(), 2); |
| + var mediaKeySession3 = mediaKeys.createSession("video/webm", initData); |
| + assert_equals(numObjectsCreated(), 3); |
| + |
| + // Run gc(). All sessions should remain as we have a reference |
| + // to each one. |
| + gc(); |
| + assert_equals(numObjectsCreated(), 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(numObjectsCreated(), 3); |
| + |
| + // Now drop the reference to MediaKeys. It and the 2 |
| + // unreferenced sessions should be collected. Run gc() twice, |
| + // in case the first pass finds the sessions first, and only |
| + // frees MediaKeys. |
|
xhwang
2014/03/26 20:58:32
Why gc() will only free MediaKeys if in the first
jrummell
2014/03/26 22:19:03
Done.
|
| + mediaKeys = null; |
| + gc(); |
| + gc(); |
| + assert_equals(numObjectsCreated(), 1); |
| + |
| + // Drop the reference to the first session. It should get |
| + // collected now since MediaKeys is gone. |
| + mediaKeySession1 = null; |
| + gc(); |
| + assert_equals(numObjectsCreated(), 0); |
| + }, "MediaKeySession lifetime using references", { timeout: 60000 }); |
|
xhwang
2014/03/26 20:58:32
s/using references/without release()/ ?
jrummell
2014/03/26 22:19:03
Done.
|
| + </script> |
| + </body> |
| +</html> |
