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

Unified Diff: chrome/browser/chromeos/status/network_menu_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/status/network_menu.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/status/network_menu_button.cc
diff --git a/chrome/browser/chromeos/status/network_menu_button.cc b/chrome/browser/chromeos/status/network_menu_button.cc
index f4ff11ca03142fa193ab1508c6f1cac9abe839d2..c16383e199ca91aa3fee7b9dbe544fd672f3bc71 100644
--- a/chrome/browser/chromeos/status/network_menu_button.cc
+++ b/chrome/browser/chromeos/status/network_menu_button.cc
@@ -98,7 +98,7 @@ void NetworkMenuButton::NetworkChanged(NetworkLibrary* cros) {
SetIcon(*rb.GetBitmapNamed(IDR_STATUSBAR_NETWORK_BARS1));
}
std::string network_name = cros->wifi_connecting() ?
- cros->wifi_name() : cros->cellular_name();
+ cros->wifi_network().name() : cros->cellular_network().name();
SetTooltipText(
l10n_util::GetStringF(IDS_STATUSBAR_NETWORK_CONNECTING_TOOLTIP,
UTF8ToWide(network_name)));
@@ -114,15 +114,17 @@ void NetworkMenuButton::NetworkChanged(NetworkLibrary* cros) {
IDS_STATUSBAR_NETWORK_CONNECTED_TOOLTIP,
l10n_util::GetString(IDS_STATUSBAR_NETWORK_DEVICE_ETHERNET)));
} else if (cros->wifi_connected()) {
- SetIcon(IconForNetworkStrength(cros->wifi_strength(), false));
+ SetIcon(IconForNetworkStrength(
+ cros->wifi_network().strength(), false));
SetTooltipText(l10n_util::GetStringF(
IDS_STATUSBAR_NETWORK_CONNECTED_TOOLTIP,
- UTF8ToWide(cros->wifi_name())));
+ UTF8ToWide(cros->wifi_network().name())));
} else if (cros->cellular_connected()) {
- SetIcon(IconForNetworkStrength(cros->cellular_strength(), false));
+ SetIcon(IconForNetworkStrength(
+ cros->cellular_network().strength(), false));
SetTooltipText(l10n_util::GetStringF(
IDS_STATUSBAR_NETWORK_CONNECTED_TOOLTIP,
- UTF8ToWide(cros->cellular_name())));
+ UTF8ToWide(cros->cellular_network().name())));
} else {
SetIcon(*rb.GetBitmapNamed(IDR_STATUSBAR_NETWORK_BARS0));
SetTooltipText(l10n_util::GetString(
« no previous file with comments | « chrome/browser/chromeos/status/network_menu.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698