diff --git a/src/com/android/settings/wifi/details/WifiDetailPreferenceController.java b/src/com/android/settings/wifi/details/WifiDetailPreferenceController.java index 15308e179e5..4ab0805d159 100644 --- a/src/com/android/settings/wifi/details/WifiDetailPreferenceController.java +++ b/src/com/android/settings/wifi/details/WifiDetailPreferenceController.java @@ -247,8 +247,7 @@ public class WifiDetailPreferenceController extends PreferenceController impleme mButtonsPref = (LayoutPreference) screen.findPreference(KEY_BUTTONS_PREF); mSignInButton = (Button) mButtonsPref.findViewById(R.id.signin_button); mSignInButton.setText(R.string.support_sign_in_button_text); - mSignInButton.setOnClickListener( - view -> mConnectivityManagerWrapper.startCaptivePortalApp(mNetwork)); + mSignInButton.setOnClickListener(view -> signIntoNetwork()); mSignalStrengthPref = (WifiDetailPreference) screen.findPreference(KEY_SIGNAL_STRENGTH_PREF); @@ -477,4 +476,13 @@ public class WifiDetailPreferenceController extends PreferenceController impleme mFragment.getActivity(), MetricsProto.MetricsEvent.ACTION_WIFI_FORGET); mFragment.getActivity().finish(); } + + /** + * Sign in to the captive portal found on this wifi network associated with this preference. + */ + private void signIntoNetwork() { + mMetricsFeatureProvider.action( + mFragment.getActivity(), MetricsProto.MetricsEvent.ACTION_WIFI_SIGNIN); + mConnectivityManagerWrapper.startCaptivePortalApp(mNetwork); + } } diff --git a/tests/robotests/src/com/android/settings/wifi/details/WifiDetailPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/wifi/details/WifiDetailPreferenceControllerTest.java index 43177f7898c..0839d8b7f77 100644 --- a/tests/robotests/src/com/android/settings/wifi/details/WifiDetailPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/wifi/details/WifiDetailPreferenceControllerTest.java @@ -715,6 +715,8 @@ public class WifiDetailPreferenceControllerTest { verify(mockSignInButton).setOnClickListener(captor.capture()); captor.getValue().onClick(mockSignInButton); verify(mockConnectivityManagerWrapper).startCaptivePortalApp(mockNetwork); + verify(mockMetricsFeatureProvider) + .action(mockActivity, MetricsProto.MetricsEvent.ACTION_WIFI_SIGNIN); } @Test