diff options
| author | acolwell@chromium.org <acolwell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-15 07:09:27 +0000 |
|---|---|---|
| committer | acolwell@chromium.org <acolwell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-15 07:09:27 +0000 |
| commit | f78d1dfc8d92a6667a355e90da7f2f100c89b832 (patch) | |
| tree | bcba6d266d612b0df072cf32241c83d1ba6b897a /chrome | |
| parent | 4e425be43fae960cad4dc81be0181bf99f2f7080 (diff) | |
| download | chromium_src-f78d1dfc8d92a6667a355e90da7f2f100c89b832.zip chromium_src-f78d1dfc8d92a6667a355e90da7f2f100c89b832.tar.gz chromium_src-f78d1dfc8d92a6667a355e90da7f2f100c89b832.tar.bz2 | |
Remove MessageLoop methods from Filter interface to
separate Filter management from MessageLoop management.
This sets the stage for filters to share threads in the
future which will reduce resource consumption when
multiple <video> tags are on the same page.
BUG=69451
TEST=None
Review URL: http://codereview.chromium.org/6171009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71548 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/renderer/media/audio_renderer_impl_unittest.cc | 1 | ||||
| -rw-r--r-- | chrome/renderer/render_view.cc | 7 |
2 files changed, 6 insertions, 2 deletions
diff --git a/chrome/renderer/media/audio_renderer_impl_unittest.cc b/chrome/renderer/media/audio_renderer_impl_unittest.cc index b5f80e6..249c987 100644 --- a/chrome/renderer/media/audio_renderer_impl_unittest.cc +++ b/chrome/renderer/media/audio_renderer_impl_unittest.cc @@ -46,7 +46,6 @@ class AudioRendererImplTest : public ::testing::Test { // Create and initialize audio renderer. renderer_ = new AudioRendererImpl(filter_); renderer_->set_host(&host_); - renderer_->set_message_loop(message_loop_.get()); renderer_->Initialize(decoder_, media::NewExpectedCallback()); // Run pending tasks and simulate responding with a created audio stream. diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc index 9395eca..f16ae6f 100644 --- a/chrome/renderer/render_view.cc +++ b/chrome/renderer/render_view.cc @@ -106,6 +106,7 @@ #include "grit/renderer_resources.h" #include "media/base/filter_collection.h" #include "media/base/media_switches.h" +#include "media/base/message_loop_factory_impl.h" #include "net/base/data_url.h" #include "net/base/escape.h" #include "net/base/net_errors.h" @@ -2811,6 +2812,8 @@ WebSharedWorker* RenderView::createSharedWorker( WebMediaPlayer* RenderView::createMediaPlayer( WebFrame* frame, WebMediaPlayerClient* client) { + scoped_ptr<media::MessageLoopFactory> message_loop_factory( + new media::MessageLoopFactoryImpl()); scoped_ptr<media::FilterCollection> collection( new media::FilterCollection()); @@ -2846,7 +2849,9 @@ WebMediaPlayer* RenderView::createMediaPlayer( video_renderer = renderer; scoped_ptr<webkit_glue::WebMediaPlayerImpl> result( - new webkit_glue::WebMediaPlayerImpl(client, collection.release())); + new webkit_glue::WebMediaPlayerImpl(client, + collection.release(), + message_loop_factory.release())); if (!result->Initialize(frame, cmd_line->HasSwitch(switches::kSimpleDataSource), video_renderer)) { |