MD Settings: cr/cros - Guest mode page visibility

chrome/browser/resources/settings/settings_main/settings_main.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-main' displays the selected settings page.
  */
 Polymer({
   is: 'settings-main',
@@ skipping 39 matching lines @@
     showBasicPage_: {
       type: Boolean,
       value: true,
     },
 
     /** @private */
     showAboutPage_: {
       type: Boolean,
       value: false,
     },
+
+    /**
+     * Dictionary defining page visibility.
+     * @type {Object}
+     */
+    pageVisibility: {
+      type: Object,
+      value: function() { return {}; },
+    },
   },
 
   /** @override */
   created: function() {
     /** @private {!PromiseResolver} */
     this.resolver_ = new PromiseResolver;
     settings.main.rendered = this.resolver_.promise;
   },
 
   attached: function() {
@@ skipping 42 matching lines @@
    */
   currentRouteChanged_: function(newRoute) {
     var isSubpage = !!newRoute.subpage.length;
 
     this.showAboutPage_ = newRoute.page == 'about';
 
     this.showAdvancedToggle_ = !this.showAboutPage_ && !isSubpage;
 
     this.showBasicPage_ = this.showAdvancedToggle_ || newRoute.page == 'basic';
 
+    if (newRoute.page == 'advanced')

[michaelpg, 2016/07/20 19:40:46]

{} around multi-line if bodies

[Moe, 2016/07/21 18:27:32]

Done.

+      assert(! /** @type {{hideAdvancedSettings: boolean}} */

[michaelpg, 2016/07/20 19:40:46]

this cast isn't right: the thing in parens is a boolean.

[Moe, 2016/07/21 18:27:32]

Done.

+          (this.pageVisibility.hideAdvancedSettings));
+
     this.showAdvancedPage_ =
         (this.isAdvancedMenuOpen_ && this.showAdvancedToggle_) ||
         newRoute.page == 'advanced';
 
     this.style.height = isSubpage ? '100%' : '';
   },
 
   /** @private */
   toggleAdvancedPage_: function() {
     this.fire('toggle-advanced-page', !this.isAdvancedMenuOpen_);
   },
 
   /**
    * Navigates to the default search page (if necessary).
    * @private
    */
   ensureInDefaultSearchPage_: function() {
     if (this.currentRoute.page != 'basic' ||
         this.currentRoute.section != '' ||
         this.currentRoute.subpage.length != 0) {
       this.currentRoute = {page: 'basic', section: '', subpage: [], url: ''};
     }
   },
 
   /**
    * @param {string} query
    */
   searchContents: function(query) {
+    assert(! /** @type {{hideToolbarSearchField: boolean}} */

[michaelpg, 2016/07/20 19:40:46]

ditto

[Moe, 2016/07/21 18:27:32]

took search related parts out.

+        (this.pageVisibility.hideToolbarSearchField));
+
     this.ensureInDefaultSearchPage_();
 
     // Trigger rendering of the basic and advanced pages and search once ready.
     // Even if those are already rendered, yield to the message loop before
     // initiating searching.
     this.showBasicPage_ = true;
     setTimeout(function() {
       settings.search(query, assert(this.$$('settings-basic-page')));
     }.bind(this), 0);
 
     this.showAdvancedPage_ = true;
     setTimeout(function() {
       settings.search(query, assert(this.$$('settings-advanced-page')));
     }.bind(this), 0);
   },
 });