Implement OnGetNetworks for net.mojom

components/arc/arc_bridge_service.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_ARC_BRIDGE_SERVICE_H_
 #define COMPONENTS_ARC_ARC_BRIDGE_SERVICE_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 75 matching lines @@
     virtual void OnClipboardInstanceClosed() {}
 
     // Called whenever the ARC input interface state changes.
     virtual void OnInputInstanceReady() {}
     virtual void OnInputInstanceClosed() {}
 
     // Called whenever the ARC notification interface state changes.
     virtual void OnNotificationsInstanceReady() {}
     virtual void OnNotificationsInstanceClosed() {}
 
+    // Called whenever the ARC net interface state changes.

[Luis Héctor Chávez, 2016/01/12 17:32:32]

nit: Remove this header and the one below to make this a simple list.

[cernekee, 2016/01/12 20:14:47]

Not sure I follow.  You're saying to remove the comments for
OnNetInstanceReady() and OnPowerInstance*(), but leave the comments for
everything else in the list?

Or to delete all of these boilerplate comments from the class definition?

[Luis Héctor Chávez, 2016/01/12 20:42:28]

Wait, the diff I was seeing was weird. The code is fine as-is. Disregard this
comment.

+    virtual void OnNetInstanceReady() {}
+
     // Called whenever the ARC power interface state changes.
     virtual void OnPowerInstanceReady() {}
     virtual void OnPowerInstanceClosed() {}
 
     // Called whenever the ARC process interface state changes.
     virtual void OnProcessInstanceReady() {}
     virtual void OnProcessInstanceClosed() {}
 
     // Called whenever the ARC settings interface state changes.
     virtual void OnSettingsInstanceReady() {}
@@ skipping 33 matching lines @@
   void RemoveObserver(Observer* observer);
 
   // Gets the Mojo interface for all the instance services. This will return
   // nullptr if that particular service is not ready yet. Use an Observer if
   // you want to be notified when this is ready. This can only be called on the
   // thread that this class was created on.
   AppInstance* app_instance() { return app_ptr_.get(); }
   AuthInstance* auth_instance() { return auth_ptr_.get(); }
   ClipboardInstance* clipboard_instance() { return clipboard_ptr_.get(); }
   InputInstance* input_instance() { return input_ptr_.get(); }
+  NetInstance* net_instance() { return net_ptr_.get(); }
   NotificationsInstance* notifications_instance() {
     return notifications_ptr_.get();
   }
   PowerInstance* power_instance() { return power_ptr_.get(); }
   ProcessInstance* process_instance() { return process_ptr_.get(); }
   SettingsInstance* settings_instance() { return settings_ptr_.get(); }
 
   int32_t app_version() const { return app_ptr_.version(); }
   int32_t auth_version() const { return auth_ptr_.version(); }
   int32_t clipboard_version() const { return clipboard_ptr_.version(); }
   int32_t input_version() const { return input_ptr_.version(); }
   int32_t notifications_version() const { return notifications_ptr_.version(); }
   int32_t power_version() const { return power_ptr_.version(); }
   int32_t process_version() const { return process_ptr_.version(); }
   int32_t settings_version() const { return settings_ptr_.version(); }
 
   // ArcHost:
   void OnAppInstanceReady(AppInstancePtr app_ptr) override;
   void OnAuthInstanceReady(AuthInstancePtr auth_ptr) override;
   void OnClipboardInstanceReady(ClipboardInstancePtr clipboard_ptr) override;
   void OnInputInstanceReady(InputInstancePtr input_ptr) override;
+  void OnNetInstanceReady(NetInstancePtr net_ptr) override;
   void OnNotificationsInstanceReady(
       NotificationsInstancePtr notifications_ptr) override;
   void OnPowerInstanceReady(PowerInstancePtr power_ptr) override;
   void OnProcessInstanceReady(ProcessInstancePtr process_ptr) override;
   void OnSettingsInstanceReady(SettingsInstancePtr process_ptr) override;
 
   // Gets the current state of the bridge service.
   State state() const { return state_; }
 
   // Gets if ARC is available in this system.
@@ skipping 40 matching lines @@
   void OnNotificationsVersionReady(int32_t version);
   void OnPowerVersionReady(int32_t version);
   void OnProcessVersionReady(int32_t version);
   void OnSettingsVersionReady(int32_t version);
 
   // Mojo interfaces.
   AppInstancePtr app_ptr_;
   AuthInstancePtr auth_ptr_;
   ClipboardInstancePtr clipboard_ptr_;
   InputInstancePtr input_ptr_;
+  NetInstancePtr net_ptr_;
   NotificationsInstancePtr notifications_ptr_;
   PowerInstancePtr power_ptr_;
   ProcessInstancePtr process_ptr_;
   SettingsInstancePtr settings_ptr_;
 
   // Temporary Mojo interfaces.  After a Mojo interface pointer has been
   // received from the other endpoint, we still need to asynchronously query
   // its version.  While that is going on, we should still return nullptr on
   // the xxx_instance() functions.
   // To keep the xxx_instance() functions being trivial, store the instance
@@ skipping 19 matching lines @@
 
   // WeakPtrFactory to use callbacks.
   base::WeakPtrFactory<ArcBridgeService> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ArcBridgeService);
 };
 
 }  // namespace arc
 
 #endif  // COMPONENTS_ARC_ARC_BRIDGE_SERVICE_H_