Implement OnGetNetworks for net.mojom

components/arc/net/arc_net_host_impl.h

+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_ARC_NET_ARC_NET_HOST_IMPL_H_
+#define COMPONENTS_ARC_NET_ARC_NET_HOST_IMPL_H_
+
+#include <stdint.h>
+#include <string>
+#include <vector>
+
+#include "base/files/scoped_file.h"
+#include "base/macros.h"
+#include "base/threading/thread_checker.h"
+#include "components/arc/arc_bridge_service.h"
+#include "components/arc/common/net.mojom.h"
+#include "mojo/public/cpp/bindings/binding.h"
+
+namespace arc {
+
+class ArcBridgeService;
+
+// Private implementation of ArcNetHost.
+class ArcNetHostImpl : public NetHost, public ArcBridgeService::Observer {
+ public:
+  // The constructor will register an Observer with ArcBridgeService.
+  explicit ArcNetHostImpl(ArcBridgeService* arc_bridge_service);
+  ~ArcNetHostImpl() override;
+
+  // Called when a GetNetworks call is sent from ARC.
+  void GetNetworks(bool configured_only,
+                   bool visible_only,
+                   const GetNetworksCallback& callback) override;
+
+  // Overridden from ArcBridgeService::Observer:
+  void OnNetInstanceReady() override;
+
+ private:
+  // Owned by ArcServiceManager which makes sure ArcBridgeService is destroyed
+  // after ArcNetHost.
+  ArcBridgeService* arc_bridge_service_;

[dcheng, 2016/01/22 07:17:43]

Nit: ArcBridgeService* const

[lhc(google), 2016/01/22 15:56:19]

Actually, remove this completely since you need to inherit from ArcService,
which already provides arc_bridge_service() for this. See other services in ToT
for examples.

[cernekee, 2016/01/22 19:58:41]

Done.

+
+  base::ThreadChecker thread_checker_;
+  mojo::Binding<arc::NetHost> binding_;
+
+  // WeakPtrFactory to use for callbacks.
+  base::WeakPtrFactory<ArcNetHostImpl> weak_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(ArcNetHostImpl);
+};
+
+}  // namespace arc
+
+#endif  // COMPONENTS_ARC_NET_ARC_NET_HOST_IMPL_H_