MD Settings: cr/cros - Guest mode page visibility

chrome/browser/resources/settings/settings_main/settings_main.html

 <link rel="import" href="chrome://resources/cr_elements/icons.html">
 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/html/promise_resolver.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.html">
 <link rel="import" href="/about_page/about_page.html">
 <link rel="import" href="/advanced_page/advanced_page.html">
 <link rel="import" href="/basic_page/basic_page.html">
 <link rel="import" href="/settings_main/settings_main_rendered.html">
 <link rel="import" href="/settings_page/main_page_behavior.html">
 <link rel="import" href="/settings_root_css.html">
@@ skipping 21 matching lines @@
       #toggleSpacer {
         padding-top: 33px;  /* Part of a 48px spacer (33px + 12px + 3px). */
       }
 
       iron-icon {
         -webkit-margin-start: 16px;
       }
     </style>
     <content select="paper-icon-button"></content>
     <template is="dom-if" if="[[showBasicPage_]]">
-      <settings-basic-page prefs="{{prefs}}" current-route="{{currentRoute}}">
+      <settings-basic-page prefs="{{prefs}}" current-route="{{currentRoute}}"
+          page-visibility="[[pageVisibility]]">
       </settings-basic-page>
     </template>
-    <template is="dom-if" if="[[showAdvancedToggle_]]">
-      <div id="toggleSpacer"></div>
-      <div id="toggleContainer">
-        <div id="advancedToggle" on-tap="toggleAdvancedPage_">
-          <span>$i18n{advancedPageTitle}</span>
-          <iron-icon icon="[[arrowState_(showAdvancedPage_)]]"></iron-icon>
+    <template is="dom-if" if="[[!pageVisibility.hideAdvancedSettings]]">

[michaelpg, 2016/07/13 00:01:35]

Please address this in currentRouteChanged_ instead of creating a new dom-if.
The logic there needs some more work to integrate this.

[Moe, 2016/07/13 20:17:17]

I did initially handle this in the observer (it needs to observe currentRoute as
well as pageVisibility). It adds more obscurity to the already
difficult-to-follow logic there. This is a lot cleaner IMO. Unless you really
want me to do it that way :)

[michaelpg, 2016/07/14 00:31:28]

Meh. It's inherently illogical that (!hideAdvancedSettings) doesn't imply
(showAdvancedPage_). But I'll be changing these to computed properties sooner or
later anyway, so that should simplify things.

[Moe, 2016/07/14 17:19:55]

+1 for computed properties.

[michaelpg, 2016/07/21 21:58:44]

FYI, unfortunately computed properties ended up being painful too, because we
now want to adjust these manually when searching the page.

I have a CL that simplifies currentRouteChanged_ a bit, maybe I'll go back and
do this later, the dom-if's are okay for now though.
https://codereview.chromium.org/2160713002/

+      <template is="dom-if" if="[[showAdvancedToggle_]]">
+        <div id="toggleSpacer"></div>
+        <div id="toggleContainer">
+          <div id="advancedToggle" on-tap="toggleAdvancedPage_">
+            <span>$i18n{advancedPageTitle}</span>
+            <iron-icon icon="[[arrowState_(showAdvancedPage_)]]"></iron-icon>
+          </div>
         </div>
-      </div>
-    </template>
-    <template is="dom-if" if="[[showAdvancedPage_]]">
-      <settings-advanced-page prefs="{{prefs}}"
-          current-route="{{currentRoute}}">
-      </settings-advanced-page>
+      </template>
+      <template is="dom-if" if="[[showAdvancedPage_]]">
+        <settings-advanced-page prefs="{{prefs}}" current-route="{{currentRoute}}"
+            page-visibility="[[pageVisibility]]">
+        </settings-advanced-page>
+      </template>
     </template>
     <template is="dom-if" if="[[showAboutPage_]]">
       <settings-about-page current-route="{{currentRoute}}">
       </settings-about-page>
     </template>
   </template>
   <script src="settings_main.js"></script>
   <script src="/search_settings.js"></script>
 </dom-module>