MSE: Start letting SourceBuffer begin to do initialization segment received algorithm

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 b181cdc5f698c54b274ab0a8c81a380c15e8c97e..ffa6a1e6ece7e7efd11c758d6a9556ed165042dd 100644
--- a/LayoutTests/http/tests/media/media-source/mediasource-util.js
+++ b/LayoutTests/http/tests/media/media-source/mediasource-util.js
@@ -360,7 +360,7 @@
         }, description, options);
     };
 
-    window['mediasource_testafterdataloaded'] = function(testFunction, description, options)
+    window['mediasource_testafterdataloaded_internal'] = function(testFunction, description, failOnMediaElementError, options)

[acolwell GONE FROM CHROMIUM, 2014/09/10 17:16:26]

Don't make this available on the window object since this is an internal method
now.

[wolenetz, 2014/09/10 21:18:22]

Done (see next comment).

     {
         mediasource_test(function(test, mediaElement, mediaSource)
         {
@@ -371,7 +371,8 @@
                 return;
             }
 
-            test.failOnEvent(mediaElement, 'error');
+            if (failOnMediaElementError)
+                test.failOnEvent(mediaElement, 'error');
 
             var sourceBuffer = mediaSource.addSourceBuffer(segmentInfo.type);
             MediaSourceUtil.loadBinaryData(test, segmentInfo.url, function(mediaData)
@@ -379,7 +380,17 @@
                 testFunction(test, mediaElement, mediaSource, segmentInfo, sourceBuffer, mediaData);
             });
         }, description, options);
-    }
+    };
+
+    window['mediasource_testafterdataloaded'] = function(testFunction, description, options)
+    {
+        mediasource_testafterdataloaded_internal(testFunction, description, true, options);
+    };
+
+    window['mediasource_testafterdataloaded_nofailonerror'] = function(testFunction, description, options)

[acolwell GONE FROM CHROMIUM, 2014/09/10 17:16:26]

any reason not to use |options| for this instead of creating a different method?

[wolenetz, 2014/09/10 21:18:22]

Done (renamed options->properties to match async_test() naming, and added a
recognizer for a new property to the original mediasource_testafterdataloaded()
function). So long as there's no name collision on this new property vs
test_harness.js over time, code is simplified.

+    {
+        mediasource_testafterdataloaded_internal(testFunction, description, false, options);
+    };
 
     function timeRangesToString(ranges)
     {