Chromium Code Reviews| Index: media/renderers/default_renderer_factory.cc |
| diff --git a/media/renderers/default_renderer_factory.cc b/media/renderers/default_renderer_factory.cc |
| index 88fd15789c27ab84cfe9c350a09bca74753072ef..457d2fa70355b42854bf5c0f55eb8dc3e314aa34 100644 |
| --- a/media/renderers/default_renderer_factory.cc |
| +++ b/media/renderers/default_renderer_factory.cc |
| @@ -69,6 +69,9 @@ ScopedVector<VideoDecoder> DefaultRendererFactory::CreateVideoDecoders( |
| // Create our video decoders and renderer. |
| ScopedVector<VideoDecoder> video_decoders; |
| + if (decoder_factory_) |
| + decoder_factory_->CreateVideoDecoders(&video_decoders); |
|
xhwang
2016/03/15 05:13:18
Add a comment why we like to put this as the first
sandersd (OOO until July 31)
2016/03/15 18:31:02
Done.
|
| + |
| // |gpu_factories_| requires that its entry points be called on its |
| // |GetTaskRunner()|. Since |pipeline_| will own decoders created from the |
| // factories, require that their message loops are identical. |
| @@ -87,9 +90,6 @@ ScopedVector<VideoDecoder> DefaultRendererFactory::CreateVideoDecoders( |
| video_decoders.push_back(new FFmpegVideoDecoder()); |
| #endif |
| - if (decoder_factory_) |
| - decoder_factory_->CreateVideoDecoders(&video_decoders); |
| - |
| return video_decoders; |
| } |