Re-Relanding 'Always notify the WebMediaPlayer of any seek' patch

LayoutTests/media/media-controller-playbackrate.html

Index: LayoutTests/media/media-controller-playbackrate.html
diff --git a/LayoutTests/media/media-controller-playbackrate.html b/LayoutTests/media/media-controller-playbackrate.html
index 3ea2ed562e1599ee5b7c41429f21ee91e6d88d32..7df5e4b92ee693d4e1384ce778d91101690dfd38 100644
--- a/LayoutTests/media/media-controller-playbackrate.html
+++ b/LayoutTests/media/media-controller-playbackrate.html
@@ -4,13 +4,15 @@
         <script src=media-file.js></script>
         <script src=video-test.js></script>
         <script>
-            var start = function() {
+            var start = function()

[philipj_slow, 2015/08/04 11:16:47]

Please revert the style changes, it's OK that different tests have different
styles as long as they're internally consistent.

[Srirama, 2015/08/04 12:03:21]

Done.

+            {
                 findMediaElement();
-                waitForEvent('canplay',canplay);
+                waitForEventOnce('canplay',canplay);
                 video.src = findMediaFile('video', 'content/test');
             };
 
-            var canplay = function() {
+            var canplay = function()
+            {
                 video.mediaGroup = "group";
 
                 // Test non-finite rates.
@@ -32,17 +34,23 @@
                 waitForEventOnce('seeked', seeked);
             };
 
-            var seeked = function() {
+            var seeked = function()
+            {
                 waitForEvent('play', play);
                 video.play();
             };
 
-            var play = function() {
+            var play = function()
+            {
                 video.controller.playbackRate = 2;
-                waitForEventOnce('timeupdate', timeupdate);
+                waitForEventOnce("playing", function()
+                {
+                    waitForEventOnce("timeupdate", timeupdate);

[philipj_slow, 2015/08/04 11:16:47]

Please break this out into a "var playing = function()". Nesting all of the
functions would also be OK, but would make for a bigger diff.

[Srirama, 2015/08/04 12:03:21]

Done.

+                });
             };
 
-            var timeupdate = function() {
+            var timeupdate = function()
+            {
                 testExpected('video.currentTime', 0, '!=');
                 endTest();
             };