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

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

Issue 2326913003: Privatize StrongBinding lifetime management (Closed)
Patch Set: rebase Created 4 years, 3 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/mojo_renderer_service.h" 5 #include "media/mojo/services/mojo_renderer_service.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "media/base/audio_renderer_sink.h" 10 #include "media/base/audio_renderer_sink.h"
11 #include "media/base/media_keys.h" 11 #include "media/base/media_keys.h"
12 #include "media/base/media_url_demuxer.h" 12 #include "media/base/media_url_demuxer.h"
13 #include "media/base/renderer.h" 13 #include "media/base/renderer.h"
14 #include "media/base/video_renderer_sink.h" 14 #include "media/base/video_renderer_sink.h"
15 #include "media/mojo/services/demuxer_stream_provider_shim.h" 15 #include "media/mojo/services/demuxer_stream_provider_shim.h"
16 #include "media/mojo/services/mojo_cdm_service_context.h" 16 #include "media/mojo/services/mojo_cdm_service_context.h"
17 17
18 namespace media { 18 namespace media {
19 19
20 // Time interval to update media time. 20 // Time interval to update media time.
21 const int kTimeUpdateIntervalMs = 50; 21 const int kTimeUpdateIntervalMs = 50;
22 22
23 MojoRendererService::MojoRendererService( 23 MojoRendererService::MojoRendererService(
24 base::WeakPtr<MojoCdmServiceContext> mojo_cdm_service_context, 24 base::WeakPtr<MojoCdmServiceContext> mojo_cdm_service_context,
25 scoped_refptr<AudioRendererSink> audio_sink, 25 scoped_refptr<AudioRendererSink> audio_sink,
26 std::unique_ptr<VideoRendererSink> video_sink, 26 std::unique_ptr<VideoRendererSink> video_sink,
27 std::unique_ptr<media::Renderer> renderer, 27 std::unique_ptr<media::Renderer> renderer)
28 mojo::InterfaceRequest<mojom::Renderer> request) 28 : mojo_cdm_service_context_(mojo_cdm_service_context),
29 : binding_(this, std::move(request)),
30 mojo_cdm_service_context_(mojo_cdm_service_context),
31 state_(STATE_UNINITIALIZED), 29 state_(STATE_UNINITIALIZED),
32 playback_rate_(0), 30 playback_rate_(0),
33 audio_sink_(std::move(audio_sink)), 31 audio_sink_(std::move(audio_sink)),
34 video_sink_(std::move(video_sink)), 32 video_sink_(std::move(video_sink)),
35 renderer_(std::move(renderer)), 33 renderer_(std::move(renderer)),
36 weak_factory_(this) { 34 weak_factory_(this) {
37 DVLOG(1) << __FUNCTION__; 35 DVLOG(1) << __FUNCTION__;
38 DCHECK(renderer_); 36 DCHECK(renderer_);
39 37
40 weak_this_ = weak_factory_.GetWeakPtr(); 38 weak_this_ = weak_factory_.GetWeakPtr();
(...skipping 191 matching lines...) Expand 10 before | Expand all | Expand 10 after
232 bool success) { 230 bool success) {
233 DVLOG(1) << __FUNCTION__ << "(" << success << ")"; 231 DVLOG(1) << __FUNCTION__ << "(" << success << ")";
234 232
235 if (success) 233 if (success)
236 cdm_ = cdm; 234 cdm_ = cdm;
237 235
238 callback.Run(success); 236 callback.Run(success);
239 } 237 }
240 238
241 } // namespace media 239 } // namespace media
OLDNEW
« no previous file with comments | « media/mojo/services/mojo_renderer_service.h ('k') | media/mojo/services/mojo_video_decoder_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698