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

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 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 180b9b689c02ded3ea87a39f0d3f49b961836265..335893ec1b7617eff2b7ee678b9fa1c8f3563f8f 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,10 @@ 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));
+ if (MaterialDesignController::IsSystemTrayMenuMaterial())
+ network_list_view_.reset(new ui::NetworkListViewMd(this));
tdanderson 2016/09/20 14:44:33 perhaps include a comment somewhere that you've fo
varkha 2016/09/23 00:35:53 Done.
+ else
+ network_list_view_.reset(new ui::NetworkListView(this));
}
}
@@ -355,7 +359,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());
@@ -404,9 +409,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();
}
@@ -925,6 +928,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