Commit 4f48bf58 authored by Alice Wang's avatar Alice Wang Committed by Chromium LUCI CQ

[Signin][Android] Adjust AccountManagementFragment after removing sign-out pref

This CL adjusts the if branch in AccountManagementFragment after
removing sign-out allowed pref value.

Bug: 1162231
Change-Id: I463913e92c45de052c0272a478282fb3c2ad1932
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2624647Reviewed-by: default avatarBoris Sazonov <bsazonov@chromium.org>
Commit-Queue: Alice Wang <aliceywang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#842923}
parent b0f448b5
...@@ -204,9 +204,9 @@ public class AccountManagementFragment extends PreferenceFragmentCompat ...@@ -204,9 +204,9 @@ public class AccountManagementFragment extends PreferenceFragmentCompat
} }
signOutPreference.setTitle(getSignOutPreferenceText()); signOutPreference.setTitle(getSignOutPreferenceText());
signOutPreference.setOnPreferenceClickListener(preference -> { signOutPreference.setOnPreferenceClickListener(preference -> {
if (!isVisible() || !isResumed()) return false; if (!isVisible() || !isResumed() || mSignedInAccountName == null) {
return false;
if (mSignedInAccountName != null) { }
SigninMetricsUtils.logProfileAccountManagementMenu( SigninMetricsUtils.logProfileAccountManagementMenu(
ProfileAccountManagementMetrics.TOGGLE_SIGNOUT, mGaiaServiceType); ProfileAccountManagementMetrics.TOGGLE_SIGNOUT, mGaiaServiceType);
...@@ -225,9 +225,6 @@ public class AccountManagementFragment extends PreferenceFragmentCompat ...@@ -225,9 +225,6 @@ public class AccountManagementFragment extends PreferenceFragmentCompat
.signOut(SignoutReason.USER_CLICKED_SIGNOUT_SETTINGS, null, false); .signOut(SignoutReason.USER_CLICKED_SIGNOUT_SETTINGS, null, false);
} }
return true; return true;
}
return false;
}); });
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment