Remove NetworkListDelegate

ash/system/network/network_list_view_base.cc

Index: ash/system/network/network_list_view_base.cc
diff --git a/ash/system/network/network_list_view_base.cc b/ash/system/network/network_list_view_base.cc
index a20383b303ea52e17ac3985bb5b1a09c90d97b77..c4b611a7c6a2397b47ddf017f9b9896ec95d1e45 100644
--- a/ash/system/network/network_list_view_base.cc
+++ b/ash/system/network/network_list_view_base.cc
@@ -4,9 +4,15 @@
 
 #include "ash/system/network/network_list_view_base.h"
 
+#include "base/logging.h"
+
 namespace ash {
 
-NetworkListViewBase::NetworkListViewBase() {}
+NetworkListViewBase::NetworkListViewBase(
+    tray::NetworkStateListDetailedView* detailed_view)
+    : detailed_view_(detailed_view) {
+  DCHECK(detailed_view_);

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

I am not sure if this DCHECK is necessary, since if someone did instantiate a
NLVB with a null |detailed_view| then the failure would be caught by the first
call to detailed_view_->*

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

Yes, but this DCHECK here makes the contract explicit that |detailed_view_|
should not be null. Also, it would be easier to debug if it is caught here
rather than some random place that calls into |detailed_view_| first.

+}
 
 NetworkListViewBase::~NetworkListViewBase() {}