Layout Test for basic MSE seek scenario.

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

 (function(window) {
     EventExpectationsManager = function(test)
     {
         this.test_ = test;
         this.eventTargetList_ = [];
         this.waitCallbacks_ = [];
+        this.timeoutIDs_ = {};

[wolenetz, 2013/08/06 22:35:55]

Oh noes! This also needs removal :)

[anandc, 2013/08/06 23:27:22]

Many thanks for catching these. Done.

     };
 
     EventExpectationsManager.prototype.expectEvent = function(object, eventName, description)
     {
         var eventInfo = { 'target': object, 'type': eventName, 'description': description};
         var expectations = this.getExpectations_(object);
         expectations.push(eventInfo);
 
         var t = this;
         var waitHandler = this.test_.step_func(function() { t.handleWaitCallback_(); });
@@ skipping 11 matching lines @@
         });
         object.addEventListener(eventName, eventHandler);
     };
 
     EventExpectationsManager.prototype.waitForExpectedEvents = function(callback)
     {
         this.waitCallbacks_.push(callback);
         setTimeout(this.handleWaitCallback_.bind(this), 0);
     };
 
+    EventExpectationsManager.prototype.waitForTimeUpdate = function(mediaElement, callback)
+    {
+        var initialTime = mediaElement.currentTime;
+
+        var onTimeUpdate = this.test_.step_func(function()
+        {
+            if (mediaElement.currentTime != initialTime) {
+                mediaElement.removeEventListener('timeupdate', onTimeUpdate);
+                callback();
+            }
+        });
+
+        mediaElement.addEventListener('timeupdate', onTimeUpdate);
+    };
+
     EventExpectationsManager.prototype.expectingEvents = function()
     {
         for (var i = 0; i < this.eventTargetList_.length; ++i) {
             if (this.eventTargetList_[i].expectations.length > 0) {
                 return true;
             }
         }
         return false;
     }
 
@@ skipping 121 matching lines @@
         {
             object.addEventListener(eventName, test.step_func(function(event) { test.done(); }));
         };
 
         test.eventExpectations_ = new EventExpectationsManager(test);
         test.expectEvent = function(object, eventName, description)
         {
             test.eventExpectations_.expectEvent(object, eventName, description);
         };
 
+        test.waitForTimeUpdate = function(mediaElement, callback)

[wolenetz, 2013/08/06 22:35:55]

nit: Please add brief documentation here that not just timeupdate is sufficient
for callback, but it will keep listening for timeupdates and only stop and
invoke callback once mediaElement.currentTime has changed.

[anandc, 2013/08/06 22:49:37]

Good point. Done.

[acolwell GONE FROM CHROMIUM, 2013/08/06 23:05:09]

nit: An alternative is to just change the method name to
waitForCurrentTimeChange.

[anandc, 2013/08/06 23:27:22]

Renaming is better. Done.

+        {
+            test.eventExpectations_.waitForTimeUpdate(mediaElement, callback);

[acolwell GONE FROM CHROMIUM, 2013/08/06 23:05:09]

nit: Why is added as part of the EventExpectationsManager? It doesn't use any
functionality in that class. I think you should put all the code in here
instead.

[anandc, 2013/08/06 23:27:22]

Of course. :-) Thank you. Done.

+        }
+
         test.waitForExpectedEvents = function(callback)
         {
             test.eventExpectations_.waitForExpectedEvents(callback);
         };
 
         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_) {

[acolwell GONE FROM CHROMIUM, 2013/08/06 23:05:09]

nit: I think this can be removed since timeoutIDs doesn't appear to be used
anywhere.

[anandc, 2013/08/06 23:27:22]

Yes, removed.

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

[acolwell GONE FROM CHROMIUM, 2013/08/06 23:05:09]

nit: ditto

[anandc, 2013/08/06 23:27:22]

Done.

+            }
             oldTestDone();
         };
     };
 
     window['MediaSourceUtil'] = MediaSourceUtil;
     window['media_test'] = function(testFunction, description, options)
     {
         options = options || {};
         return async_test(function(test)
         {
@@ skipping 21 matching lines @@
             };
 
             openMediaSource_(test, mediaTag, function(mediaSource)
             {
                 testFunction(test, mediaTag, mediaSource);
             });
         }, description, options);
 
     };
 })(window);