Remove NetworkListDelegate

ash/system/network/network_list_view_base.h

Index: ash/system/network/network_list_view_base.h
diff --git a/ash/system/network/network_list_view_base.h b/ash/system/network/network_list_view_base.h
index c5eee110acd47054b8ee19ca83f0e319675298f9..cd37c9580ce1f22947e09f822e1f0172f3852e9b 100644
--- a/ash/system/network/network_list_view_base.h
+++ b/ash/system/network/network_list_view_base.h
@@ -14,12 +14,14 @@ class View;
 }
 
 namespace ash {
+namespace tray {
+class NetworkStateListDetailedView;
+}
 
 // Base class for a list of available networks (and, in the case of VPNs, the
 // list of available VPN providers).
 class NetworkListViewBase {
  public:
-  NetworkListViewBase();
   virtual ~NetworkListViewBase();
 
   void set_container(views::View* container) { container_ = container; }
@@ -33,9 +35,18 @@ class NetworkListViewBase {
   virtual bool IsNetworkEntry(views::View* view, std::string* guid) const = 0;
 
  protected:
-  views::View* container() { return container_; }
+  explicit NetworkListViewBase(
+      tray::NetworkStateListDetailedView* detailed_view);
+
+  tray::NetworkStateListDetailedView* detailed_view() const {
+    return detailed_view_;
+  }
+
+  views::View* container() const { return container_; }
 
  private:
+  tray::NetworkStateListDetailedView* const detailed_view_;

[tdanderson, 2017/04/28 16:36:20]

Can you clarify the reasoning for the consts here?

[mohsen, 2017/04/28 21:51:32]

This is something that is set once in the constructor and is not supposed to
change (the variable itself is const, not the thing it points to).

+
   // The container that holds the actual list entries.
   views::View* container_ = nullptr;