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

Unified Diff: src/js/i18n.js

Issue 2412533002: [i18n] Avoid name conflicts caused by naive minifier (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/js/i18n.js
diff --git a/src/js/i18n.js b/src/js/i18n.js
index cb3476e3f698d7a8aefd63fbb0fbda75e81deac8..072676b2965fa7a1e0f5e3369db9673905040eef 100644
--- a/src/js/i18n.js
+++ b/src/js/i18n.js
@@ -76,9 +76,9 @@ function AddBoundMethod(obj, methodName, implementation, length, type) {
if (IS_UNDEFINED(this[internalName])) {
var boundMethod;
if (IS_UNDEFINED(length) || length === 2) {
- boundMethod = ANONYMOUS_FUNCTION((x, y) => implementation(this, x, y));
+ boundMethod = ANONYMOUS_FUNCTION((fst, snd) => implementation(this, fst, snd));
Yang 2016/10/12 04:56:55 can we keep the 80-char limit?
jgruber 2016/10/12 06:55:08 Done.
} else if (length === 1) {
- boundMethod = ANONYMOUS_FUNCTION(x => implementation(this, x));
+ boundMethod = ANONYMOUS_FUNCTION(fst => implementation(this, fst));
} else {
boundMethod = ANONYMOUS_FUNCTION((...args) => {
// DateTimeFormat.format needs to be 0 arg method, but can still
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698