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

Side by Side Diff: components/arc/video/arc_video_bridge.cc

Issue 1596663002: arc-bridge: Introduce the ArcService class (Closed) Base URL: https://chromium.googlesource.com/a/chromium/src.git@master
Patch Set: Rebased + protected ArcService direct instantiation Created 4 years, 11 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 | « components/arc/video/arc_video_bridge.h ('k') | ui/arc/notification/arc_notification_manager.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "components/arc/video/arc_video_bridge.h" 5 #include "components/arc/video/arc_video_bridge.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 namespace arc { 9 namespace arc {
10 10
11 ArcVideoBridge::ArcVideoBridge( 11 ArcVideoBridge::ArcVideoBridge(
12 ArcBridgeService* bridge_service,
12 scoped_ptr<VideoHostDelegate> video_host_delegate) 13 scoped_ptr<VideoHostDelegate> video_host_delegate)
13 : video_host_delegate_(std::move(video_host_delegate)), 14 : ArcService(bridge_service),
14 binding_(video_host_delegate_.get()) {} 15 video_host_delegate_(std::move(video_host_delegate)),
16 binding_(video_host_delegate_.get()) {
17 arc_bridge_service()->AddObserver(this);
18 }
15 19
16 ArcVideoBridge::~ArcVideoBridge() { 20 ArcVideoBridge::~ArcVideoBridge() {
17 arc::ArcBridgeService* bridge_service = arc::ArcBridgeService::Get(); 21 arc_bridge_service()->RemoveObserver(this);
18 DCHECK(bridge_service);
19 bridge_service->RemoveObserver(this);
20 }
21
22 void ArcVideoBridge::StartObservingBridgeServiceChanges() {
23 arc::ArcBridgeService* bridge_service = arc::ArcBridgeService::Get();
24 DCHECK(bridge_service);
25 bridge_service->AddObserver(this);
26
27 // If VideoInstance was ready before we AddObserver(), we won't get
28 // OnVideoInstanceReady events. For such case, we have to call it explicitly.
29 if (bridge_service->video_instance())
30 OnVideoInstanceReady();
31 } 22 }
32 23
33 void ArcVideoBridge::OnStateChanged(arc::ArcBridgeService::State state) { 24 void ArcVideoBridge::OnStateChanged(arc::ArcBridgeService::State state) {
34 switch (state) { 25 switch (state) {
35 case arc::ArcBridgeService::State::STOPPING: 26 case arc::ArcBridgeService::State::STOPPING:
36 video_host_delegate_->OnStopping(); 27 video_host_delegate_->OnStopping();
37 break; 28 break;
38 default: 29 default:
39 break; 30 break;
40 } 31 }
41 } 32 }
42 33
43 void ArcVideoBridge::OnVideoInstanceReady() { 34 void ArcVideoBridge::OnVideoInstanceReady() {
44 arc::ArcBridgeService* bridge_service = arc::ArcBridgeService::Get();
45 DCHECK(bridge_service);
46
47 arc::VideoHostPtr host; 35 arc::VideoHostPtr host;
48 binding_.Bind(mojo::GetProxy(&host)); 36 binding_.Bind(mojo::GetProxy(&host));
49 bridge_service->video_instance()->Init(std::move(host)); 37 arc_bridge_service()->video_instance()->Init(std::move(host));
50 } 38 }
51 39
52 } // namespace arc 40 } // namespace arc
OLDNEW
« no previous file with comments | « components/arc/video/arc_video_bridge.h ('k') | ui/arc/notification/arc_notification_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698