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

Unified Diff: components/arc/net/arc_net_host_impl.h

Issue 2133503002: arc: Revamp the ArcBridgeService interface (Closed) Base URL: https://chromium.googlesource.com/a/chromium/src.git@master
Patch Set: Fix ui_arc_unittests Created 4 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: components/arc/net/arc_net_host_impl.h
diff --git a/components/arc/net/arc_net_host_impl.h b/components/arc/net/arc_net_host_impl.h
index 8aa52b75982e88623330d0ee50e01a1a09535309..2085764d328d2821428544d9e93814944bf3301b 100644
--- a/components/arc/net/arc_net_host_impl.h
+++ b/components/arc/net/arc_net_host_impl.h
@@ -30,10 +30,11 @@ namespace arc {
class ArcBridgeService;
// Private implementation of ArcNetHost.
-class ArcNetHostImpl : public ArcService,
- public ArcBridgeService::Observer,
- public chromeos::NetworkStateHandlerObserver,
- public mojom::NetHost {
+class ArcNetHostImpl
+ : public ArcService,
+ public ArcBridgeService::InstanceObserver<mojom::NetInstance>,
+ public chromeos::NetworkStateHandlerObserver,
+ public mojom::NetHost {
public:
// The constructor will register an Observer with ArcBridgeService.
explicit ArcNetHostImpl(ArcBridgeService* arc_bridge_service);
@@ -77,8 +78,9 @@ class ArcNetHostImpl : public ArcService,
void DeviceListChanged() override;
void GetDefaultNetwork(const GetDefaultNetworkCallback& callback) override;
- // Overridden from ArcBridgeService::Observer:
- void OnNetInstanceReady() override;
+ // Overridden from ArcBridgeService::InterfaceObserver<mojom::NetInstance>:
+ void OnInstanceReady(mojom::NetInstance* net_instance,
+ uint32_t version) override;
private:
void DefaultNetworkSuccessCallback(const std::string& service_path,

Powered by Google App Engine
This is Rietveld 408576698