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

Unified Diff: LayoutTests/http/tests/media/media-source/mediasource-util.js

Issue 20114005: Layout Test for basic MSE seek scenario. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fix indentation. Created 7 years, 4 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/http/tests/media/media-source/mediasource-util.js
diff --git a/LayoutTests/http/tests/media/media-source/mediasource-util.js b/LayoutTests/http/tests/media/media-source/mediasource-util.js
index 456f012f4d99dc613dbdf3ac07f224d518097e76..b1cdf5a7e558883821078f74dfa14e80500a7bd9 100644
--- a/LayoutTests/http/tests/media/media-source/mediasource-util.js
+++ b/LayoutTests/http/tests/media/media-source/mediasource-util.js
@@ -4,6 +4,7 @@
this.test_ = test;
this.eventTargetList_ = [];
this.waitCallbacks_ = [];
+ this.timeoutIDs_ = {};
};
EventExpectationsManager.prototype.expectEvent = function(object, eventName, description)
@@ -29,6 +30,29 @@
object.addEventListener(eventName, eventHandler);
};
+ EventExpectationsManager.prototype.expectDelayedCallback = function(callback, delay)
+ {
+ var timeoutIDHolder = 0;
+ var timeoutIDs = this.timeoutIDs_;
+ // Define a wrapper function that will:
+ // 1. To indicate that this expected callback did fire, remove this
+ // timeoutID from the list of IDs we are maintaining.
+ // 2. Execute the callback that has been passed-in.
+ var callbackWrapper = function(callback) {
acolwell GONE FROM CHROMIUM 2013/08/05 21:06:06 nit: callback parameter isn't necessary here. Just
anandc 2013/08/06 18:03:30 Done.
+ delete timeoutIDs[timeoutIDHolder];
+ callback();
+ }
+
+ // Execute the wrapper after the specified delay.
+ timeoutIDHolder = setTimeout(
+ function() { callbackWrapper(callback); },
acolwell GONE FROM CHROMIUM 2013/08/05 21:06:06 nit: put the whole setTimeout call all on a single
anandc 2013/08/06 18:03:30 Done.
+ delay);
+
+ // Add to list of timeoutIDs, which we'll use eventually to check that
+ // all expected delayed callbacks did get executed.
+ timeoutIDs[timeoutIDHolder] = timeoutIDHolder;
+ };
+
EventExpectationsManager.prototype.waitForExpectedEvents = function(callback)
{
this.waitCallbacks_.push(callback);
@@ -176,6 +200,11 @@
test.eventExpectations_.expectEvent(object, eventName, description);
};
+ test.expectDelayedCallback = function(callback, delay)
+ {
+ test.eventExpectations_.expectDelayedCallback(callback, delay);
+ }
+
test.waitForExpectedEvents = function(callback)
{
test.eventExpectations_.waitForExpectedEvents(callback);
@@ -184,8 +213,14 @@
var oldTestDone = test.done.bind(test);
test.done = function()
{
- if (test.status == test.PASS)
+ // loop through and clear timeoutIDs
+ for (var i in test.eventExpectations_.timeoutIDs_) {
+ clearTimeout(test.eventExpectations_.timeoutIDs_[i]);
+ }
+ if (test.status == test.PASS) {
assert_false(test.eventExpectations_.expectingEvents(), "No pending event expectations.");
+ assert_equals(Object.keys(test.eventExpectations_.timeoutIDs_).length, 0);
+ }
oldTestDone();
};
};

Powered by Google App Engine
This is Rietveld 408576698