Layout Test for basic MSE seek scenario.

LayoutTests/http/tests/media/media-source/mediasource-util.js

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..370f820c06e03a98f0a0f6351cb1936fc0503f8a 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_ = new Array();

[wolenetz, 2013/08/02 20:04:25]

new Array() is the same as []. At least according to JSON.stringify(). Do you
mean to use {} or []? Any of these three appear to work when I tested in private
(with the caveat that removing '[i]' in the enumeration below is necessary to
make any of these options work - see my later comment), so I'm not sure why 'new
Array()' is used here. Also, I'm not sure Array is most efficient here vs just
Object properties by way of {}.

[anandc, 2013/08/02 20:40:38]

Thanks for trying out the different options. Going with {} now.

     };
 
     EventExpectationsManager.prototype.expectEvent = function(object, eventName, description)
@@ -29,6 +30,26 @@
         object.addEventListener(eventName, eventHandler);
     };
 
+    EventExpectationsManager.prototype.expectDelayedCallback = function(callback, delay)
+    {
+        var timeoutIDHolder = 0;
+        // 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, timeoutIDs) {
+          delete timeoutIDs[timeoutIDHolder];
+          callback();
+        }
+
+        // Execute the wrapper after the specified delay.
+        var timeoutIDs = this.timeoutIDs_;
+        timeoutIDHolder = setTimeout(function() {callbackWrapper(callback, timeoutIDs);}, delay);
+        // Add to list of timeoutIDs, which we'll use eventually to check that
+        // all expected delayed callbacks did get executed.
+        this.timeoutIDs_[timeoutIDHolder] = timeoutIDHolder;
+    };
+
     EventExpectationsManager.prototype.waitForExpectedEvents = function(callback)
     {
         this.waitCallbacks_.push(callback);
@@ -176,6 +197,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 +210,15 @@
         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_[i]) {

[wolenetz, 2013/08/02 20:04:25]

I believe this [i] breaks the enumeration code.
Example (compare the resulting s and v, below):
a=new Array();
--> []
a
--> []
a[100]=100;
--> 100
a
--> Array[101]
s = new String();
--> String {}
for (var i in a) { s+=(a[i]).toString()+',' }
--> "100,"
s
--> "100,"
v = new String();
--> String {}
for (var i in a[i]) { v+=(a[i]).toString()+',' }
--> undefined
v
--> String {}

[anandc, 2013/08/02 20:40:38]

Stupid bug. Thanks for catching. Fixed.

+                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();
         };
     };