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

Unified Diff: ash/common/system/chromeos/network/network_state_list_detailed_view.cc

Issue 2342793005: [ash-md] Adds Wi-Fi header row to system tray network detailed view (Closed)
Patch Set: [ash-md] Materializes system tray network detailed view (rebased) Created 4 years, 3 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/common/system/chromeos/network/network_state_list_detailed_view.cc
diff --git a/ash/common/system/chromeos/network/network_state_list_detailed_view.cc b/ash/common/system/chromeos/network/network_state_list_detailed_view.cc
index 207c5ec6f41689f5a270bed8b74fa6d723c26ef8..be0cc58d44fa78516a83d38d4e5e3f437d9c9699 100644
--- a/ash/common/system/chromeos/network/network_state_list_detailed_view.cc
+++ b/ash/common/system/chromeos/network/network_state_list_detailed_view.cc
@@ -50,6 +50,7 @@
#include "ui/chromeos/network/network_icon_animation.h"
#include "ui/chromeos/network/network_info.h"
#include "ui/chromeos/network/network_list.h"
+#include "ui/chromeos/network/network_list_md.h"
#include "ui/chromeos/network/network_list_view_base.h"
#include "ui/chromeos/resources/grit/ui_chromeos_resources.h"
#include "ui/compositor/layer.h"
@@ -324,7 +325,12 @@ NetworkStateListDetailedView::NetworkStateListDetailedView(
network_list_view_.reset(new VPNListView(this));
} else {
// Use a common class to list any other network types.
- network_list_view_.reset(new ui::NetworkListView(this));
+ // TODO(varkha): NetworkListViewMd is a temporary fork of NetworkListView.
tdanderson 2016/09/23 22:14:10 nit: please add "See crbug.com/614453"
varkha 2016/09/29 20:31:00 Done.
+ // NetworkListView will go away when Material Design becomes default.
+ if (MaterialDesignController::IsSystemTrayMenuMaterial())
+ network_list_view_.reset(new ui::NetworkListViewMd(this));
+ else
+ network_list_view_.reset(new ui::NetworkListView(this));
}
}
@@ -355,7 +361,8 @@ void NetworkStateListDetailedView::Init() {
scanning_throbber_ = nullptr;
CreateScrollableList();
- CreateNetworkExtra();
+ if (!MaterialDesignController::IsSystemTrayMenuMaterial())
+ CreateNetworkExtra();
CreateTitleRow(IDS_ASH_STATUS_TRAY_NETWORK);
network_list_view_->set_container(scroll_content());
@@ -401,9 +408,7 @@ void NetworkStateListDetailedView::HandleButtonPressed(views::Button* sender,
} else if (sender == other_mobile_) {
delegate->ShowOtherNetworkDialog(shill::kTypeCellular);
} else if (sender == other_wifi_) {
- WmShell::Get()->RecordUserMetricsAction(
- UMA_STATUS_AREA_NETWORK_JOIN_OTHER_CLICKED);
- delegate->ShowOtherNetworkDialog(shill::kTypeWifi);
+ OnOtherWifiClicked();
} else {
NOTREACHED();
}
@@ -929,6 +934,13 @@ void NetworkStateListDetailedView::OnNetworkEntryClicked(views::View* sender) {
HandleViewClicked(sender);
}
+void NetworkStateListDetailedView::OnOtherWifiClicked() {
+ WmShell::Get()->RecordUserMetricsAction(
+ UMA_STATUS_AREA_NETWORK_JOIN_OTHER_CLICKED);
+ SystemTrayDelegate* delegate = WmShell::Get()->system_tray_delegate();
+ delegate->ShowOtherNetworkDialog(shill::kTypeWifi);
+}
+
void NetworkStateListDetailedView::RelayoutScrollList() {
scroller()->Layout();
}

Powered by Google App Engine
This is Rietveld 408576698