Reland: chromeos: Convert ash VPNDelegate interface to mojo

ash/common/system/chromeos/network/vpn_list.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 ASH_COMMON_SYSTEM_CHROMEOS_NETWORK_VPN_DELEGATE_H_
-#define ASH_COMMON_SYSTEM_CHROMEOS_NETWORK_VPN_DELEGATE_H_
+#ifndef ASH_COMMON_SYSTEM_CHROMEOS_NETWORK_VPN_LIST_H_
+#define ASH_COMMON_SYSTEM_CHROMEOS_NETWORK_VPN_LIST_H_
 
 #include <string>
 #include <vector>
 
 #include "ash/ash_export.h"
+#include "ash/public/interfaces/vpn_list.mojom.h"
 #include "base/macros.h"
 #include "base/observer_list.h"
+#include "mojo/public/cpp/bindings/binding_set.h"
 
 namespace chromeos {
 class NetworkState;
 }
 
 namespace ash {
 
 // Describes a VPN provider.
 struct ASH_EXPORT VPNProvider {
   struct Key {
@@ skipping 24 matching lines @@
   Key key;
 
   // Human-readable name.
   std::string name;
 };
 
 // This delegate provides UI code in ash, e.g. |VPNListView|, with access to the
 // list of VPN providers enabled in the primary user's profile. The delegate
 // furthermore allows the UI code to request that a VPN provider show its "add
 // network" dialog.
-// TODO(jamescook): Rename this to VpnList as part of mojo conversion because it
-// won't be a delegate anymore.
-class ASH_EXPORT VPNDelegate {
+class ASH_EXPORT VpnList : public mojom::VpnList {
  public:
   // An observer that is notified whenever the list of VPN providers enabled in
   // the primary user's profile changes.
   class Observer {
    public:
     virtual void OnVPNProvidersChanged() = 0;
 
    protected:
     virtual ~Observer();
 
    private:
     DISALLOW_ASSIGN(Observer);
   };
 
-  VPNDelegate();
-  virtual ~VPNDelegate();
+  VpnList();
+  ~VpnList() override;
 
   const std::vector<VPNProvider>& vpn_providers() { return vpn_providers_; }
 
   // Returns |true| if at least one third-party VPN provider is enabled in the
   // primary user's profile, in addition to the built-in OpenVPN/L2TP provider.
   bool HaveThirdPartyVPNProviders() const;
 
   // Requests that the VPN provider identified by |key| show its "add network"
   // dialog.
-  virtual void ShowAddPage(const VPNProvider::Key& key) = 0;
+  void ShowAddPage(const VPNProvider::Key& key);
 
   void AddObserver(Observer* observer);
   void RemoveObserver(Observer* observer);
 
-  // Sets the list of extension-backed VPN providers. The list may be empty.
-  // Notifies observers. Public for testing.
-  // TODO(jamescook): Convert to mojo interface.
-  void SetThirdPartyVpnProviders(const std::vector<VPNProvider>& providers);
+  // Binds the mojom::VpnList interface to this object.
+  void BindRequest(mojom::VpnListRequest request);
+
+  // mojom::VpnList:
+  void SetClient(mojom::VpnListClientPtr client) override;
+  void SetThirdPartyVpnProviders(
+      std::vector<mojom::ThirdPartyVpnProviderPtr> providers) override;
 
  private:
   // Notify observers that the list of VPN providers enabled in the primary
   // user's profile has changed.
   void NotifyObservers();
 
   // Adds the built-in OpenVPN/L2TP provider to |vpn_providers_|.
   void AddBuiltInProvider();
 
+  // May be null in tests or when Chrome is not running.
+  mojom::VpnListClientPtr vpn_list_client_;
+
+  // Bindings for the mojom::VpnList interface.
+  mojo::BindingSet<mojom::VpnList> bindings_;
+
   // Cache of VPN providers, including the built-in OpenVPN/L2TP provider and
   // other providers added by extensions in the primary user's profile.
   std::vector<VPNProvider> vpn_providers_;
 
   base::ObserverList<Observer> observer_list_;
 
-  DISALLOW_COPY_AND_ASSIGN(VPNDelegate);
+  DISALLOW_COPY_AND_ASSIGN(VpnList);
 };
 
 }  // namespace ash
 
-#endif  // ASH_COMMON_SYSTEM_CHROMEOS_NETWORK_VPN_DELEGATE_H_
+#endif  // ASH_COMMON_SYSTEM_CHROMEOS_NETWORK_VPN_LIST_H_