Commit 14a80f73 authored by Nohemi Fernandez's avatar Nohemi Fernandez Committed by Commit Bot

[iOS] Add flag to switch between sign-in notification infobar titles.

This patch gives us the ability to switch between infobar titles, either
with "Sign in as [fullname]" or "Sign in to Chrome", depending on final
UX decisions.

Bug: 1145592
Change-Id: Ifba41e4aa025ec51d80e76ec79015ace375be103
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526061
Commit-Queue: Nohemi Fernandez <fernandex@chromium.org>
Reviewed-by: default avatarJérôme Lebel <jlebel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#825317}
parent edc474d2
...@@ -4677,6 +4677,11 @@ ...@@ -4677,6 +4677,11 @@
// wallet-service-use-sandbox on Android. // wallet-service-use-sandbox on Android.
"expiry_milestone": -1 "expiry_milestone": -1
}, },
{
"name": "use-username-for-signin-notification-infobar-title",
"owners": [ "fernandex", "chrome-signin-team" ],
"expiry_milestone": 96
},
{ {
"name": "use-wallpaper-staging-url", "name": "use-wallpaper-staging-url",
"owners": [ "cros-customization@google.com", "hsuregan", "khorimoto" ], "owners": [ "cros-customization@google.com", "hsuregan", "khorimoto" ],
......
...@@ -19,4 +19,8 @@ const base::Feature kRestoreGaiaCookiesIfDeleted{ ...@@ -19,4 +19,8 @@ const base::Feature kRestoreGaiaCookiesIfDeleted{
const char kDelayThresholdMinutesToUpdateGaiaCookie[] = const char kDelayThresholdMinutesToUpdateGaiaCookie[] =
"minutes-delay-to-restore-gaia-cookies-if-deleted"; "minutes-delay-to-restore-gaia-cookies-if-deleted";
const base::Feature kSigninNotificationInfobarUsernameInTitle{
"SigninNotificationInfobarUsernameInTitle",
base::FEATURE_DISABLED_BY_DEFAULT};
} // namespace signin } // namespace signin
...@@ -22,6 +22,9 @@ extern const base::Feature kRestoreGaiaCookiesIfDeleted; ...@@ -22,6 +22,9 @@ extern const base::Feature kRestoreGaiaCookiesIfDeleted;
// for the existence of Gaia cookies for google.com. // for the existence of Gaia cookies for google.com.
extern const char kDelayThresholdMinutesToUpdateGaiaCookie[]; extern const char kDelayThresholdMinutesToUpdateGaiaCookie[];
// Feature controlling whether to use full username in sign-in notifications.
extern const base::Feature kSigninNotificationInfobarUsernameInTitle;
} // namespace signin } // namespace signin
#endif // COMPONENTS_SIGNIN_IOS_BROWSER_FEATURES_H_ #endif // COMPONENTS_SIGNIN_IOS_BROWSER_FEATURES_H_
...@@ -604,6 +604,11 @@ const flags_ui::FeatureEntry kFeatureEntries[] = { ...@@ -604,6 +604,11 @@ const flags_ui::FeatureEntry kFeatureEntries[] = {
flag_descriptions::kRestoreGaiaCookiesIfDeletedDescription, flag_descriptions::kRestoreGaiaCookiesIfDeletedDescription,
flags_ui::kOsIos, flags_ui::kOsIos,
FEATURE_VALUE_TYPE(signin::kRestoreGaiaCookiesIfDeleted)}, FEATURE_VALUE_TYPE(signin::kRestoreGaiaCookiesIfDeleted)},
{"use-username-for-signin-notification-infobar-title",
flag_descriptions::kSigninNotificationInfobarUsernameInTitleName,
flag_descriptions::kSigninNotificationInfobarUsernameInTitleDescription,
flags_ui::kOsIos,
FEATURE_VALUE_TYPE(signin::kSigninNotificationInfobarUsernameInTitle)},
{"minutes-delay-to-restore-gaia-cookies-if-deleted", {"minutes-delay-to-restore-gaia-cookies-if-deleted",
flag_descriptions::kDelayThresholdMinutesToUpdateGaiaCookieName, flag_descriptions::kDelayThresholdMinutesToUpdateGaiaCookieName,
flag_descriptions::kDelayThresholdMinutesToUpdateGaiaCookieDescription, flag_descriptions::kDelayThresholdMinutesToUpdateGaiaCookieDescription,
......
...@@ -271,6 +271,12 @@ const char kInfobarUIRebootOnlyiOS13Name[] = "Infobar UI Reboot iOS13"; ...@@ -271,6 +271,12 @@ const char kInfobarUIRebootOnlyiOS13Name[] = "Infobar UI Reboot iOS13";
const char kInfobarUIRebootOnlyiOS13Description[] = const char kInfobarUIRebootOnlyiOS13Description[] =
"When enabled, Infobar will use the new UI only on iOS13"; "When enabled, Infobar will use the new UI only on iOS13";
const char kSigninNotificationInfobarUsernameInTitleName[] =
"Sign-in notification infobar title";
const char kSigninNotificationInfobarUsernameInTitleDescription[] =
"When enabled, uses the authenticated user's full name in the infobar "
"title.";
const char kInProductHelpDemoModeName[] = "In-Product Help Demo Mode"; const char kInProductHelpDemoModeName[] = "In-Product Help Demo Mode";
const char kInProductHelpDemoModeDescription[] = const char kInProductHelpDemoModeDescription[] =
"When enabled, in-product help promotions occur exactly once per cold " "When enabled, in-product help promotions occur exactly once per cold "
......
...@@ -86,6 +86,11 @@ extern const char kAutofillUseRendererIDsDescription[]; ...@@ -86,6 +86,11 @@ extern const char kAutofillUseRendererIDsDescription[];
extern const char kLogBreadcrumbsName[]; extern const char kLogBreadcrumbsName[];
extern const char kLogBreadcrumbsDescription[]; extern const char kLogBreadcrumbsDescription[];
// Title and description for the flag that controls the sign-in notification
// infobar title.
extern const char kSigninNotificationInfobarUsernameInTitleName[];
extern const char kSigninNotificationInfobarUsernameInTitleDescription[];
// Title and description for the flag that controls synthetic crash reports // Title and description for the flag that controls synthetic crash reports
// generation for Unexplained Termination Events. // generation for Unexplained Termination Events.
extern const char kSyntheticCrashReportsForUteName[]; extern const char kSyntheticCrashReportsForUteName[];
......
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