Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 185c0ef7 authored by Daniel Sandler's avatar Daniel Sandler Committed by Android (Google) Code Review
Browse files

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

parents 7eeaf3f0 2e481b91
Loading
Loading
Loading
Loading
+47 −27
Original line number Diff line number Diff line
@@ -890,11 +890,19 @@ public class NetworkController extends BroadcastReceiver {

        if (!mHasMobileDataFeature) {
            mDataSignalIconId = mPhoneSignalIconId = 0;
        } else if (mDataConnected) {
            mobileLabel = mNetworkName;
            mobileLabel = "";
        } 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) {
                mobileLabel += "yyyyYYYYyyyyYYYY";
            }

            // Now for things that should only be shown when actually using mobile data.
            if (mDataConnected) {
                combinedSignalIconId = mDataSignalIconId;
                switch (mDataActivity) {
                    case TelephonyManager.DATA_ACTIVITY_IN:
@@ -915,9 +923,7 @@ public class NetworkController extends BroadcastReceiver {
                combinedActivityIconId = mMobileActivityIconId;
                combinedSignalIconId = mDataSignalIconId; // set by updateDataIcon()
                mContentDescriptionCombinedSignal = mContentDescriptionDataType;
        } else {
            mobileLabel = mHasMobileDataFeature ?
                context.getString(R.string.status_bar_settings_signal_meter_disconnected) : "";
            }
        }

        if (mWifiConnected) {
@@ -949,6 +955,12 @@ public class NetworkController extends BroadcastReceiver {
            combinedLabel = wifiLabel;
            combinedSignalIconId = mWifiIconId; // set by updateWifiIcons()
            mContentDescriptionCombinedSignal = mContentDescriptionWifi;
        } else {
            if (mHasMobileDataFeature) {
                wifiLabel = "";
            } else {
                wifiLabel = context.getString(R.string.status_bar_settings_signal_meter_disconnected);
            }
        }

        if (mBluetoothTethered) {
@@ -969,9 +981,17 @@ public class NetworkController extends BroadcastReceiver {
            mDataTypeIconId = 0;

            // combined values from connected wifi take precedence over airplane mode
            if (!mWifiConnected) {
            if (mWifiConnected) {
                // Suppress "No internet connection." from mobile if wifi connected.
                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;
                combinedSignalIconId = mDataSignalIconId;
            }