[MD settings] remove unneeded properties

chrome/browser/resources/settings/settings_menu/settings_menu.js

Index: chrome/browser/resources/settings/settings_menu/settings_menu.js
diff --git a/chrome/browser/resources/settings/settings_menu/settings_menu.js b/chrome/browser/resources/settings/settings_menu/settings_menu.js
index 45ddc5f339629396d2beb0397473b98b7db86687..998b008ef1d5d72687b5bf72fa82e3594adcdfd6 100644
--- a/chrome/browser/resources/settings/settings_menu/settings_menu.js
+++ b/chrome/browser/resources/settings/settings_menu/settings_menu.js
@@ -15,12 +15,6 @@ Polymer({
   is: 'settings-menu',
 
   properties: {
-    /** @private */
-    advancedOpened_: Boolean,
-
-    /** @private */
-    basicOpened_: Boolean,
-
     /**
      * The current active route.
      */
@@ -33,15 +27,21 @@ Polymer({
 
   /** @private */
   currentRouteChanged_: function() {
-    var submenu = this.shadowRoot.querySelector(
-        'paper-submenu[data-page="' + this.currentRoute.page + '"]');
-    if (submenu)
-      submenu.opened = true;
+    // Sync URL changes to the side nav menu.
+
+    this.$.advancedPage.opened = this.currentRoute.page == 'advanced';

[dpapad, 2016/05/03 01:00:02]

Did you consider updating those 4 properties via bindings instead?

No need to change, I am just curious because I have faced the same dilemma in
the past (update N properties in a centralized spot  programatically VS spread N
data bindings in HTML).

[dschuyler, 2016/05/03 20:48:53]

Yeah, it's nicer if these are in bound in the html, imo.
The reason for them being here is for URL changes. I.e.
navigating directly to a url such as chrome://md-settings/reset

+    this.$.basicPage.opened = this.currentRoute.page == 'basic';
+
+    if (this.$.advancedPage.opened)
+      this.$.advancedMenu.selected = this.currentRoute.section;
+
+    if (this.$.basicPage.opened)
+      this.$.basicMenu.selected = this.currentRoute.section;
   },
 
   /** @private */
   openPage_: function(event) {
-    var submenuRoute = event.currentTarget.dataset.page;
+    var submenuRoute = event.currentTarget.parentNode.dataset.page;
     if (submenuRoute) {
       this.currentRoute = {
         page: submenuRoute,