Commit aa597178 authored by Boris Sazonov's avatar Boris Sazonov Committed by Commit Bot

[Signin][Android] Fix access point assert in SigninFragment

SigninFragment is never used with START_PAGE access point (we have
SigninFirstRunFragment for that), so assert shouldn't allow thi value.

Bug: 657587
Change-Id: I238b8366d6042db8b53c4318f1e457799254759b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2089876
Auto-Submit: Boris Sazonov <bsazonov@chromium.org>
Commit-Queue: Alice Wang <aliceywang@chromium.org>
Reviewed-by: default avatarAlice Wang <aliceywang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748152}
parent 1e44c878
...@@ -104,9 +104,8 @@ public class SigninFragment extends SigninFragmentBase { ...@@ -104,9 +104,8 @@ public class SigninFragment extends SigninFragmentBase {
|| accessPoint == SigninAccessPoint.NTP_CONTENT_SUGGESTIONS || accessPoint == SigninAccessPoint.NTP_CONTENT_SUGGESTIONS
|| accessPoint == SigninAccessPoint.RECENT_TABS || accessPoint == SigninAccessPoint.RECENT_TABS
|| accessPoint == SigninAccessPoint.SETTINGS || accessPoint == SigninAccessPoint.SETTINGS
|| accessPoint == SigninAccessPoint.SIGNIN_PROMO
|| accessPoint || accessPoint
== SigninAccessPoint.START_PAGE : "invalid access point: " + accessPoint; == SigninAccessPoint.SIGNIN_PROMO : "invalid access point: " + accessPoint;
mSigninAccessPoint = accessPoint; mSigninAccessPoint = accessPoint;
mPromoAction = mPromoAction =
getSigninArguments().getInt(ARGUMENT_PERSONALIZED_PROMO_ACTION, PromoAction.NONE); getSigninArguments().getInt(ARGUMENT_PERSONALIZED_PROMO_ACTION, PromoAction.NONE);
......
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