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

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

Issue 3923002: Revert 63135 - Update icons to show lowdata and very lowdata for 3G data.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: 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_button.h ('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
===================================================================
--- chrome/browser/chromeos/status/network_menu_button.cc (revision 63144)
+++ chrome/browser/chromeos/status/network_menu_button.cc (working copy)
@@ -120,16 +120,11 @@
IDS_STATUSBAR_NETWORK_CONNECTED_TOOLTIP,
UTF8ToWide(cros->wifi_network().name())));
} else if (cros->cellular_connected()) {
- const CellularNetwork& cellular = cros->cellular_network();
- if (cellular.data_left() == CellularNetwork::DATA_NONE) {
- // If no data, then we show 0 bars.
- SetIcon(*rb.GetBitmapNamed(IDR_STATUSBAR_NETWORK_BARS0));
- } else {
- SetIcon(IconForNetworkStrength(cellular));
- }
+ SetIcon(IconForNetworkStrength(
+ cros->cellular_network().strength(), false));
SetTooltipText(l10n_util::GetStringF(
IDS_STATUSBAR_NETWORK_CONNECTED_TOOLTIP,
- UTF8ToWide(cellular.name())));
+ UTF8ToWide(cros->cellular_network().name())));
} else {
SetIcon(*rb.GetBitmapNamed(IDR_STATUSBAR_NETWORK_BARS0));
SetTooltipText(l10n_util::GetString(
@@ -141,20 +136,9 @@
SetBadge(*rb.GetBitmapNamed(IDR_STATUSBAR_NETWORK_DISCONNECTED));
} else if (!cros->ethernet_connected() && !cros->wifi_connected() &&
(cros->cellular_connecting() || cros->cellular_connected())) {
- int id;
- switch (cros->cellular_network().data_left()) {
- case CellularNetwork::DATA_NONE:
- case CellularNetwork::DATA_VERY_LOW:
- id = IDR_STATUSBAR_NETWORK_3G_VLOWDATA;
- break;
- case CellularNetwork::DATA_LOW:
- id = IDR_STATUSBAR_NETWORK_3G_LOWDATA;
- break;
- case CellularNetwork::DATA_NORMAL:
- id = IDR_STATUSBAR_NETWORK_3G;
- break;
- }
- SetBadge(*rb.GetBitmapNamed(id));
+ // TODO(chocobo): Check cellular network 3g/edge.
+ SetBadge(*rb.GetBitmapNamed(IDR_STATUSBAR_NETWORK_3G));
+// SetBadge(*rb.GetBitmapNamed(IDR_STATUSBAR_NETWORK_EDGE));
} else {
SetBadge(SkBitmap());
}
@@ -169,11 +153,6 @@
UpdateMenu();
}
-void NetworkMenuButton::CellularDataPlanChanged(NetworkLibrary* cros) {
- // Call NetworkChanged which will update the icon.
- NetworkChanged(cros);
-}
-
void NetworkMenuButton::SetBadge(const SkBitmap& badge) {
badge_ = badge;
}
« no previous file with comments | « chrome/browser/chromeos/status/network_menu_button.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698