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

Unified Diff: chrome/browser/resources/print_preview/settings/dpi_settings.js

Issue 2002783002: Use plural formats and native digits (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comment changes: l10n_util Created 4 years, 7 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/browser/resources/print_preview/settings/dpi_settings.js
diff --git a/chrome/browser/resources/print_preview/settings/dpi_settings.js b/chrome/browser/resources/print_preview/settings/dpi_settings.js
index a06dc9e9df4e838708dc7965998b6d3e1c079581..8adef40e7189ef65fa27f57608c516366aca415e 100644
--- a/chrome/browser/resources/print_preview/settings/dpi_settings.js
+++ b/chrome/browser/resources/print_preview/settings/dpi_settings.js
@@ -24,8 +24,11 @@ cr.define('print_preview', function() {
var hDpi = option.horizontal_dpi || 0;
var vDpi = option.vertical_dpi || 0;
if (hDpi > 0 && vDpi > 0 && hDpi != vDpi)
Dan Beam 2016/06/01 21:33:24 needs { curlies } now
jungshik at Google 2016/06/01 23:20:22 Done.
- return loadTimeData.getStringF('nonIsotropicDpiItemLabel', hDpi, vDpi);
- return loadTimeData.getStringF('dpiItemLabel', hDpi || vDpi);
+ return loadTimeData.getStringF('nonIsotropicDpiItemLabel',
+ hDpi.toLocaleString(),
+ vDpi.toLocaleString());
+ return loadTimeData.getStringF('dpiItemLabel',
+ (hDpi || vDpi).toLocaleString());
}
};

Powered by Google App Engine
This is Rietveld 408576698