Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1082)

Side by Side Diff: media/mojo/services/service_factory_impl.cc

Issue 2411573002: media: Use new wrapper types for media mojo interfaces (Closed)
Patch Set: comments addressed Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « media/mojo/services/service_factory_impl.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/mojo/services/service_factory_impl.h" 5 #include "media/mojo/services/service_factory_impl.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/threading/thread_task_runner_handle.h" 8 #include "base/threading/thread_task_runner_handle.h"
9 #include "media/base/media_log.h" 9 #include "media/base/media_log.h"
10 #include "media/mojo/services/mojo_media_client.h" 10 #include "media/mojo/services/mojo_media_client.h"
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 void ServiceFactoryImpl::CreateVideoDecoder( 78 void ServiceFactoryImpl::CreateVideoDecoder(
79 mojom::VideoDecoderRequest request) { 79 mojom::VideoDecoderRequest request) {
80 #if defined(ENABLE_MOJO_VIDEO_DECODER) 80 #if defined(ENABLE_MOJO_VIDEO_DECODER)
81 mojo::MakeStrongBinding( 81 mojo::MakeStrongBinding(
82 base::MakeUnique<MojoVideoDecoderService>(mojo_media_client_), 82 base::MakeUnique<MojoVideoDecoderService>(mojo_media_client_),
83 std::move(request)); 83 std::move(request));
84 #endif // defined(ENABLE_MOJO_VIDEO_DECODER) 84 #endif // defined(ENABLE_MOJO_VIDEO_DECODER)
85 } 85 }
86 86
87 void ServiceFactoryImpl::CreateRenderer( 87 void ServiceFactoryImpl::CreateRenderer(
88 const mojo::String& audio_device_id, 88 const std::string& audio_device_id,
89 mojo::InterfaceRequest<mojom::Renderer> request) { 89 mojo::InterfaceRequest<mojom::Renderer> request) {
90 #if defined(ENABLE_MOJO_RENDERER) 90 #if defined(ENABLE_MOJO_RENDERER)
91 RendererFactory* renderer_factory = GetRendererFactory(); 91 RendererFactory* renderer_factory = GetRendererFactory();
92 if (!renderer_factory) 92 if (!renderer_factory)
93 return; 93 return;
94 94
95 scoped_refptr<base::SingleThreadTaskRunner> task_runner( 95 scoped_refptr<base::SingleThreadTaskRunner> task_runner(
96 base::ThreadTaskRunnerHandle::Get()); 96 base::ThreadTaskRunnerHandle::Get());
97 auto audio_sink = 97 auto audio_sink =
98 mojo_media_client_->CreateAudioRendererSink(audio_device_id); 98 mojo_media_client_->CreateAudioRendererSink(audio_device_id);
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
139 CdmFactory* ServiceFactoryImpl::GetCdmFactory() { 139 CdmFactory* ServiceFactoryImpl::GetCdmFactory() {
140 if (!cdm_factory_) { 140 if (!cdm_factory_) {
141 cdm_factory_ = mojo_media_client_->CreateCdmFactory(interfaces_.get()); 141 cdm_factory_ = mojo_media_client_->CreateCdmFactory(interfaces_.get());
142 LOG_IF(ERROR, !cdm_factory_) << "CdmFactory not available."; 142 LOG_IF(ERROR, !cdm_factory_) << "CdmFactory not available.";
143 } 143 }
144 return cdm_factory_.get(); 144 return cdm_factory_.get();
145 } 145 }
146 #endif // defined(ENABLE_MOJO_CDM) 146 #endif // defined(ENABLE_MOJO_CDM)
147 147
148 } // namespace media 148 } // namespace media
OLDNEW
« no previous file with comments | « media/mojo/services/service_factory_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698