Refactoring BufferedDataSource to work with WebURLLoader instead of a MediaResourceLoaderBridge.

chrome/renderer/render_view.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/renderer/render_view.h"
 
 #include <algorithm>
 #include <cmath>
 #include <string>
 #include <vector>
@@ skipping 2644 matching lines @@
       return NULL;
 
     // Add the hardware video decoder factory.
     // TODO(hclam): This will cause the renderer process to crash on context
     // lost.
     bool ret = context->makeContextCurrent();
     CHECK(ret) << "Failed to switch context";
     collection->AddVideoDecoder(new IpcVideoDecoder(
         MessageLoop::current(), context->context()));
   }
-
+/*
   WebApplicationCacheHostImpl* appcache_host =

[scherkus (not reviewing), 2010/11/30 20:05:39]

has this been addressed yet?

I'd ping michaeln over IM

[annacc, 2010/11/30 21:51:45]

Will do.

       WebApplicationCacheHostImpl::FromFrame(frame);
-
-  // TODO(hclam): obtain the following parameters from |client|.
-  // Create two bridge factory for two data sources.
-  webkit_glue::MediaResourceLoaderBridgeFactory* bridge_factory_simple =
-      new webkit_glue::MediaResourceLoaderBridgeFactory(
-          GURL(frame->url()),  // referrer
-          "null",  // frame origin
-          "null",  // main_frame_origin
-          base::GetCurrentProcId(),
-          appcache_host ? appcache_host->host_id() : appcache::kNoHostId,
-          routing_id());
-
-  webkit_glue::MediaResourceLoaderBridgeFactory* bridge_factory_buffered =
-      new webkit_glue::MediaResourceLoaderBridgeFactory(
-          GURL(frame->url()),  // referrer
-          "null",  // frame origin
-          "null",  // main_frame_origin
-          base::GetCurrentProcId(),
-          appcache_host ? appcache_host->host_id() : appcache::kNoHostId,
-          routing_id());
-
+*/
   scoped_refptr<webkit_glue::WebVideoRenderer> video_renderer;
   bool pts_logging = cmd_line->HasSwitch(switches::kEnableVideoLogging);
   scoped_refptr<webkit_glue::VideoRendererImpl> renderer(
       new webkit_glue::VideoRendererImpl(pts_logging));
   collection->AddVideoRenderer(renderer);
   video_renderer = renderer;
 
   return new webkit_glue::WebMediaPlayerImpl(
-      client, collection.release(), bridge_factory_simple,
-      bridge_factory_buffered,
-      cmd_line->HasSwitch(switches::kSimpleDataSource),video_renderer);
+      client, collection.release(), frame,
+      cmd_line->HasSwitch(switches::kSimpleDataSource),
+      video_renderer);
 }
 
 WebApplicationCacheHost* RenderView::createApplicationCacheHost(
     WebFrame* frame, WebApplicationCacheHostClient* client) {
   return new RendererWebApplicationCacheHostImpl(
       FromWebView(frame->view()), client,
       RenderThread::current()->appcache_dispatcher()->backend_proxy());
 }
 
 WebCookieJar* RenderView::cookieJar() {
@@ skipping 2791 matching lines @@
       IPC::PlatformFileForTransitToPlatformFile(file_for_transit),
       message_id);
 }
 
 #if defined(OS_MACOSX)
 void RenderView::OnSelectPopupMenuItem(int selected_index) {
   external_popup_menu_->DidSelectItem(selected_index);
   external_popup_menu_.reset();
 }
 #endif