Notify ARC of default network changes

components/arc/net/arc_net_host_impl.h

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 af5df412b62de3aec91267505796c475ea05f4bb..c6561d7c4a21f56521f3dd395223836de713d320 100644
--- a/components/arc/net/arc_net_host_impl.h
+++ b/components/arc/net/arc_net_host_impl.h
@@ -66,6 +66,8 @@ class ArcNetHostImpl : public ArcService,
   // Overriden from chromeos::NetworkStateHandlerObserver.
   void ScanCompleted(const chromeos::DeviceState* /*unused*/) override;
   void OnShuttingDown() override;
+  void DefaultNetworkChanged(const chromeos::NetworkState* network) override;
+  void GetDefaultNetwork(const GetDefaultNetworkCallback& callback) override;
 
   // Overridden from ArcBridgeService::Observer:
   void OnNetInstanceReady() override;
@@ -73,6 +75,7 @@ class ArcNetHostImpl : public ArcService,
  private:
   base::ThreadChecker thread_checker_;
   mojo::Binding<arc::mojom::NetHost> binding_;
+  std::string default_network_;

[stevenjb, 2016/05/16 19:29:35]

Why store this?

[Kevin Cernekee, 2016/05/17 01:56:27]

c&p from the related comment on arc_net_host_impl.cc:

We set it as a member because we usually get a DefaultNetworkChanged event
before ArcBridgeService comes up.  In that case, ARC needs to explicitly query
the default network through the GetDefaultNetwork mojo call, so it is convenient
to have it cached.

[stevenjb, 2016/05/17 18:02:41]

I think it would be better just to call NetworkStateHandler::DefaultNetwork() in
that case.

 
   DISALLOW_COPY_AND_ASSIGN(ArcNetHostImpl);
 };