Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2736)

Unified Diff: ash/system/network/network_list.h

Issue 2843163003: Remove NetworkListDelegate (Closed)
Patch Set: Cleanup Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ash/system/network/network_list.h
diff --git a/ash/system/network/network_list.h b/ash/system/network/network_list.h
index 94152d74c51b7944a0b068c8fab53e5ca7253955..77a883754d4cc434bcc1fad2eae20b98fbd37d4b 100644
--- a/ash/system/network/network_list.h
+++ b/ash/system/network/network_list.h
@@ -26,9 +26,7 @@ class View;
}
namespace ash {
-
struct NetworkInfo;
-class NetworkListDelegate;
class TriView;
// A list of available networks of a given type. This class is used for all
@@ -38,7 +36,7 @@ class NetworkListView : public NetworkListViewBase,
public:
class SectionHeaderRowView;
- explicit NetworkListView(NetworkListDelegate* delegate);
+ explicit NetworkListView(tray::NetworkStateListDetailedView* detailed_view);
~NetworkListView() override;
// NetworkListViewBase:
@@ -72,6 +70,18 @@ class NetworkListView : public NetworkListViewBase,
// being used.
TriView* CreateConnectionWarning();
+ // Creates and returns a View with the information in |info|.
+ views::View* CreateViewForNetwork(const NetworkInfo& info);
+
+ // Updates |view| with the information in |info|. Note that |view| is
+ // guaranteed to be a View returned from |CreateViewForNetwork()|.
+ void UpdateViewForNetwork(views::View* view, const NetworkInfo& info);
+
+ // Creates the view of an extra icon appearing next to the network name
+ // indicating that the network is controlled by an extension. If no extension
+ // is registered for this network, returns |nullptr|.
+ views::View* CreateControlledByExtensionView(const NetworkInfo& info);
+
// Adds or updates child views representing the network connections when
// |is_wifi| is matching the attribute of a network connection starting at
// |child_index|. Returns a set of guids for the added network
@@ -112,7 +122,6 @@ class NetworkListView : public NetworkListViewBase,
bool NeedUpdateViewForNetwork(const NetworkInfo& info) const;
bool needs_relayout_;
- NetworkListDelegate* delegate_;
views::Label* no_wifi_networks_view_;
views::Label* no_cellular_networks_view_;

Powered by Google App Engine
This is Rietveld 408576698