Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(48)

Unified Diff: LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-reference.html

Issue 209103002: Add lifetime tests for MediaKeySession (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Move gc() Created 6 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-reference-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..2c25a174f9bc67b3dea6a4161a9898a3bc16ac13
--- /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>
+ <div id="log"></div>
+ <script>
+ // 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()");
+ </script>
+ </body>
+</html>
« no previous file with comments | « no previous file | LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-reference-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698