Commit 4db0a23f authored by Tanmoy Mollik's avatar Tanmoy Mollik Committed by Commit Bot

[Android] Rename buttons in Personalized Promo view

This cl renames the SigninButton and ChooseAccountButton to Primary and
Secondary button respectively.

Bug: 1095628
Change-Id: I6c502c8f6509a4e45395204f986ccb554fad1f91
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2273238
Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org>
Reviewed-by: default avatarAlice Wang <aliceywang@chromium.org>
Reviewed-by: default avatarBoris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#795543}
parent 7b362a27
...@@ -23,8 +23,8 @@ public class PersonalizedSigninPromoView extends LinearLayout { ...@@ -23,8 +23,8 @@ public class PersonalizedSigninPromoView extends LinearLayout {
private ImageButton mDismissButton; private ImageButton mDismissButton;
private TextView mStatus; private TextView mStatus;
private TextView mDescription; private TextView mDescription;
private ButtonCompat mSigninButton; private ButtonCompat mPrimaryButton;
private Button mChooseAccountButton; private Button mSecondaryButton;
public PersonalizedSigninPromoView(Context context, AttributeSet attrs) { public PersonalizedSigninPromoView(Context context, AttributeSet attrs) {
super(context, attrs); super(context, attrs);
...@@ -38,8 +38,8 @@ public class PersonalizedSigninPromoView extends LinearLayout { ...@@ -38,8 +38,8 @@ public class PersonalizedSigninPromoView extends LinearLayout {
mDismissButton = findViewById(R.id.signin_promo_close_button); mDismissButton = findViewById(R.id.signin_promo_close_button);
mStatus = findViewById(R.id.signin_promo_status_message); mStatus = findViewById(R.id.signin_promo_status_message);
mDescription = findViewById(R.id.signin_promo_description); mDescription = findViewById(R.id.signin_promo_description);
mSigninButton = findViewById(R.id.signin_promo_signin_button); mPrimaryButton = findViewById(R.id.signin_promo_signin_button);
mChooseAccountButton = findViewById(R.id.signin_promo_choose_account_button); mSecondaryButton = findViewById(R.id.signin_promo_choose_account_button);
} }
/** /**
...@@ -73,14 +73,14 @@ public class PersonalizedSigninPromoView extends LinearLayout { ...@@ -73,14 +73,14 @@ public class PersonalizedSigninPromoView extends LinearLayout {
/** /**
* @return A reference to the signin button. * @return A reference to the signin button.
*/ */
public ButtonCompat getSigninButton() { public ButtonCompat getPrimaryButton() {
return mSigninButton; return mPrimaryButton;
} }
/** /**
* @return A reference to the choose account button. * @return A reference to the choose account button.
*/ */
public Button getChooseAccountButton() { public Button getSecondaryButton() {
return mChooseAccountButton; return mSecondaryButton;
} }
} }
...@@ -267,10 +267,10 @@ public class SigninPromoController { ...@@ -267,10 +267,10 @@ public class SigninPromoController {
view.getDescription().setText(mDescriptionStringIdNoAccount); view.getDescription().setText(mDescriptionStringIdNoAccount);
view.getSigninButton().setText(R.string.sign_in_to_chrome); view.getPrimaryButton().setText(R.string.sign_in_to_chrome);
view.getSigninButton().setOnClickListener(v -> signinWithNewAccount(context)); view.getPrimaryButton().setOnClickListener(v -> signinWithNewAccount(context));
view.getChooseAccountButton().setVisibility(View.GONE); view.getSecondaryButton().setVisibility(View.GONE);
} }
private void setupHotState(final Context context, PersonalizedSigninPromoView view) { private void setupHotState(final Context context, PersonalizedSigninPromoView view) {
...@@ -282,12 +282,12 @@ public class SigninPromoController { ...@@ -282,12 +282,12 @@ public class SigninPromoController {
String signinButtonText = context.getString( String signinButtonText = context.getString(
R.string.signin_promo_continue_as, mProfileData.getGivenNameOrFullNameOrEmail()); R.string.signin_promo_continue_as, mProfileData.getGivenNameOrFullNameOrEmail());
view.getSigninButton().setText(signinButtonText); view.getPrimaryButton().setText(signinButtonText);
view.getSigninButton().setOnClickListener(v -> signinWithDefaultAccount(context)); view.getPrimaryButton().setOnClickListener(v -> signinWithDefaultAccount(context));
view.getChooseAccountButton().setText(R.string.signin_promo_choose_another_account); view.getSecondaryButton().setText(R.string.signin_promo_choose_another_account);
view.getChooseAccountButton().setOnClickListener(v -> signinWithNotDefaultAccount(context)); view.getSecondaryButton().setOnClickListener(v -> signinWithNotDefaultAccount(context));
view.getChooseAccountButton().setVisibility(View.VISIBLE); view.getSecondaryButton().setVisibility(View.VISIBLE);
} }
private int getNumImpressions() { private int getNumImpressions() {
......
...@@ -132,8 +132,8 @@ public class SigninPromoUtil { ...@@ -132,8 +132,8 @@ public class SigninPromoUtil {
SigninPromoController.OnDismissListener listener) { SigninPromoController.OnDismissListener listener) {
setupSigninPromoViewFromCache(signinPromoController, profileDataCache, view, listener); setupSigninPromoViewFromCache(signinPromoController, profileDataCache, view, listener);
view.getStatusMessage().setVisibility(View.VISIBLE); view.getStatusMessage().setVisibility(View.VISIBLE);
view.getSigninButton().setText(R.string.sync_promo_turn_on_sync); view.getPrimaryButton().setText(R.string.sync_promo_turn_on_sync);
view.getChooseAccountButton().setVisibility(View.GONE); view.getSecondaryButton().setVisibility(View.GONE);
} }
/** /**
......
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