[i18n] use Polymer data binding to change locale strings

ui/webui/resources/js/i18n_behavior.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview
- * 'I18nBehavior' is a behavior to mix in loading of
- * internationalization strings.
- *
- * Example:
- *   behaviors: [
- *     I18nBehavior,
- *   ],
+ * 'I18nBehavior' is a behavior to mix in loading of internationalization
+ * strings.
  */
 
 /** @polymerBehavior */
 var I18nBehavior = {
+  properties: {
+    /**
+     * The language the UI is presented in. Used to signal dynamic language
+     * change.
+     */
+    locale: {
+      type: String,
+      value: '',
+    },
+  },
+
   /**
    * Returns a translated string where $1 to $9 are replaced by the given
    * values.
    * @param {string} id The ID of the string to translate.
    * @param {...string} var_args Values to replace the placeholders $1 to $9
    *     in the string.
    * @return {string} A translated, substituted string.
    * @private
    */
   i18nRaw_: function(id, var_args) {
@@ skipping 25 matching lines @@
    *          tags: (Array<string>|undefined)}} opts
    */
   i18nAdvanced: function(id, opts) {
     var args = [id].concat(opts.substitutions || []);
     var rawString = this.i18nRaw_.apply(this, args);
     return parseHtmlSubset('<b>' + rawString + '</b>', opts.tags, opts.attrs)
         .firstChild.innerHTML;
   },
 
   /**
+   * Similar to 'i18n', with an unused |locale| parameter used to trigger
+   * updates when |this.locale| changes.
+   * @param {string} locale The UI language used.
+   * @param {string} id The ID of the string to translate.
+   * @param {...string} var_args Values to replace the placeholders $1 to $9
+   *     in the string.
+   * @return {string} A translated, sanitized, substituted string.
+   */
+  i18nDynamic: function(language, id, var_args) {
+    return this.i18n.apply(this, Array.prototype.slice.call(arguments, 1));

[Dan Beam, 2017/05/22 23:39:02]

i think this slice here is fine

[dschuyler, 2017/05/23 00:03:19]

Acknowledged.

+  },
+
+  /**
    * Returns true if a translation exists for |id|.
    * @param {string} id
    * @return {boolean}
    */
   i18nExists: function(id) {
     return loadTimeData.valueExists(id);
   },
+
+  /**
+   * Call this when UI strings may have changed (e.g. because the locale
+   * changed). This will send an update to any data bindings to
+   * i18nDynamic(locale, ...).
+   */
+  updateLocalizedContent: function() {
+    this.locale = loadTimeData.getString('language');
+  },
 };
 
 /**
  * TODO(stevenjb): Replace with an interface. b/24294625
  * @typedef {{
  *   i18n: function(string, ...string): string}},
  *   i18nAdvanced: function({
  *     substitutions: (Array<string>|undefined),
  *     attrs: (Object<function(Node, string):boolean>|undefined),
  *     tags: (Array<string>|undefined)}, opts),
  *   i18nExists: function(string)
  * }}
  */
 I18nBehavior.Proto;