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

Side by Side Diff: chromecast/browser/media/cma_media_pipeline_client.cc

Issue 1531543002: [Chromecast] Adding pause/resume operations to VideoPlaneController. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Just modified a comment back the way it was. Created 5 years 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 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 "chromecast/browser/media/cma_media_pipeline_client.h" 5 #include "chromecast/browser/media/cma_media_pipeline_client.h"
6 #include "chromecast/public/cast_media_shlib.h" 6 #include "chromecast/public/cast_media_shlib.h"
7 7
8 namespace chromecast { 8 namespace chromecast {
9 namespace media { 9 namespace media {
10 10
(...skipping 18 matching lines...) Expand all
29 } 29 }
30 30
31 void CmaMediaPipelineClient::OnMediaPipelineBackendDestroyed() { 31 void CmaMediaPipelineClient::OnMediaPipelineBackendDestroyed() {
32 DCHECK(thread_checker_.CalledOnValidThread()); 32 DCHECK(thread_checker_.CalledOnValidThread());
33 media_pipeline_count_--; 33 media_pipeline_count_--;
34 34
35 if (media_pipeline_count_ == 0) 35 if (media_pipeline_count_ == 0)
36 NotifyResourceReleased(CastResource::kResourceNone); 36 NotifyResourceReleased(CastResource::kResourceNone);
37 } 37 }
38 38
39 void CmaMediaPipelineClient::InitializeResource(
40 CastResource::Resource resource) {}
byungchul 2015/12/18 00:28:28 wrap line between { and }?
esum 2015/12/21 00:39:21 git cl format wants the braces on the same line.
41
39 void CmaMediaPipelineClient::ReleaseResource(CastResource::Resource resource) { 42 void CmaMediaPipelineClient::ReleaseResource(CastResource::Resource resource) {
40 CastResource::Resource audio_video_resource = 43 CastResource::Resource audio_video_resource =
41 static_cast<CastResource::Resource>(CastResource::kResourceAudio | 44 static_cast<CastResource::Resource>(CastResource::kResourceAudio |
42 CastResource::kResourceScreenPrimary); 45 CastResource::kResourceScreenPrimary);
43 46
44 // TODO(yucliu): media pipeline need to stop audio video seperately 47 // TODO(yucliu): media pipeline need to stop audio video seperately
45 if (!(resource & audio_video_resource)) 48 if (!(resource & audio_video_resource))
46 NotifyResourceReleased(audio_video_resource); 49 NotifyResourceReleased(audio_video_resource);
47 } 50 }
48 51
49 } // namespace media 52 } // namespace media
50 } // namespace chromecast 53 } // namespace chromecast
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698