Revert "SystemUI: Allow Wi-Fi/cell tiles to co-exist with provider model" (8acab11a) · Commits · e / os / android_frameworks_base

packages/SystemUI/src/com/android/systemui/qs/tiles/InternetTile.java

+1 −4

Original line number	Diff line number	Diff line
		@@ -303,9 +303,6 @@ public class InternetTile extends QSTileImpl<QSTile.BooleanState> {
		if (DEBUG) {
		Log.d(TAG, "setWifiIndicators: " + indicators);
		}
		if (!indicators.isDefault) {
		return;
		}
		synchronized (mWifiInfo) {
		mWifiInfo.mEnabled = indicators.enabled;
		mWifiInfo.mSsid = indicators.description;
		@@ -331,7 +328,7 @@ public class InternetTile extends QSTileImpl<QSTile.BooleanState> {
		if (DEBUG) {
		Log.d(TAG, "setMobileDataIndicators: " + indicators);
		}
		if (indicators.qsIcon == null \|\| !indicators.isDefault) {
		if (indicators.qsIcon == null) {
		// Not data sim, don't display.
		return;
		}

packages/SystemUI/src/com/android/systemui/statusbar/connectivity/MobileSignalController.java

+6 −2

Original line number	Diff line number	Diff line
		@@ -289,8 +289,7 @@ public class MobileSignalController extends SignalController<MobileState, Mobile
		qsInfo.description,
		mSubscriptionInfo.getSubscriptionId(),
		mCurrentState.roaming,
		sbInfo.showTriangle,
		mCurrentState.isDefault);
		sbInfo.showTriangle);
		callback.setMobileDataIndicators(mobileDataIndicators);
		}

		@@ -300,6 +299,11 @@ public class MobileSignalController extends SignalController<MobileState, Mobile
		CharSequence qsDescription = null;

		if (mCurrentState.dataSim) {
		// only show QS icons if the state is also default
		if (!mCurrentState.isDefault) {
		return new QsInfo(qsTypeIcon, qsIcon, qsDescription);
		}

		if (mCurrentState.showQuickSettingsRatIcon() \|\| mConfig.alwaysShowDataRatIcon) {
		qsTypeIcon = dataTypeIcon;
		}

packages/SystemUI/src/com/android/systemui/statusbar/connectivity/SignalCallback.kt

+2 −6

Original line number	Diff line number	Diff line
		@@ -113,8 +113,7 @@ data class WifiIndicators(
		@JvmField val activityOut: Boolean,
		@JvmField val description: String?,
		@JvmField val isTransient: Boolean,
		@JvmField val statusLabel: String?,
		@JvmField val isDefault: Boolean
		@JvmField val statusLabel: String?
		) {
		override fun toString(): String {
		return StringBuilder("WifiIndicators[")
		@@ -126,7 +125,6 @@ data class WifiIndicators(
		.append(",qsDescription=").append(description)
		.append(",isTransient=").append(isTransient)
		.append(",statusLabel=").append(statusLabel)
		.append(",isDefault=").append(isDefault)
		.append(']').toString()
		}
		}
		@@ -144,8 +142,7 @@ data class MobileDataIndicators(
		@JvmField val qsDescription: CharSequence?,
		@JvmField val subId: Int,
		@JvmField val roaming: Boolean,
		@JvmField val showTriangle: Boolean,
		@JvmField val isDefault: Boolean
		@JvmField val showTriangle: Boolean
		) {
		override fun toString(): String {
		return java.lang.StringBuilder("MobileDataIndicators[")
		@@ -161,7 +158,6 @@ data class MobileDataIndicators(
		.append(",subId=").append(subId)
		.append(",roaming=").append(roaming)
		.append(",showTriangle=").append(showTriangle)
		.append(",isDefault=").append(isDefault)
		.append(']').toString()
		}
		}

packages/SystemUI/src/com/android/systemui/statusbar/connectivity/WifiSignalController.java

+9 −8

Original line number	Diff line number	Diff line
		@@ -112,16 +112,18 @@ public class WifiSignalController extends SignalController<WifiState, IconGroup>
		}
		IconState statusIcon = new IconState(
		wifiVisible, getCurrentIconId(), contentDescription);
		IconState qsIcon = new IconState(mCurrentState.connected,
		IconState qsIcon = null;
		if (mCurrentState.isDefault \|\| (!mNetworkController.isRadioOn()
		&& !mNetworkController.isEthernetDefault())) {
		qsIcon = new IconState(mCurrentState.connected,
		mWifiTracker.isCaptivePortal ? R.drawable.ic_qs_wifi_disconnected
		: getQsCurrentIconId(), contentDescription);
		boolean isDefault = mCurrentState.isDefault \|\| (!mNetworkController.isRadioOn()
		&& !mNetworkController.isEthernetDefault());
		}
		WifiIndicators wifiIndicators = new WifiIndicators(
		mCurrentState.enabled, statusIcon, qsIcon,
		ssidPresent && mCurrentState.activityIn,
		ssidPresent && mCurrentState.activityOut,
		wifiDesc, mCurrentState.isTransient, mCurrentState.statusLabel, isDefault
		wifiDesc, mCurrentState.isTransient, mCurrentState.statusLabel
		);
		callback.setWifiIndicators(wifiIndicators);
		}
		@@ -154,8 +156,7 @@ public class WifiSignalController extends SignalController<WifiState, IconGroup>
		statusIcon, qsIcon, typeIcon, qsTypeIcon,
		mCurrentState.activityIn, mCurrentState.activityOut, dataContentDescription,
		dataContentDescriptionHtml, description,
		mCurrentState.subId, /* roaming= / false, / showTriangle= */ true,
		/* isDefault= */ qsIcon != null
		mCurrentState.subId, /* roaming= / false, / showTriangle= */ true
		);
		callback.setMobileDataIndicators(mobileDataIndicators);
		}