Merge "Stop showing "No internet connection" when there is one." into ics-mr1

This commit is contained in:
Daniel Sandler
2012-01-24 12:54:48 -08:00
committed by Android (Google) Code Review

View File

@ -890,34 +890,40 @@ public class NetworkController extends BroadcastReceiver {
if (!mHasMobileDataFeature) { if (!mHasMobileDataFeature) {
mDataSignalIconId = mPhoneSignalIconId = 0; mDataSignalIconId = mPhoneSignalIconId = 0;
} else if (mDataConnected) { mobileLabel = "";
mobileLabel = mNetworkName; } else {
// We want to show the carrier name even if data is not being routed over that link, so
// we look only at the service state here.
mobileLabel = hasService()
? mNetworkName
: context.getString(R.string.status_bar_settings_signal_meter_disconnected);
if (DEBUG) { if (DEBUG) {
mobileLabel += "yyyyYYYYyyyyYYYY"; mobileLabel += "yyyyYYYYyyyyYYYY";
} }
combinedSignalIconId = mDataSignalIconId;
switch (mDataActivity) {
case TelephonyManager.DATA_ACTIVITY_IN:
mMobileActivityIconId = R.drawable.stat_sys_signal_in;
break;
case TelephonyManager.DATA_ACTIVITY_OUT:
mMobileActivityIconId = R.drawable.stat_sys_signal_out;
break;
case TelephonyManager.DATA_ACTIVITY_INOUT:
mMobileActivityIconId = R.drawable.stat_sys_signal_inout;
break;
default:
mMobileActivityIconId = 0;
break;
}
combinedLabel = mobileLabel; // Now for things that should only be shown when actually using mobile data.
combinedActivityIconId = mMobileActivityIconId; if (mDataConnected) {
combinedSignalIconId = mDataSignalIconId; // set by updateDataIcon() combinedSignalIconId = mDataSignalIconId;
mContentDescriptionCombinedSignal = mContentDescriptionDataType; switch (mDataActivity) {
} else { case TelephonyManager.DATA_ACTIVITY_IN:
mobileLabel = mHasMobileDataFeature ? mMobileActivityIconId = R.drawable.stat_sys_signal_in;
context.getString(R.string.status_bar_settings_signal_meter_disconnected) : ""; break;
case TelephonyManager.DATA_ACTIVITY_OUT:
mMobileActivityIconId = R.drawable.stat_sys_signal_out;
break;
case TelephonyManager.DATA_ACTIVITY_INOUT:
mMobileActivityIconId = R.drawable.stat_sys_signal_inout;
break;
default:
mMobileActivityIconId = 0;
break;
}
combinedLabel = mobileLabel;
combinedActivityIconId = mMobileActivityIconId;
combinedSignalIconId = mDataSignalIconId; // set by updateDataIcon()
mContentDescriptionCombinedSignal = mContentDescriptionDataType;
}
} }
if (mWifiConnected) { if (mWifiConnected) {
@ -949,6 +955,12 @@ public class NetworkController extends BroadcastReceiver {
combinedLabel = wifiLabel; combinedLabel = wifiLabel;
combinedSignalIconId = mWifiIconId; // set by updateWifiIcons() combinedSignalIconId = mWifiIconId; // set by updateWifiIcons()
mContentDescriptionCombinedSignal = mContentDescriptionWifi; mContentDescriptionCombinedSignal = mContentDescriptionWifi;
} else {
if (mHasMobileDataFeature) {
wifiLabel = "";
} else {
wifiLabel = context.getString(R.string.status_bar_settings_signal_meter_disconnected);
}
} }
if (mBluetoothTethered) { if (mBluetoothTethered) {
@ -969,9 +981,17 @@ public class NetworkController extends BroadcastReceiver {
mDataTypeIconId = 0; mDataTypeIconId = 0;
// combined values from connected wifi take precedence over airplane mode // combined values from connected wifi take precedence over airplane mode
if (!mWifiConnected) { if (mWifiConnected) {
wifiLabel = context.getString(R.string.status_bar_settings_signal_meter_disconnected); // Suppress "No internet connection." from mobile if wifi connected.
combinedLabel = wifiLabel; mobileLabel = "";
} else {
if (mHasMobileDataFeature) {
// let the mobile icon show "No internet connection."
wifiLabel = "";
} else {
wifiLabel = context.getString(R.string.status_bar_settings_signal_meter_disconnected);
combinedLabel = wifiLabel;
}
mContentDescriptionCombinedSignal = mContentDescriptionPhoneSignal; mContentDescriptionCombinedSignal = mContentDescriptionPhoneSignal;
combinedSignalIconId = mDataSignalIconId; combinedSignalIconId = mDataSignalIconId;
} }