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

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

Issue 209103002: Add lifetime tests for MediaKeySession (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: remove onMessage() 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
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..84daaa4de6c3ffed3dfcea03447983ba038c32be
--- /dev/null
+++ b/LayoutTests/media/encrypted-media/encrypted-media-lifetime-mediakeysession-release.html
@@ -0,0 +1,76 @@
+<!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;
+ }
+
+ var mediaKeys = new MediaKeys("org.w3.clearkey");
+ var mediaKeySession1 = null;
+ var mediaKeySession2 = null;
+ var sessionsCreated = 0;
+
+ 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.
xhwang 2014/03/27 00:46:40 So really it's used so that finish() is called asy
jrummell 2014/03/27 01:22:23 Done. If we get flakiness on ASAN tests, I'll know
xhwang 2014/03/27 17:07:00 :) well, then at least we know something isn't wor
+ setTimeout(finish, 250);
+ }
+
+ function finish()
+ {
+ // Since both sessions have been released, dropping the
+ // reference to them from JS will result in the session
+ // being garbage-collected.
+ consoleWrite("Finish");
+ 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();
+ }
+
+ // Create 2 sessions.
+ mediaKeySession1 = mediaKeys.createSession('video/webm', new Uint8Array([1, 2, 3]));
+ mediaKeySession1.release();
+ waitForEventAndRunStep("close", mediaKeySession1, onClose, test);
+ mediaKeySession2 = mediaKeys.createSession('video/webm', new Uint8Array([4, 5]));
+ mediaKeySession2.release();
+ waitForEventAndRunStep("close", mediaKeySession2, onClose, test);
xhwang 2014/03/27 00:46:40 Can we move these up to replace l.30-31 to be cons
jrummell 2014/03/27 01:22:23 Done.
+ sessionsCreated = 2;
+ }, "MediaKeySession lifetime after release()", { timeout: 60000 });
+ </script>
+ </body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698