[Wi-Fi] Apply new mock for supporting WiFi detail page for OpenRoaming feature.

Issue: Openroaming detail page have new UI Mock.
Solution: Apply the mock design to rollback previous UI and add one Subcription preference.

Bug: 146669261
Test: manually push API and check.
Change-Id: Ie3243c9bcd371db1909834ef257c741454abec11
This commit is contained in:
govenliu
2020-02-24 15:31:34 +08:00
parent 34fdec2668
commit 7d8d8d3400
8 changed files with 24 additions and 69 deletions

View File

@@ -2029,8 +2029,10 @@
<string name="wifi_ip_settings">IP settings</string> <string name="wifi_ip_settings">IP settings</string>
<!-- Label for the spinner to show Wifi MAC randomization [CHAR LIMIT=25] --> <!-- Label for the spinner to show Wifi MAC randomization [CHAR LIMIT=25] -->
<string name="wifi_privacy_settings">Privacy</string> <string name="wifi_privacy_settings">Privacy</string>
<!-- Label for the subscription detail preference. [CHAR LIMIT=32] --> <!-- Label for the subscription preference. [CHAR LIMIT=32] -->
<string name="wifi_subscription_detail">Subscription details</string> <string name="wifi_subscription">Subscription</string>
<!-- Summary text for the subscription preference. [CHAR LIMIT=NONE] -->
<string name="wifi_subscription_summary">View or change subscription</string>
<!-- Summary for Wifi MAC randomization option when it is ephemeral network [CHAR LIMIT=50] --> <!-- Summary for Wifi MAC randomization option when it is ephemeral network [CHAR LIMIT=50] -->
<string name="wifi_privacy_settings_ephemeral_summary">Randomized MAC</string> <string name="wifi_privacy_settings_ephemeral_summary">Randomized MAC</string>
<!-- Title for the fragment to add a device into the wifi network [CHAR LIMIT=50] --> <!-- Title for the fragment to add a device into the wifi network [CHAR LIMIT=50] -->

View File

@@ -69,6 +69,12 @@
android:entries="@array/wifi_privacy_entries" android:entries="@array/wifi_privacy_entries"
android:entryValues="@array/wifi_privacy_values"/> android:entryValues="@array/wifi_privacy_values"/>
<Preference
android:key="subscription_detail"
android:title="@string/wifi_subscription"
android:summary="@string/wifi_subscription_summary"
settings:allowDividerAbove="true"/>
<SwitchPreference <SwitchPreference
android:key="auto_connect" android:key="auto_connect"
android:title="@string/wifi_auto_connect_title" android:title="@string/wifi_auto_connect_title"
@@ -139,9 +145,4 @@
settings:enableCopying="true"/> settings:enableCopying="true"/>
</PreferenceCategory> </PreferenceCategory>
<Preference
android:key="subscription_detail"
android:title="@string/wifi_subscription_detail"
settings:allowDividerAbove="true"/>
</PreferenceScreen> </PreferenceScreen>

View File

@@ -51,11 +51,8 @@ public class AddDevicePreferenceController2 extends BasePreferenceController {
@Override @Override
public int getAvailabilityStatus() { public int getAvailabilityStatus() {
if (!WifiDppUtils.isSupportConfiguratorQrCodeScanner(mContext, mWifiEntry) return WifiDppUtils.isSupportConfiguratorQrCodeScanner(mContext, mWifiEntry) ? AVAILABLE
|| mWifiEntry.canManageSubscription()) { : CONDITIONALLY_UNAVAILABLE;
return CONDITIONALLY_UNAVAILABLE;
}
return AVAILABLE;
} }
@Override @Override

View File

@@ -145,7 +145,6 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
static final String KEY_IPV6_CATEGORY = "ipv6_category"; static final String KEY_IPV6_CATEGORY = "ipv6_category";
@VisibleForTesting @VisibleForTesting
static final String KEY_IPV6_ADDRESSES_PREF = "ipv6_addresses"; static final String KEY_IPV6_ADDRESSES_PREF = "ipv6_addresses";
static final String KEY_IP_DETAILS_CATEGORY = "ip_details_category";
private final WifiEntry mWifiEntry; private final WifiEntry mWifiEntry;
private final ConnectivityManager mConnectivityManager; private final ConnectivityManager mConnectivityManager;
@@ -165,7 +164,6 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
private ActionButtonsPreference mButtonsPref; private ActionButtonsPreference mButtonsPref;
private EntityHeaderController mEntityHeaderController; private EntityHeaderController mEntityHeaderController;
private Preference mSignalStrengthPref; private Preference mSignalStrengthPref;
private PreferenceCategory mIpDetailsCategory;
private Preference mTxLinkSpeedPref; private Preference mTxLinkSpeedPref;
private Preference mRxLinkSpeedPref; private Preference mRxLinkSpeedPref;
private Preference mFrequencyPref; private Preference mFrequencyPref;
@@ -196,9 +194,7 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
mLinkProperties = lp; mLinkProperties = lp;
refreshEntityHeader(); refreshEntityHeader();
refreshButtons(); refreshButtons();
if (!mWifiEntry.canManageSubscription()) { refreshIpLayerInfo();
refreshIpLayerInfo();
}
} }
} }
@@ -238,9 +234,7 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
} }
mNetworkCapabilities = nc; mNetworkCapabilities = nc;
refreshButtons(); refreshButtons();
if (!mWifiEntry.canManageSubscription()) { refreshIpLayerInfo();
refreshIpLayerInfo();
}
} }
} }
@@ -335,7 +329,6 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
updateCaptivePortalButton(); updateCaptivePortalButton();
mSignalStrengthPref = screen.findPreference(KEY_SIGNAL_STRENGTH_PREF); mSignalStrengthPref = screen.findPreference(KEY_SIGNAL_STRENGTH_PREF);
mIpDetailsCategory = screen.findPreference(KEY_IP_DETAILS_CATEGORY);
mTxLinkSpeedPref = screen.findPreference(KEY_TX_LINK_SPEED); mTxLinkSpeedPref = screen.findPreference(KEY_TX_LINK_SPEED);
mRxLinkSpeedPref = screen.findPreference(KEY_RX_LINK_SPEED); mRxLinkSpeedPref = screen.findPreference(KEY_RX_LINK_SPEED);
mFrequencyPref = screen.findPreference(KEY_FREQUENCY_PREF); mFrequencyPref = screen.findPreference(KEY_FREQUENCY_PREF);
@@ -351,13 +344,6 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
mIpv6Category = screen.findPreference(KEY_IPV6_CATEGORY); mIpv6Category = screen.findPreference(KEY_IPV6_CATEGORY);
mIpv6AddressPref = screen.findPreference(KEY_IPV6_ADDRESSES_PREF); mIpv6AddressPref = screen.findPreference(KEY_IPV6_ADDRESSES_PREF);
if (mWifiEntry.canManageSubscription()) {
mIpDetailsCategory.setVisible(false);
mIpv6Category.setVisible(false);
mSignalStrengthPref.setVisible(false);
mFrequencyPref.setVisible(false);
mSecurityPref.setVisible(false);
}
mSecurityPref.setSummary(mWifiEntry.getSecurityString(false /* concise */)); mSecurityPref.setSummary(mWifiEntry.getSecurityString(false /* concise */));
} }
@@ -505,16 +491,10 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
// refresh header // refresh header
refreshEntityHeader(); refreshEntityHeader();
refreshEntityHeaderIcon();
// refresh Buttons // refresh Buttons
refreshButtons(); refreshButtons();
// When support manage subscription, there won't have any detail information, so don't
// need to update those detail UIs.
if (mWifiEntry.canManageSubscription()) {
return;
}
// Update Connection Header icon and Signal Strength Preference // Update Connection Header icon and Signal Strength Preference
refreshRssiViews(); refreshRssiViews();
// Frequency Pref // Frequency Pref
@@ -531,11 +511,7 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
refreshMacAddress(); refreshMacAddress();
} }
private void refreshEntityHeaderIcon() { private void refreshRssiViews() {
if (mEntityHeaderController == null) {
return;
}
int signalLevel = mWifiEntry.getLevel(); int signalLevel = mWifiEntry.getLevel();
// Disappears signal view if not in range. e.g. for saved networks. // Disappears signal view if not in range. e.g. for saved networks.
@@ -550,23 +526,13 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
} }
mRssiSignalLevel = signalLevel; mRssiSignalLevel = signalLevel;
Drawable wifiIcon = mIconInjector.getIcon(mRssiSignalLevel); Drawable wifiIcon = mIconInjector.getIcon(mRssiSignalLevel);
mEntityHeaderController
.setIcon(redrawIconForHeader(wifiIcon)).done(mFragment.getActivity(),
true /* rebind */);
}
private void refreshRssiViews() { if (mEntityHeaderController != null) {
int signalLevel = mWifiEntry.getLevel(); mEntityHeaderController
.setIcon(redrawIconForHeader(wifiIcon)).done(mFragment.getActivity(),
// Disappears signal view if not in range. e.g. for saved networks. true /* rebind */);
if (signalLevel == WifiEntry.WIFI_LEVEL_UNREACHABLE) {
mSignalStrengthPref.setVisible(false);
mRssiSignalLevel = -1;
return;
} }
mRssiSignalLevel = signalLevel;
Drawable wifiIcon = mIconInjector.getIcon(mRssiSignalLevel);
Drawable wifiIconDark = wifiIcon.getConstantState().newDrawable().mutate(); Drawable wifiIconDark = wifiIcon.getConstantState().newDrawable().mutate();
wifiIconDark.setTintList(Utils.getColorAttr(mContext, android.R.attr.colorControlNormal)); wifiIconDark.setTintList(Utils.getColorAttr(mContext, android.R.attr.colorControlNormal));
mSignalStrengthPref.setIcon(wifiIconDark); mSignalStrengthPref.setIcon(wifiIconDark);

View File

@@ -55,7 +55,7 @@ public class WifiMeteredPreferenceController2 extends BasePreferenceController i
@Override @Override
public int getAvailabilityStatus() { public int getAvailabilityStatus() {
return mWifiEntry.canManageSubscription() ? CONDITIONALLY_UNAVAILABLE : AVAILABLE; return AVAILABLE;
} }
@Override @Override

View File

@@ -115,10 +115,6 @@ public class WifiNetworkDetailsFragment2 extends DashboardFragment implements
@Override @Override
public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) { public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
if (mNetworkDetailsTracker.getWifiEntry().canManageSubscription()) {
return;
}
MenuItem item = menu.add(0, Menu.FIRST, 0, R.string.wifi_modify); MenuItem item = menu.add(0, Menu.FIRST, 0, R.string.wifi_modify);
item.setIcon(com.android.internal.R.drawable.ic_mode_edit); item.setIcon(com.android.internal.R.drawable.ic_mode_edit);
item.setShowAsAction(MenuItem.SHOW_AS_ACTION_ALWAYS); item.setShowAsAction(MenuItem.SHOW_AS_ACTION_ALWAYS);

View File

@@ -55,11 +55,8 @@ public class WifiPrivacyPreferenceController2 extends BasePreferenceController i
@Override @Override
public int getAvailabilityStatus() { public int getAvailabilityStatus() {
if (!mWifiManager.isConnectedMacRandomizationSupported() return mWifiManager.isConnectedMacRandomizationSupported()
|| mWifiEntry.canManageSubscription()) { ? AVAILABLE : CONDITIONALLY_UNAVAILABLE;
return CONDITIONALLY_UNAVAILABLE;
}
return AVAILABLE;
} }
@Override @Override

View File

@@ -41,11 +41,7 @@ public class WifiSubscriptionDetailPreferenceController2 extends BasePreferenceC
@Override @Override
public int getAvailabilityStatus() { public int getAvailabilityStatus() {
if (mWifiEntry.canManageSubscription()) { return mWifiEntry.canManageSubscription() ? AVAILABLE : CONDITIONALLY_UNAVAILABLE;
return AVAILABLE;
}
return CONDITIONALLY_UNAVAILABLE;
} }
@Override @Override