MD Settings: CrOS: Add secondary user banner

chrome/browser/resources/settings/basic_page/basic_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-basic-page' is the settings page containing the actual settings.
  */
 Polymer({
   is: 'settings-basic-page',
@@ skipping 35 matching lines @@
      * True if the basic page should currently display the reset profile banner.
      * @private {boolean}
      */
     showResetProfileBanner_: {
       type: Boolean,
       value: function() {
         return loadTimeData.getBoolean('showResetProfileBanner');
       },
     },
 
+// <if expr="chromeos">
+    /** @private Whether the user is a secondary user. */
+    showSecondaryUserBanner_: {
+      type: Boolean,
+      computed: 'computeShowSecondaryUserBanner_(hasExpandedSection_)',
+    },
+// </if>

[dschuyler, 2017/02/11 02:54:56]

Can this var be removed and computeShowSecondaryUserBanner_ be used from HTML?

[stevenjb, 2017/02/13 18:28:00]

This needs to be computed after loadTimeData is available. Added a comment.

+
     /** @private {!settings.Route|undefined} */
     currentRoute_: Object,
   },
 
   listeners: {
     'subpage-expand': 'onSubpageExpanded_',
   },
 
   /** @override */
   attached: function() {
@@ skipping 35 matching lines @@
         query, assert(this.$$('#basicPage')));
 
     if (this.pageVisibility.advancedSettings !== false) {
       assert(whenSearchDone === settings.getSearchManager().search(
           query, assert(this.$.advancedPageTemplate.get())));
     }
 
     return whenSearchDone;
   },
 
+// <if expr="chromeos">
+  /**
+   * @return {boolean}
+   * @private
+   */
+  computeShowSecondaryUserBanner_: function() {
+    return !this.hasExpandedSection_ &&
+        loadTimeData.getBoolean('isSecondaryUser');
+  },
+// </if>
+
   /** @private */
   onResetDone_: function() {
     this.showResetProfileBanner_ = false;
   },
 
   /**
    * @return {boolean}
    * @private
    */
   shouldShowAndroidApps_: function() {
@@ skipping 43 matching lines @@
 
   /**
    * @param {!settings.Route} currentRoute
    * @param {boolean} inSearchMode
    * @param {boolean} hasExpandedSection
    * @param {boolean} advancedToggleExpanded
    * @return {boolean} Whether to show the advanced page, taking into account
    *     both routing and search state.
    * @private
    */
-  showAdvancedPage_: function(currentRoute, inSearchMode, hasExpandedSection,
-                              advancedToggleExpanded) {
+  showAdvancedPage_: function(
+      currentRoute, inSearchMode, hasExpandedSection, advancedToggleExpanded) {
     return hasExpandedSection ?
         settings.Route.ADVANCED.contains(currentRoute) :
         advancedToggleExpanded || inSearchMode;
   },
 
   /**
    * @param {(boolean|undefined)} visibility
    * @return {boolean} True unless visibility is false.
    * @private
    */
   showAdvancedSettings_: function(visibility) {
     return visibility !== false;
   },
 
   /**
    * @param {boolean} opened Whether the menu is expanded.
    * @return {string} Icon name.
    * @private
    */
   getArrowIcon_: function(opened) {
     return opened ? 'cr:arrow-drop-up' : 'cr:arrow-drop-down';
   },
 });