MD Settings: Replace language detail page with dropdown menu items

chrome/browser/resources/settings/languages_page/languages_page.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 'settings-languages-page' is the settings page
  * for language and input method settings.
  */
 (function() {
 'use strict';
@@ skipping 107 matching lines @@
 
   /**
    * @param {!Object} change Polymer change object for languages.enabled.*.
    * @return {boolean} True if there are less than 2 languages.
    */
   isHelpTextHidden_: function(change) {
     return this.languages.enabled.length <= 1;
   },
 
   /**
+   * @param {!LanguageState} languageState
+   * @param {string} prospectiveUILanguage The chosen UI language.
+   * @return {boolean} True if the given language cannot be set/unset as the
+   *     prospective UI language by the user.
+   * @private
+   */
+  disableUILanguageCheckbox_: function(languageState, prospectiveUILanguage) {
+    // UI language setting belongs to the primary user.
+    if (this.isSecondaryUser_())
+      return true;
+
+    // UI language setting belongs to the primary user.
+    if (this.isSecondaryUser_())
+      return true;

[stevenjb, 2016/09/20 16:17:59]

Dupe

[michaelpg, 2016/09/21 19:28:13]

Done.

+
+    // If the language cannot be a UI language, we can't set/unset it as the
+    // prospective UI language.
+    if (!languageState.language.supportsUI)
+      return true;
+
+    // If the language already is the prospective UI language, it can't be unset
+    // if it is also the *actual* UI language, as we wouldn't know what other
+    // language to set as the prospective UI language.
+    if (languageState.language.code == navigator.language &&
+        (!prospectiveUILanguage ||
+         languageState.language.code == prospectiveUILanguage)) {
+      return true;
+    }
+
+    // Otherwise, the prospective language can changed to/from this language.

[stevenjb, 2016/09/20 16:17:59]

can be changed

[michaelpg, 2016/09/21 19:28:13]

Done.

+    return false;
+  },
+
+  /**
+   * @return {boolean} True for a secondary user in a multi-profile session.
+   * @private
+   */
+  isSecondaryUser_: function() {
+    return cr.isChromeOS && loadTimeData.getBoolean('isSecondaryUser');
+  },
+
+  /**
+   * Handler for changes to the UI language checkbox.
+   * @param {!{target: !PaperCheckboxElement}} e
+   * @private
+   */
+  onUILanguageChange_: function(e) {
+    if (e.target.checked) {
+      this.languageHelper.setUILanguage(this.detailLanguage_.language.code);
+    } else if (this.detailLanguage_.language.code ==
+        this.languageHelper.getProspectiveUILanguage()) {
+      // Reset the chosen UI language to the actual UI language.
+      this.languageHelper.resetUILanguage();
+    }
+  },
+
+   /**
+   * @param {!chrome.languageSettingsPrivate.Language} language
+   * @param {string} targetLanguageCode The default translate target language.
+   * @return {boolean} True if the translate checkbox should be disabled.
+   * @private
+   */
+  disableTranslateCheckbox_: function(language, targetLanguageCode) {
+    if (!language.supportsTranslate)
+      return true;
+
+    return this.languageHelper.convertLanguageCodeForTranslate(language.code) ==
+        targetLanguageCode;
+  },
+
+  /**
+   * Handler for changes to the translate checkbox.
+   * @param {!{target: !PaperCheckboxElement}} e
+   * @private
+   */
+  onTranslateCheckboxChange_: function(e) {
+    if (e.target.checked) {
+      this.languageHelper.enableTranslateLanguage(
+          this.detailLanguage_.language.code);
+    } else {
+      this.languageHelper.disableTranslateLanguage(
+          this.detailLanguage_.language.code);
+    }
+  },
+
+  /**
    * Moves the language up in the list.
    * @private
    */
   onMoveUpTap_: function() {
     this.menu_.closeMenu();
     this.languageHelper.moveLanguage(this.detailLanguage_.language.code, -1);
   },
 
   /**
    * Moves the language down in the list.
    * @private
    */
   onMoveDownTap_: function() {
     this.menu_.closeMenu();
     this.languageHelper.moveLanguage(this.detailLanguage_.language.code, 1);
   },
 
   /**
    * Disables the language.
    * @private
    */
   onRemoveLanguageTap_: function() {
     this.menu_.closeMenu();
     this.languageHelper.disableLanguage(this.detailLanguage_.language.code);
   },
 
   /**
-   * Opens the Language Detail page for the language.
-   * @private
-   */
-  onShowLanguageDetailTap_: function() {
-    this.menu_.closeMenu();
-    settings.navigateTo(settings.Route.LANGUAGES_DETAIL);
-  },
-
-  /**
    * Opens the Manage Input Methods page.
    * @private
    */
   onManageInputMethodsTap_: function() {
     assert(cr.isChromeOS);
     settings.navigateTo(settings.Route.INPUT_METHODS);
   },
 
   /**
    * Handler for clicking an input method on the main page, which sets it as
@@ skipping 66 matching lines @@
    * @private
    */
   onEditDictionaryTap_: function() {
     assert(!cr.isMac);
     settings.navigateTo(settings.Route.EDIT_DICTIONARY);
     this.forceRenderList_('settings-edit-dictionary-page');
   },
 
   /**
    * Checks whether the prospective UI language (the pref that indicates what
-   * language to use in Chrome) matches the current language. This pref is only
-   * on Chrome OS and Windows; we don't control the UI language elsewhere.
+   * language to use in Chrome) matches the current language. This pref is used
+   * only on Chrome OS and Windows; we don't control the UI language elsewhere.
    * @param {string} languageCode The language code identifying a language.
    * @param {string} prospectiveUILanguage The prospective UI language.
    * @return {boolean} True if the given language matches the prospective UI
    *     pref (which may be different from the actual UI language).
    * @private
    */
   isProspectiveUILanguage_: function(languageCode, prospectiveUILanguage) {
     assert(cr.isChromeOS || cr.isWindows);
     return languageCode == this.languageHelper.getProspectiveUILanguage();
   },
@@ skipping 13 matching lines @@
    * selected on Chrome OS and Windows.
    * @param {string} languageCode The language code identifying a language.
    * @param {string} prospectiveUILanguage The prospective UI language.
    * @param {boolean} supportsUI Whether Chrome's UI can be shown in this
    *     language.
    * @return {string} The class name for the language item.
    * @private
    */
   getLanguageItemClass_: function(languageCode, prospectiveUILanguage,
       supportsUI) {
-    var classes = [];
+    if ((cr.isChromeOS || cr.isWindows) &&
+        this.isProspectiveUILanguage_(languageCode, prospectiveUILanguage)) {
+      return 'selected';
+    }
+    return '';
+  },
 
-    if (cr.isChromeOS || cr.isWindows) {
-      if (supportsUI && !loadTimeData.getBoolean('isGuest'))
-        classes.push('list-button');  // Makes the item look "actionable".
-
-      if (this.isProspectiveUILanguage_(languageCode, prospectiveUILanguage))
-        classes.push('selected');
-    }
-
-    return classes.join(' ');
+   /**
+   * @param {string} languageCode The language code identifying a language.
+   * @param {string} prospectiveUILanguage The prospective UI language.
+   * @return {boolean} True if the prospective UI language is set to
+   *     |languageCode| but requires a restart to take effect.
+   * @private
+   */
+  isRestartRequired_: function(languageCode, prospectiveUILanguage) {
+    return prospectiveUILanguage == languageCode &&
+        navigator.language != languageCode;
   },
 
   /**
    * @param {string} id The input method ID.
    * @param {string} currentId The ID of the currently enabled input method.
    * @return {boolean} True if the IDs match.
    * @private
    */
   isCurrentInputMethod_: function(id, currentId) {
     assert(cr.isChromeOS);
@@ skipping 35 matching lines @@
    * @param {!Event} e
    * @private
    */
   toggleMenu_: function(e) {
     e.stopPropagation();  // Prevent the tap event from closing the menu.
 
     this.detailLanguage_ =
         /** @type {!{model: !{item: !LanguageState}}} */(e).model.item;
 
     // Ensure the template has been stamped.
-    if (!this.menu_) {
-      this.$.menuTemplate.if = true;
-      this.$.menuTemplate.render();
-      this.menu_ = /** @type {CrSharedMenuElement} */(
-          this.$$('cr-shared-menu'));
+    if (!this.menu_)
+      this.initializeMenu_();
+
+    this.menu_.toggleMenu(e.target);
+  },
+
+  /**
+   * Stamps the menu and applies Chrome OS session tweaks.
+   * @private
+   */
+  initializeMenu_: function() {
+    this.$.menuTemplate.if = true;
+    this.$.menuTemplate.render();

[stevenjb, 2016/09/20 16:17:59]

This is interesting. Can you point me to the documentation for it (it's a
difficult term to search for...)

[michaelpg, 2016/09/21 19:28:13]

Changed this to use history's cr-lazy-render. But the documentation is here:
https://www.polymer-project.org/1.0/docs/api/dom-if#method-render

[stevenjb, 2016/09/21 19:45:56]

Ah, I missed that, thanks. 

+    this.menu_ = /** @type {CrSharedMenuElement} */(
+        this.$$('cr-shared-menu'));
+
+    // In a CrOS multi-user session, the primary user controls the UI language.
+    // TODO(michaelpg): The language selection should not be hidden, but should
+    // show a policy indicator. crbug.com/648498
+    if (this.isSecondaryUser_())
+      this.menu_.querySelector('#uiLanguageItem').hidden = true;
+
+    // The UI language choice doesn't persist for guests.
+    if (cr.isChromeOS &&
+        (uiAccountTweaks.UIAccountTweaks.loggedInAsGuest() ||
+         uiAccountTweaks.UIAccountTweaks.loggedInAsPublicAccount())) {
+      this.menu_.querySelector('#uiLanguageItem').hidden = true;
     }
-    this.menu_.toggleMenu(e.target);
+  },
+
+  /**
+   * Handler for the restart button.
+   * @private
+   */
+  onRestartTap_: function() {
+<if expr="chromeos">
+    settings.LifetimeBrowserProxyImpl.getInstance().logOutAndRestart();
+</if>
+<if expr="not chromeos">
+    settings.LifetimeBrowserProxyImpl.getInstance().restart();
+</if>

[stevenjb, 2016/09/20 16:17:59]

Ugh. It would be nice if we could make logOutAndRestart() behave the same as
restart() (or vice versa) on non chromeos.

[michaelpg, 2016/09/21 19:28:13]

CrOS can also "restart" without the logOut, e.g. when changing chrome://flags.
Not sure when one is preferred, just following the existing code.

[stevenjb, 2016/09/21 19:45:56]

signOutAndRestart I think is correct on Chrome OS, but personally I would also
implement that on non-chromeos and make it just do a restart.

   },
 });
 })();