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

Unified Diff: components/arc/arc_bridge_host_impl.h

Issue 2552213002: Remove explicit singletonness of ArcBridgeService part 1. (Closed)
Patch Set: git cl format Created 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | components/arc/arc_bridge_host_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/arc/arc_bridge_host_impl.h
diff --git a/components/arc/arc_bridge_host_impl.h b/components/arc/arc_bridge_host_impl.h
index b99a4953f2fa87d5b62d6a0a9b3888b87e64d77a..b42072583a46b74ad5465168d45af6620b7677b3 100644
--- a/components/arc/arc_bridge_host_impl.h
+++ b/components/arc/arc_bridge_host_impl.h
@@ -17,6 +17,8 @@
namespace arc {
+class ArcBridgeService;
+
// Implementation of the ArcBridgeHost.
// The lifetime of ArcBridgeHost and ArcBridgeInstance mojo channels are tied
// to this instance. Also, any ARC related Mojo channel will be closed if
@@ -31,7 +33,8 @@ class ArcBridgeHostImpl : public mojom::ArcBridgeHost {
// Interface to keep the Mojo channel InterfacePtr.
class MojoChannel;
- explicit ArcBridgeHostImpl(mojom::ArcBridgeInstancePtr instance);
+ ArcBridgeHostImpl(ArcBridgeService* arc_bridge_service,
+ mojom::ArcBridgeInstancePtr instance);
~ArcBridgeHostImpl() override;
// ArcBridgeHost overrides.
@@ -86,6 +89,9 @@ class ArcBridgeHostImpl : public mojom::ArcBridgeHost {
base::ThreadChecker thread_checker_;
+ // Owned by ArcServiceManager.
+ ArcBridgeService* const arc_bridge_service_;
+
mojo::Binding<mojom::ArcBridgeHost> binding_;
mojom::ArcBridgeInstancePtr instance_;
« no previous file with comments | « no previous file | components/arc/arc_bridge_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698