Skip to content

Commit

Permalink
[unified-consent] Update settings strings
Browse files Browse the repository at this point in the history
Changes:
 - String update for footer in privacy section
 - Minor string tweak for Autocomplete setting title
 - Update text for "turn off sync" confirmation dialogue
 - String tweak for activity and interactions

Bug: 864755, 864752, 863965, 863934
Change-Id: Idf94963717f54832be123f4c575178c9d2399298
Reviewed-on: https://chromium-review.googlesource.com/1141868
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576136}
  • Loading branch information
Thomas Tangl authored and Commit Bot committed Jul 18, 2018
1 parent c89a636 commit 47e1781
Show file tree
Hide file tree
Showing 4 changed files with 40 additions and 15 deletions.
2 changes: 1 addition & 1 deletion chrome/app/generated_resources.grd
Original file line number Diff line number Diff line change
Expand Up @@ -6641,7 +6641,7 @@ the Bookmarks menu.">
You can customize what information Google collects in <ph name="BEGIN_LINK">&lt;a id="settingsLink" href="chrome://settings"&gt;</ph>Settings<ph name="END_LINK">&lt;/a&gt;<ex>&lt;/a&gt;</ex></ph> anytime.
</message>
<message name="IDS_SYNC_CONFIRMATION_DICE_SYNC_SETTINGS_DESCRIPTION" desc="Body of the personalize services section of the sync confirmation dialog in the tab modal signin flow for child accounts">
Google may use content on sites you visit and browsing activity and interactions to personalize Chrome and other Google services like Translate, Search, and ads. You can customize this in Settings.
Google may use content on sites you visit, plus browser activity and interactions, to personalize Chrome and other Google services like Translate, Search, and ads. You can customize this in Settings.
</message>
<message name="IDS_SYNC_CONFIRMATION_CONFIRM_BUTTON_LABEL" desc="Label of the confirmation button in the sync confirmation dialog of the tab modal signin flow">
Ok, got it
Expand Down
16 changes: 14 additions & 2 deletions chrome/app/settings_strings.grdp
Original file line number Diff line number Diff line change
Expand Up @@ -2400,7 +2400,7 @@
</message>
</if>
<message name="IDS_SETTINGS_SUGGEST_PREF_UNIFIED_CONSENT" desc="The documentation string of the 'Use Suggest' preference">
Autocomplete searches and web addresses
Autocomplete searches and URLs
</message>
<message name="IDS_SETTINGS_SUGGEST_PREF_DESC_UNIFIED_CONSENT" desc="The description of the 'Use Suggest' preference">
Sends searches from the address bar and search box and some cookies to your default search engine
Expand Down Expand Up @@ -2498,6 +2498,9 @@
<message name="IDS_SETTINGS_PRIVACY_MORE_SETTINGS" desc="The text with a link to point users to more sync and personalization settings">
For more settings that relate to privacy, security, and data collection, see <ph name="BEGIN_LINK">&lt;a href=&quot;chrome://settings/syncSetup&quot;&gt;<ex>&lt;a href=&quot;chrome://settings/syncSetup&quot;&gt;</ex></ph>Sync and personalization<ph name="END_LINK">&lt;/a&gt;<ex>&lt;/a&gt;</ex></ph>
</message>
<message name="IDS_SETTINGS_PRIVACY_MORE_SETTINGS_UNIFIED_CONSENT" desc="The text with a link to point users to Sync and Google services settings">
For more settings that relate to privacy, security, and data collection, see <ph name="BEGIN_LINK">&lt;a href=&quot;chrome://settings/syncSetup&quot;&gt;<ex>&lt;a href=&quot;chrome://settings/syncSetup&quot;&gt;</ex></ph>Sync and Google services<ph name="END_LINK">&lt;/a&gt;<ex>&lt;/a&gt;</ex></ph>
</message>

<!-- Reset Settings Page -->
<message name="IDS_SETTINGS_RESET" desc="Title for an item in the 'Reset and clean up' section of Chrome Settings. If the user clicks this option, browser settings will be returned to their default values, after a confirmation by the user." meaning="Chrome Cleanup feature. Try to use the same translation for 'Reset' in 'Reset and cleanup' string.">
Expand Down Expand Up @@ -3444,6 +3447,9 @@
<message name="IDS_SETTINGS_TURN_OFF_SYNC_AND_SIGN_OUT_DIALOG_TITLE" desc="The text to display on the title of the dialog to turn off sync and sign out.">
Turn off sync and sign out?
</message>
<message name="IDS_SETTINGS_TURN_OFF_SYNC_AND_SIGN_OUT_DIALOG_TITLE_UNIFIED_CONSENT" desc="The text to display on the title of the dialog to turn off sync and sign out when unified consent is enabled.">
Turn off sync and personalization?
</message>
<message name="IDS_SETTINGS_TURN_OFF_SYNC_DIALOG_MANAGED_CONFIRM" desc="The text to display on the confirm button of the dialog to turn off sync, when the profile is managed by admin.">
Clear and continue
</message>
Expand All @@ -3464,6 +3470,9 @@
<message name="IDS_SETTINGS_SYNC_DISCONNECT_AND_SIGN_OUT_EXPLANATION" desc="The text to display in the Sign out of Chrome dialog to stop syncing and sign-out for non-managed profiles.">
This will sign you out of your Google accounts. Changes to your bookmarks, history, passwords, and other settings will no longer be synced to your Google Account. However, your existing data will remain stored in your Google Account and can be managed on <ph name="BEGIN_LINK">&lt;a href="$1" target=&quot;_blank&quot;&gt;<ex>&lt;a href="$1" target=&quot;_blank&quot;&gt;</ex></ph>Google Dashboard<ph name="END_LINK">&lt;/a&gt;<ex>&lt;/a&gt;</ex></ph>.
</message>
<message name="IDS_SETTINGS_SYNC_DISCONNECT_AND_SIGN_OUT_EXPLANATION_UNIFIED_CONSENT" desc="The text to display in the Sign out of Chrome dialog to stop syncing and sign-out for non-managed profiles when unified consent is enabled.">
This will sign you out of your Google Accounts. Your bookmarks, history, passwords, and more will no longer be synced.
</message>
<message name="IDS_SETTINGS_SYNC_DISCONNECT_EXPAND_ACCESSIBILITY_LABEL" desc="Label for the button in the Sign out of Chrome dialog that toggles showing the profile stats. Only visible by screen reader software.">
Show profile stats
</message>
Expand Down Expand Up @@ -3556,7 +3565,7 @@
Activity and interactions
</message>
<message name="IDS_SETTINGS_USER_EVENTS_CHECKBOX_TEXT" desc="Description text for the checkbox which enables or disables recording user events.">
User content on sites you visit and browser activity and interactions for personalization
Uses content on sites you visit, plus browser activity and interactions, for personalization
</message>
<message name="IDS_SETTINGS_MANAGE_SYNCED_DATA_TITLE" desc="Title for the link to manage Chrome Sync data via Google Dashboard.">
Manage synced data on Google Dashboard
Expand All @@ -3579,6 +3588,9 @@
<message name="IDS_SETTINGS_PASSPHRASE_RESET_HINT" desc="Informs user how to change their encryption setting.">
To change this setting, <ph name="BEGIN_LINK">&lt;a href="$1" target=&quot;_blank&quot;&gt;<ex>&lt;a href="$1" target=&quot;_blank&quot;&gt;</ex></ph>reset sync<ph name="END_LINK">&lt;/a&gt;<ex>&lt;/a&gt;</ex></ph>.
</message>
<message name="IDS_SETTINGS_PASSPHRASE_RESET_HINT_UNIFIED_CONSENT" desc="Informs user how to change their encryption setting when unified consent is enabled.">
To turn this on, <ph name="BEGIN_LINK">&lt;a href="$1" target=&quot;_blank&quot;&gt;<ex>&lt;a href="$1" target=&quot;_blank&quot;&gt;</ex></ph>reset sync<ph name="END_LINK">&lt;/a&gt;<ex>&lt;/a&gt;</ex></ph> to remove your sync passphrase
</message>
<message name="IDS_SETTINGS_EMPTY_PASSPHRASE_ERROR" desc="Error message when the passphrase is empty.">
Empty passphrase is not allowed
</message>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1739,8 +1739,11 @@ void AddPeopleStrings(content::WebUIDataSource* html_source, Profile* profile) {
base::ASCIIToUTF16(sync_dashboard_url)));
html_source->AddString(
"passphraseResetHint",
l10n_util::GetStringFUTF8(IDS_SETTINGS_PASSPHRASE_RESET_HINT,
base::ASCIIToUTF16(sync_dashboard_url)));
l10n_util::GetStringFUTF8(
IsUnifiedConsentEnabled(profile)
? IDS_SETTINGS_PASSPHRASE_RESET_HINT_UNIFIED_CONSENT
: IDS_SETTINGS_PASSPHRASE_RESET_HINT,
base::ASCIIToUTF16(sync_dashboard_url)));
html_source->AddString(
"passphraseRecover",
l10n_util::GetStringFUTF8(IDS_SETTINGS_PASSPHRASE_RECOVER,
Expand All @@ -1762,7 +1765,9 @@ void AddPeopleStrings(content::WebUIDataSource* html_source, Profile* profile) {
LocalizedString sync_disconnect_strings[] = {
{"syncDisconnect", IDS_SETTINGS_PEOPLE_SYNC_TURN_OFF},
{"syncDisconnectTitle",
IDS_SETTINGS_TURN_OFF_SYNC_AND_SIGN_OUT_DIALOG_TITLE},
IsUnifiedConsentEnabled(profile)
? IDS_SETTINGS_TURN_OFF_SYNC_AND_SIGN_OUT_DIALOG_TITLE_UNIFIED_CONSENT
: IDS_SETTINGS_TURN_OFF_SYNC_AND_SIGN_OUT_DIALOG_TITLE},
{"syncDisconnectDeleteProfile",
IDS_SETTINGS_TURN_OFF_SYNC_DIALOG_CHECKBOX},
{"syncDisconnectConfirm",
Expand All @@ -1771,11 +1776,17 @@ void AddPeopleStrings(content::WebUIDataSource* html_source, Profile* profile) {
AddLocalizedStringsBulk(html_source, sync_disconnect_strings,
arraysize(sync_disconnect_strings));

html_source->AddString(
"syncDisconnectExplanation",
l10n_util::GetStringFUTF8(
IDS_SETTINGS_SYNC_DISCONNECT_AND_SIGN_OUT_EXPLANATION,
base::ASCIIToUTF16(sync_dashboard_url)));
if (IsUnifiedConsentEnabled(profile)) {
html_source->AddLocalizedString(
"syncDisconnectExplanation",
IDS_SETTINGS_SYNC_DISCONNECT_AND_SIGN_OUT_EXPLANATION_UNIFIED_CONSENT);
} else {
html_source->AddString(
"syncDisconnectExplanation",
l10n_util::GetStringFUTF8(
IDS_SETTINGS_SYNC_DISCONNECT_AND_SIGN_OUT_EXPLANATION,
base::ASCIIToUTF16(sync_dashboard_url)));
}
}
#endif

Expand Down Expand Up @@ -1924,7 +1935,6 @@ void AddPrivacyStrings(content::WebUIDataSource* html_source,
IDS_SETTINGS_SAFEBROWSING_ENABLEPROTECTION},
{"safeBrowsingEnableProtectionDesc",
IDS_SETTINGS_SAFEBROWSING_ENABLEPROTECTION_DESC},
{"syncAndPersonalizationLink", IDS_SETTINGS_PRIVACY_MORE_SETTINGS},
{"urlKeyedAnonymizedDataCollection",
IDS_SETTINGS_ENABLE_URL_KEYED_ANONYMIZED_DATA_COLLECTION},
{"urlKeyedAnonymizedDataCollectionDesc",
Expand All @@ -1950,6 +1960,8 @@ void AddPrivacyStrings(content::WebUIDataSource* html_source,
{"spellingPref", IDS_SETTINGS_SPELLING_PREF_UNIFIED_CONSENT},
{"spellingDescription",
IDS_SETTINGS_SPELLING_DESCRIPTION_UNIFIED_CONSENT},
{"syncAndPersonalizationLink",
IDS_SETTINGS_PRIVACY_MORE_SETTINGS_UNIFIED_CONSENT},
{"enableLogging", IDS_SETTINGS_ENABLE_LOGGING_UNIFIED_CONSENT},
{"enableLoggingDesc", IDS_SETTINGS_ENABLE_LOGGING_DESC_UNIFIED_CONSENT},
};
Expand All @@ -1966,6 +1978,7 @@ void AddPrivacyStrings(content::WebUIDataSource* html_source,
{"linkDoctorPrefDesc", IDS_SETTINGS_EMPTY_STRING},
{"spellingPref", IDS_SETTINGS_SPELLING_PREF},
{"spellingDescription", IDS_SETTINGS_SPELLING_DESCRIPTION},
{"syncAndPersonalizationLink", IDS_SETTINGS_PRIVACY_MORE_SETTINGS},
#if defined(OS_CHROMEOS)
{"enableLogging", IDS_SETTINGS_ENABLE_LOGGING_DIAGNOSTIC_AND_USAGE_DATA},
#else
Expand Down
6 changes: 3 additions & 3 deletions chrome/test/data/webui/signin/sync_confirmation_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,9 +50,9 @@ cr.define('signin_sync_confirmation', function() {
'Personalized Google services like Google Pay',
'Improve Chrome and its security by sending system and usage ' +
'information to Google',
'Google may use content on sites you visit and browsing activity and ' +
'interactions to personalize Chrome and other Google services like ' +
'Translate, Search, and ads. You can customize this in Settings.',
'Google may use content on sites you visit, plus browser activity and ' +
'interactions, to personalize Chrome and other Google services ' +
'like Translate, Search, and ads. You can customize this in Settings.'
];


Expand Down

0 comments on commit 47e1781

Please sign in to comment.