Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(575)

Unified Diff: chrome/android/java/strings/android_chrome_strings.grd

Issue 2106823002: Disable the autoplay setting entry when Data Saver is ON. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@autoplay-string
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/strings/android_chrome_strings.grd
diff --git a/chrome/android/java/strings/android_chrome_strings.grd b/chrome/android/java/strings/android_chrome_strings.grd
index 266cc5a543ee277da8a482591cffb406839e428f..6c80e4093389648033b3360d437ec41113cbd40d 100644
--- a/chrome/android/java/strings/android_chrome_strings.grd
+++ b/chrome/android/java/strings/android_chrome_strings.grd
@@ -669,6 +669,9 @@ Your Google account may have other forms of browsing history like searches and a
<message name="IDS_WEBSITE_SETTINGS_CATEGORY_AUTOPLAY_ALLOWED" desc="Summary text explaining that sites are allowed to automatically play videos and that it is the recommended setting.">
Allow sites to automatically play videos (recommended)
</message>
+ <message name="IDS_WEBSITE_SETTINGS_CATEGORY_AUTOPLAY_DISABLED_DATA_SAVER" desc="Summary text explaning that autoplay is disabled because Data Saver is ON.">
+ Not allowed because Data Saver is ON
Miguel Garcia 2016/06/28 16:04:05 I assume you will get UX approval on this one too
mlamouri (slow - plz ping) 2016/06/28 16:09:51 Yes, will make sure they are happy with the string
+ </message>
<message name="IDS_WEBSITE_SETTINGS_CATEGORY_JAVASCRIPT_ALLOWED" desc="Summary text explaining that sites are allowed to run Javascript and that it is the recommended setting.">
Allow sites to run JavaScript (recommended)
</message>

Powered by Google App Engine
This is Rietveld 408576698