Remove DemuxerFactory and URL parameter from Pipeline.

chrome/test/data/prerender/prerender_html5_common.js

Index: chrome/test/data/prerender/prerender_html5_common.js
diff --git a/chrome/test/data/prerender/prerender_html5_common.js b/chrome/test/data/prerender/prerender_html5_common.js
index 09d196b4744c3b9fb631a7009b318c0912807461..7779c5a48693d663e841f4f7dbbd05f02cb2b7ca 100644
--- a/chrome/test/data/prerender/prerender_html5_common.js
+++ b/chrome/test/data/prerender/prerender_html5_common.js
@@ -20,7 +20,6 @@ function assert(bool) {
 var canPlaySeen = false;
 var playingSeen = false;
 var canPlayThroughSeen = false;
-var stalledSeen = false;
 var loadStartSeen = false;
 var hasError = false;
 
@@ -49,16 +48,16 @@ function mediaEventHandler(e) {
       loadStartSeen = true;
       break;
     case 'stalled':
-      assert(loadStartSeen);
-      stalledSeen = true;
+      // We should never see a stalled event during the display portion of the
+      // test.
+      assert(false);

[scherkus (not reviewing), 2012/03/28 00:24:34]

shishir: can you take a look at this?

[Shishir, 2012/03/28 18:41:33]

From the webkit code
(third_party/WebKit/Source/WebCore/html/HTMLMediaElement.cpp), if there is no
progress events for 3 seconds, the stalled will fire. That seemed to be
happening when the test was written. Has something changed?

[scherkus (not reviewing), 2012/03/30 21:40:45]

This CL apparently fixes that behaviour, which I would consider to be a bug. I'm
still digging in to see why that is.

       break;
   }
 
-  var stallDone = !testNetworkEvents || stalledSeen;
   var progressDone = (willPlay && canPlayThroughSeen && playingSeen) ||
       (!willPlay && canPlayThroughSeen && !playingSeen);
 
-  if (stallDone && progressDone)
+  if (progressDone)
     document.title = 'PASS';
 }