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

Unified Diff: chrome/browser/chromeos/status/network_dropdown_button.cc

Issue 3744009: chromium-os:5494 Networks with identical names not handled properly. (Closed)
Patch Set: Rebase from trunk. Created 10 years, 2 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
« no previous file with comments | « chrome/browser/chromeos/network_list.cc ('k') | chrome/browser/chromeos/status/network_menu.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/status/network_dropdown_button.cc
diff --git a/chrome/browser/chromeos/status/network_dropdown_button.cc b/chrome/browser/chromeos/status/network_dropdown_button.cc
index 2f83d6e42c0d5c2a91a7973dee40ee4d0a7010ac..d5fd453c366c1eeff6896116dc2053ccae9793fd 100644
--- a/chrome/browser/chromeos/status/network_dropdown_button.cc
+++ b/chrome/browser/chromeos/status/network_dropdown_button.cc
@@ -85,12 +85,14 @@ void NetworkDropdownButton::NetworkChanged(NetworkLibrary* cros) {
SetText(l10n_util::GetString(IDS_STATUSBAR_NETWORK_DEVICE_ETHERNET));
} else if (cros->wifi_connected()) {
animation_connecting_.Stop();
- SetIcon(IconForNetworkStrength(cros->wifi_strength(), true));
- SetText(ASCIIToWide(cros->wifi_name()));
+ SetIcon(IconForNetworkStrength(
+ cros->wifi_network().strength(), true));
+ SetText(ASCIIToWide(cros->wifi_network().name()));
} else if (cros->cellular_connected()) {
animation_connecting_.Stop();
- SetIcon(IconForNetworkStrength(cros->cellular_strength(), false));
- SetText(ASCIIToWide(cros->cellular_name()));
+ SetIcon(IconForNetworkStrength(
+ cros->cellular_network().strength(), false));
+ SetText(ASCIIToWide(cros->cellular_network().name()));
} else if (cros->wifi_connecting() || cros->cellular_connecting()) {
if (!animation_connecting_.is_animating()) {
animation_connecting_.Reset();
@@ -99,9 +101,9 @@ void NetworkDropdownButton::NetworkChanged(NetworkLibrary* cros) {
}
if (cros->wifi_connecting())
- SetText(ASCIIToWide(cros->wifi_name()));
+ SetText(ASCIIToWide(cros->wifi_network().name()));
else if (cros->cellular_connecting())
- SetText(ASCIIToWide(cros->cellular_name()));
+ SetText(ASCIIToWide(cros->cellular_network().name()));
}
if (!cros->Connected() && !cros->Connecting()) {
« no previous file with comments | « chrome/browser/chromeos/network_list.cc ('k') | chrome/browser/chromeos/status/network_menu.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698