diff --git a/packages/SettingsLib/src/com/android/settingslib/SignalIcon.java b/packages/SettingsLib/src/com/android/settingslib/SignalIcon.java index bc0c6f36e96ad..794b0eb665199 100644 --- a/packages/SettingsLib/src/com/android/settingslib/SignalIcon.java +++ b/packages/SettingsLib/src/com/android/settingslib/SignalIcon.java @@ -15,6 +15,8 @@ */ package com.android.settingslib; +import com.android.settingslib.mobile.TelephonyIcons; + import java.text.SimpleDateFormat; import java.util.Objects; @@ -40,9 +42,17 @@ public class SignalIcon { // For logging. public final String name; - public IconGroup(String name, int[][] sbIcons, int[][] qsIcons, int[] contentDesc, - int sbNullState, int qsNullState, int sbDiscState, int qsDiscState, - int discContentDesc) { + public IconGroup( + String name, + int[][] sbIcons, + int[][] qsIcons, + int[] contentDesc, + int sbNullState, + int qsNullState, + int sbDiscState, + int qsDiscState, + int discContentDesc + ) { this.name = name; this.sbIcons = sbIcons; this.qsIcons = qsIcons; @@ -131,6 +141,19 @@ public class SignalIcon { && other.activityOut == activityOut && other.rssi == rssi; } + + @Override + public int hashCode() { + return Objects.hash( + connected, + enabled, + level, + inetCondition, + iconGroup, + activityIn, + activityOut, + rssi); + } } /** @@ -139,18 +162,31 @@ public class SignalIcon { public static class MobileIconGroup extends IconGroup { public final int dataContentDescription; // mContentDescriptionDataType public final int dataType; - public final boolean isWide; - public final int qsDataType; - public MobileIconGroup(String name, int[][] sbIcons, int[][] qsIcons, int[] contentDesc, - int sbNullState, int qsNullState, int sbDiscState, int qsDiscState, - int discContentDesc, int dataContentDesc, int dataType, boolean isWide) { - super(name, sbIcons, qsIcons, contentDesc, sbNullState, qsNullState, sbDiscState, - qsDiscState, discContentDesc); + public MobileIconGroup( + String name, + int[][] sbIcons, + int[][] qsIcons, + int[] contentDesc, + int sbNullState, + int qsNullState, + int sbDiscState, + int qsDiscState, + int discContentDesc, + int dataContentDesc, + int dataType + ) { + super(name, + sbIcons, + qsIcons, + contentDesc, + sbNullState, + qsNullState, + sbDiscState, + qsDiscState, + discContentDesc); this.dataContentDescription = dataContentDesc; this.dataType = dataType; - this.isWide = isWide; - this.qsDataType = dataType; // TODO: remove this field } } @@ -187,6 +223,27 @@ public class SignalIcon { defaultDataOff = state.defaultDataOff; } + /** @return true if this state is disabled or not default data */ + public boolean isDataDisabledOrNotDefault() { + return (iconGroup == TelephonyIcons.DATA_DISABLED + || (iconGroup == TelephonyIcons.NOT_DEFAULT_DATA)) && userSetup; + } + + /** @return if this state is considered to have inbound activity */ + public boolean hasActivityIn() { + return dataConnected && !carrierNetworkChangeMode && activityIn; + } + + /** @return if this state is considered to have outbound activity */ + public boolean hasActivityOut() { + return dataConnected && !carrierNetworkChangeMode && activityOut; + } + + /** @return true if this state should show a RAT icon in quick settings */ + public boolean showQuickSettingsRatIcon() { + return dataConnected || isDataDisabledOrNotDefault(); + } + @Override protected void toString(StringBuilder builder) { super.toString(builder); @@ -202,23 +259,40 @@ public class SignalIcon { builder.append("carrierNetworkChangeMode=").append(carrierNetworkChangeMode) .append(','); builder.append("userSetup=").append(userSetup).append(','); - builder.append("defaultDataOff=").append(defaultDataOff); + builder.append("defaultDataOff=").append(defaultDataOff).append(','); + builder.append("showQuickSettingsRatIcon=").append(showQuickSettingsRatIcon()); } @Override public boolean equals(Object o) { return super.equals(o) - && Objects.equals(((MobileState) o).networkName, networkName) - && Objects.equals(((MobileState) o).networkNameData, networkNameData) - && ((MobileState) o).dataSim == dataSim - && ((MobileState) o).dataConnected == dataConnected - && ((MobileState) o).isEmergency == isEmergency - && ((MobileState) o).airplaneMode == airplaneMode - && ((MobileState) o).carrierNetworkChangeMode == carrierNetworkChangeMode - && ((MobileState) o).userSetup == userSetup - && ((MobileState) o).isDefault == isDefault - && ((MobileState) o).roaming == roaming - && ((MobileState) o).defaultDataOff == defaultDataOff; + && Objects.equals(((MobileState) o).networkName, networkName) + && Objects.equals(((MobileState) o).networkNameData, networkNameData) + && ((MobileState) o).dataSim == dataSim + && ((MobileState) o).dataConnected == dataConnected + && ((MobileState) o).isEmergency == isEmergency + && ((MobileState) o).airplaneMode == airplaneMode + && ((MobileState) o).carrierNetworkChangeMode == carrierNetworkChangeMode + && ((MobileState) o).userSetup == userSetup + && ((MobileState) o).isDefault == isDefault + && ((MobileState) o).roaming == roaming + && ((MobileState) o).defaultDataOff == defaultDataOff; + } + + @Override + public int hashCode() { + return Objects.hash(super.hashCode(), + networkName, + networkNameData, + dataSim, + dataConnected, + isEmergency, + airplaneMode, + carrierNetworkChangeMode, + userSetup, + isDefault, + roaming, + defaultDataOff); } } } diff --git a/packages/SettingsLib/src/com/android/settingslib/mobile/TelephonyIcons.java b/packages/SettingsLib/src/com/android/settingslib/mobile/TelephonyIcons.java index f8565bc2279f7..d4e58f7a2fc43 100644 --- a/packages/SettingsLib/src/com/android/settingslib/mobile/TelephonyIcons.java +++ b/packages/SettingsLib/src/com/android/settingslib/mobile/TelephonyIcons.java @@ -50,178 +50,194 @@ public class TelephonyIcons { null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.carrier_network_change_mode, - 0, - false); + 0 + ); public static final MobileIconGroup THREE_G = new MobileIconGroup( "3G", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_3g, - TelephonyIcons.ICON_3G, - true); + TelephonyIcons.ICON_3G + ); public static final MobileIconGroup WFC = new MobileIconGroup( "WFC", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], - 0, 0, false); + 0, + 0); public static final MobileIconGroup UNKNOWN = new MobileIconGroup( "Unknown", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], - 0, 0, false); + 0, + 0); public static final MobileIconGroup E = new MobileIconGroup( "E", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_edge, - TelephonyIcons.ICON_E, - false); + TelephonyIcons.ICON_E + ); public static final MobileIconGroup ONE_X = new MobileIconGroup( "1X", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_cdma, - TelephonyIcons.ICON_1X, - true); + TelephonyIcons.ICON_1X + ); public static final MobileIconGroup G = new MobileIconGroup( "G", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_gprs, - TelephonyIcons.ICON_G, - false); + TelephonyIcons.ICON_G + ); public static final MobileIconGroup H = new MobileIconGroup( "H", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_3_5g, - TelephonyIcons.ICON_H, - false); + TelephonyIcons.ICON_H + ); public static final MobileIconGroup H_PLUS = new MobileIconGroup( "H+", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_3_5g_plus, - TelephonyIcons.ICON_H_PLUS, - false); + TelephonyIcons.ICON_H_PLUS + ); public static final MobileIconGroup FOUR_G = new MobileIconGroup( "4G", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_4g, - TelephonyIcons.ICON_4G, - true); + TelephonyIcons.ICON_4G + ); public static final MobileIconGroup FOUR_G_PLUS = new MobileIconGroup( "4G+", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_4g_plus, - TelephonyIcons.ICON_4G_PLUS, - true); + TelephonyIcons.ICON_4G_PLUS + ); public static final MobileIconGroup LTE = new MobileIconGroup( "LTE", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_lte, - TelephonyIcons.ICON_LTE, - true); + TelephonyIcons.ICON_LTE + ); public static final MobileIconGroup LTE_PLUS = new MobileIconGroup( "LTE+", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_lte_plus, - TelephonyIcons.ICON_LTE_PLUS, - true); + TelephonyIcons.ICON_LTE_PLUS + ); public static final MobileIconGroup LTE_CA_5G_E = new MobileIconGroup( "5Ge", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_5ge_html, - TelephonyIcons.ICON_5G_E, - true); + TelephonyIcons.ICON_5G_E + ); public static final MobileIconGroup NR_5G = new MobileIconGroup( "5G", @@ -234,8 +250,8 @@ public class TelephonyIcons { 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_5g, - TelephonyIcons.ICON_5G, - true); + TelephonyIcons.ICON_5G + ); public static final MobileIconGroup NR_5G_PLUS = new MobileIconGroup( "5G_PLUS", @@ -248,34 +264,36 @@ public class TelephonyIcons { 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_5g_plus, - TelephonyIcons.ICON_5G_PLUS, - true); + TelephonyIcons.ICON_5G_PLUS + ); public static final MobileIconGroup DATA_DISABLED = new MobileIconGroup( "DataDisabled", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.cell_data_off_content_description, - 0, - false); + 0 + ); public static final MobileIconGroup NOT_DEFAULT_DATA = new MobileIconGroup( "NotDefaultData", null, null, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH, - 0, 0, + 0, + 0, 0, 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.not_default_data_content_description, - 0, - false); + 0 + ); public static final MobileIconGroup CARRIER_MERGED_WIFI = new MobileIconGroup( "CWF", @@ -288,8 +306,8 @@ public class TelephonyIcons { /* qsDiscState= */ 0, AccessibilityContentDescriptions.PHONE_SIGNAL_STRENGTH[0], R.string.data_connection_carrier_wifi, - TelephonyIcons.ICON_CWF, - /* isWide= */ true); + TelephonyIcons.ICON_CWF + ); // When adding a new MobileIconGround, check if the dataContentDescription has to be filtered // in QSCarrier#hasValidTypeContentDescription diff --git a/packages/SettingsLib/tests/robotests/src/com/android/settingslib/MobileStateTest.java b/packages/SettingsLib/tests/robotests/src/com/android/settingslib/MobileStateTest.java new file mode 100644 index 0000000000000..92a32bce17992 --- /dev/null +++ b/packages/SettingsLib/tests/robotests/src/com/android/settingslib/MobileStateTest.java @@ -0,0 +1,133 @@ +/* + * Copyright (C) 2021 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package com.android.settingslib; + +import static junit.framework.Assert.assertFalse; +import static junit.framework.Assert.assertTrue; + +import com.android.settingslib.mobile.TelephonyIcons; + +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +import org.robolectric.RobolectricTestRunner; + +@RunWith(RobolectricTestRunner.class) +public class MobileStateTest { + + private SignalIcon.MobileState mState = new SignalIcon.MobileState(); + + @Before + public void setUp() { + } + + @Test + public void testIsDataDisabledOrNotDefault_dataDisabled() { + mState.iconGroup = TelephonyIcons.DATA_DISABLED; + mState.userSetup = true; + + assertTrue(mState.isDataDisabledOrNotDefault()); + } + + @Test + public void testIsDataDisabledOrNotDefault_notDefaultData() { + mState.iconGroup = TelephonyIcons.NOT_DEFAULT_DATA; + mState.userSetup = true; + + assertTrue(mState.isDataDisabledOrNotDefault()); + } + + @Test + public void testIsDataDisabledOrNotDefault_notDisabled() { + mState.iconGroup = TelephonyIcons.G; + mState.userSetup = true; + + assertFalse(mState.isDataDisabledOrNotDefault()); + } + + @Test + public void testHasActivityIn_noData_noActivity() { + mState.dataConnected = false; + mState.carrierNetworkChangeMode = false; + mState.activityIn = false; + + assertFalse(mState.hasActivityIn()); + } + + @Test + public void testHasActivityIn_noData_activityIn() { + mState.dataConnected = false; + mState.carrierNetworkChangeMode = false; + mState.activityIn = true; + + assertFalse(mState.hasActivityIn()); + } + + @Test + public void testHasActivityIn_dataConnected_activityIn() { + mState.dataConnected = true; + mState.carrierNetworkChangeMode = false; + mState.activityIn = true; + + assertTrue(mState.hasActivityIn()); + } + + @Test + public void testHasActivityIn_carrierNetworkChange() { + mState.dataConnected = true; + mState.carrierNetworkChangeMode = true; + mState.activityIn = true; + + assertFalse(mState.hasActivityIn()); + } + + @Test + public void testHasActivityOut_noData_noActivity() { + mState.dataConnected = false; + mState.carrierNetworkChangeMode = false; + mState.activityOut = false; + + assertFalse(mState.hasActivityOut()); + } + + @Test + public void testHasActivityOut_noData_activityOut() { + mState.dataConnected = false; + mState.carrierNetworkChangeMode = false; + mState.activityOut = true; + + assertFalse(mState.hasActivityOut()); + } + + @Test + public void testHasActivityOut_dataConnected_activityOut() { + mState.dataConnected = true; + mState.carrierNetworkChangeMode = false; + mState.activityOut = true; + + assertTrue(mState.hasActivityOut()); + } + + @Test + public void testHasActivityOut_carrierNetworkChange() { + mState.dataConnected = true; + mState.carrierNetworkChangeMode = true; + mState.activityOut = true; + + assertFalse(mState.hasActivityOut()); + } +} diff --git a/packages/SystemUI/src/com/android/systemui/qs/tiles/CellularTile.java b/packages/SystemUI/src/com/android/systemui/qs/tiles/CellularTile.java index 8e886e82d352c..b1af8416e06d6 100644 --- a/packages/SystemUI/src/com/android/systemui/qs/tiles/CellularTile.java +++ b/packages/SystemUI/src/com/android/systemui/qs/tiles/CellularTile.java @@ -275,7 +275,7 @@ public class CellularTile extends QSTileImpl { return; } mInfo.dataSubscriptionName = mController.getMobileDataNetworkName(); - mInfo.dataContentDescription = indicators.description != null + mInfo.dataContentDescription = indicators.qsDescription != null ? indicators.typeContentDescriptionHtml : null; mInfo.activityIn = indicators.activityIn; mInfo.activityOut = indicators.activityOut; diff --git a/packages/SystemUI/src/com/android/systemui/qs/tiles/InternetTile.java b/packages/SystemUI/src/com/android/systemui/qs/tiles/InternetTile.java index cc9e7485dcffa..530804ed09e7e 100644 --- a/packages/SystemUI/src/com/android/systemui/qs/tiles/InternetTile.java +++ b/packages/SystemUI/src/com/android/systemui/qs/tiles/InternetTile.java @@ -279,9 +279,9 @@ public class InternetTile extends QSTileImpl { // Not data sim, don't display. return; } - mCellularInfo.mDataSubscriptionName = indicators.description == null - ? mController.getMobileDataNetworkName() : indicators.description; - mCellularInfo.mDataContentDescription = indicators.description != null + mCellularInfo.mDataSubscriptionName = indicators.qsDescription == null + ? mController.getMobileDataNetworkName() : indicators.qsDescription; + mCellularInfo.mDataContentDescription = indicators.qsDescription != null ? indicators.typeContentDescriptionHtml : null; mCellularInfo.mMobileSignalIconId = indicators.qsIcon.icon; mCellularInfo.mQsTypeIcon = indicators.qsType; diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/policy/MobileSignalController.java b/packages/SystemUI/src/com/android/systemui/statusbar/policy/MobileSignalController.java index 3490e1567ea81..a543c7c95d16f 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/policy/MobileSignalController.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/policy/MobileSignalController.java @@ -91,8 +91,7 @@ public class MobileSignalController extends SignalController mNetworkToIconLookup; + private Map mNetworkToIconLookup; // Since some pieces of the phone state are interdependent we store it locally, // this could potentially become part of MobileState for simplification/complication @@ -108,8 +107,6 @@ public class MobileSignalController extends SignalController, D .append(",qsIcon=").append(qsIcon == null ? "" : qsIcon.toString()) .append(",activityIn=").append(activityIn) .append(",activityOut=").append(activityOut) - .append(",description=").append(description) + .append(",qsDescription=").append(description) .append(",isTransient=").append(isTransient) .append(",statusLabel=").append(statusLabel) .append(']').toString(); @@ -100,8 +100,7 @@ public interface NetworkController extends CallbackController, D public boolean activityOut; public CharSequence typeContentDescription; public CharSequence typeContentDescriptionHtml; - public CharSequence description; - public boolean isWide; + public CharSequence qsDescription; public int subId; public boolean roaming; public boolean showTriangle; @@ -109,7 +108,7 @@ public interface NetworkController extends CallbackController, D public MobileDataIndicators(IconState statusIcon, IconState qsIcon, int statusType, int qsType, boolean activityIn, boolean activityOut, CharSequence typeContentDescription, CharSequence typeContentDescriptionHtml, - CharSequence description, boolean isWide, int subId, boolean roaming, + CharSequence qsDescription, int subId, boolean roaming, boolean showTriangle) { this.statusIcon = statusIcon; this.qsIcon = qsIcon; @@ -119,8 +118,7 @@ public interface NetworkController extends CallbackController, D this.activityOut = activityOut; this.typeContentDescription = typeContentDescription; this.typeContentDescriptionHtml = typeContentDescriptionHtml; - this.description = description; - this.isWide = isWide; + this.qsDescription = qsDescription; this.subId = subId; this.roaming = roaming; this.showTriangle = showTriangle; @@ -137,8 +135,7 @@ public interface NetworkController extends CallbackController, D .append(",activityOut=").append(activityOut) .append(",typeContentDescription=").append(typeContentDescription) .append(",typeContentDescriptionHtml=").append(typeContentDescriptionHtml) - .append(",description=").append(description) - .append(",isWide=").append(isWide) + .append(",description=").append(qsDescription) .append(",subId=").append(subId) .append(",roaming=").append(roaming) .append(",showTriangle=").append(showTriangle) @@ -186,11 +183,13 @@ public interface NetworkController extends CallbackController, D default void setCallIndicator(IconState statusIcon, int subId) {} } - public interface EmergencyListener { + /** */ + interface EmergencyListener { void setEmergencyCallsOnly(boolean emergencyOnly); } - public static class IconState { + /** */ + class IconState { public final boolean visible; public final int icon; public final String contentDescription; @@ -220,7 +219,7 @@ public interface NetworkController extends CallbackController, D * Tracks changes in access points. Allows listening for changes, scanning for new APs, * and connecting to new ones. */ - public interface AccessPointController { + interface AccessPointController { void addAccessPointCallback(AccessPointCallback callback); void removeAccessPointCallback(AccessPointCallback callback); void scanForAccessPoints(); @@ -230,7 +229,7 @@ public interface NetworkController extends CallbackController, D boolean canConfigWifi(); boolean canConfigMobileData(); - public interface AccessPointCallback { + interface AccessPointCallback { void onAccessPointsChanged(List accessPoints); void onSettingsActivityTriggered(Intent settingsIntent); } diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/policy/WifiSignalController.java b/packages/SystemUI/src/com/android/systemui/statusbar/policy/WifiSignalController.java index fc19564bf554f..3f7ddc6c15389 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/policy/WifiSignalController.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/policy/WifiSignalController.java @@ -163,7 +163,7 @@ public class WifiSignalController extends int qsTypeIcon = 0; IconState qsIcon = null; if (sbVisible) { - qsTypeIcon = icons.qsDataType; + qsTypeIcon = icons.dataType; qsIcon = new IconState(mCurrentState.connected, getQsCurrentIconIdForCarrierWifi(), contentDescription); } @@ -172,7 +172,7 @@ public class WifiSignalController extends MobileDataIndicators mobileDataIndicators = new MobileDataIndicators( statusIcon, qsIcon, typeIcon, qsTypeIcon, mCurrentState.activityIn, mCurrentState.activityOut, dataContentDescription, - dataContentDescriptionHtml, description, icons.isWide, + dataContentDescriptionHtml, description, mCurrentState.subId, /* roaming= */ false, /* showTriangle= */ true ); callback.setMobileDataIndicators(mobileDataIndicators); diff --git a/packages/SystemUI/tests/src/com/android/systemui/qs/carrier/QSCarrierGroupControllerTest.java b/packages/SystemUI/tests/src/com/android/systemui/qs/carrier/QSCarrierGroupControllerTest.java index 126b332af30db..a1b72104509bf 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/qs/carrier/QSCarrierGroupControllerTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/qs/carrier/QSCarrierGroupControllerTest.java @@ -232,7 +232,7 @@ public class QSCarrierGroupControllerTest extends LeakCheckedTest { MobileDataIndicators indicators = new MobileDataIndicators( mock(NetworkController.IconState.class), mock(NetworkController.IconState.class), - 0, 0, true, true, "", "", "", true, 0, true, true); + 0, 0, true, true, "", "", "", 0, true, true); mSignalCallback.setMobileDataIndicators(indicators); } diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/policy/CallbackHandlerTest.java b/packages/SystemUI/tests/src/com/android/systemui/statusbar/policy/CallbackHandlerTest.java index 24182434f1bae..3c55df934f76a 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/policy/CallbackHandlerTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/policy/CallbackHandlerTest.java @@ -124,7 +124,7 @@ public class CallbackHandlerTest extends SysuiTestCase { boolean roaming = true; MobileDataIndicators indicators = new MobileDataIndicators( status, qs, type, qsType, in, out, typeDescription, - typeDescriptionHtml, description, wide, subId, roaming, true); + typeDescriptionHtml, description, subId, roaming, true); mHandler.setMobileDataIndicators(indicators); waitForCallbacks(); @@ -141,8 +141,7 @@ public class CallbackHandlerTest extends SysuiTestCase { assertEquals(out, expected.activityOut); assertEquals(typeDescription, expected.typeContentDescription); assertEquals(typeDescriptionHtml, expected.typeContentDescriptionHtml); - assertEquals(description, expected.description); - assertEquals(wide, expected.isWide); + assertEquals(description, expected.qsDescription); assertEquals(subId, expected.subId); assertTrue(expected.roaming); assertTrue(expected.showTriangle);