MD Settings: cr/cros - Guest mode page visibility

chrome/browser/resources/settings/settings_ui/settings_ui.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-ui' implements the UI for the Settings page.
  *
  * Example:
  *
@@ skipping 42 matching lines @@
         url += encodeURIComponent(description);
         return url;
       },
     },
 
     /** @private {boolean} */
     toolbarSpinnerActive_: {
       type: Boolean,
       value: false,
     },
+
+    /**
+     * Dictionary defining page visibility.
+     * @private {!GuestModePageVisibility}
+     */
+    pageVisibility_: {
+      type: Object,
+    },
   },
 
   listeners: {
     'sideNav.iron-activate': 'onIronActivate_',
   },
 
+  /**
+   * @override
+   * @suppress {es5Strict} Object literals cannot contain duplicate keys in ES5
+   *     strict mode.
+   */
+  ready: function() {
+    this.$$('cr-toolbar').addEventListener('search-changed', function(e) {
+      this.$$('settings-main').searchContents(e.detail);
+    }.bind(this));
+
+    if (loadTimeData.getBoolean('isGuest')) {

[dpapad, 2016/07/22 22:40:48]

Can you not move this logic at line 69 as follows?

pageVisibility_: {
  type: Object,
  value: function() {
    if (loadTimeData.getBoolean('isGuest')) {
     return ....;
    }
    return ....;
  },
},

[Moe, 2016/07/24 23:54:27]

Unfortunately not. @suppress won't work for declared properties. we would also
have to have a return value so we can't leave pageVisibility undefined. 

+      this.pageVisibility_ = {
+        people: false,
+        onStartup: false,
+        reset: false,
+<if expr="not chromeos">
+        appearance: false,
+        defaultBrowser: false,
+        advancedSettings: false,
+</if>
+<if expr="chromeos">
+        appearance: {
+          setWallpaper: false,
+          setTheme: false,
+          homeButton: false,
+          bookmarksBar: false,
+          pageZoom: false,
+        },
+        advancedSettings: true,
+        dateTime: {
+          timeZoneSelector: false,
+        },
+        privacy: {
+          searchPrediction: false,
+          networkPrediction: false,
+        },
+        passwordsAndForms: false,
+        downloads: {
+          googleDrive: false,
+        },
+</if>
+      };
+    }
+  },
+
   /** @private */
   onCloseAppealTap_: function() {
     window.sessionStorage.appealClosed_ = this.appealClosed_ = true;
   },
 
   /**
    * @param {Event} event
    * @private
    */
   onIronActivate_: function(event) {
     if (event.detail.item.id != 'advancedPage')
       this.$$('app-drawer').close();
   },
 
   /** @private */
   onMenuButtonTap_: function() {
     this.$$('app-drawer').toggle();
   },
 
-  /** @override */
-  ready: function() {
-    this.$$('cr-toolbar').addEventListener('search-changed', function(e) {
-      this.$$('settings-main').searchContents(e.detail);
-    }.bind(this));
-  },
-
   /** @private */
   directionDelegateChanged_: function() {
     this.$$('app-drawer').align = this.directionDelegate.isRtl() ?
         'right' : 'left';
   },
 });