MD Settings menu should be visible

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 18 matching lines @@
     toolbarSpinnerActive_: {
       type: Boolean,
       value: false,
     },
 
     /**
      * Dictionary defining page visibility.
      * @private {!GuestModePageVisibility}
      */
     pageVisibility_: Object,
-
-    /** @private */
-    drawerOpened_: Boolean,
   },
 
   /**
    * @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));
 
+    // Lazy-create the drawer the first time it is opened or swiped into view.
+    var drawer = assert(this.$$('app-drawer'));
+    listenOnce(drawer, 'opened-changed', this.onDrawerFirstShown_.bind(this));
+    listenOnce(drawer, 'track', this.onDrawerFirstShown_.bind(this));

[Dan Beam, 2016/09/01 01:33:13]

don't we really want this to fire once only on the first event, not on the first
of *each* event?

[michaelpg, 2016/09/01 02:20:26]

Yeah. It wasn't a big deal since it's basically a no-op the second time but this
is more correcter.

+
     window.addEventListener('popstate', function(e) {
       this.$$('app-drawer').close();
     }.bind(this));
 
     if (loadTimeData.getBoolean('isGuest')) {
       this.pageVisibility_ = {
         people: false,
         onStartup: false,
         reset: false,
 <if expr="not chromeos">
@@ skipping 38 matching lines @@
   /** @private */
   onMenuButtonTap_: function() {
     this.$$('app-drawer').toggle();
   },
 
   /** @private */
   directionDelegateChanged_: function() {
     this.$$('app-drawer').align = this.directionDelegate.isRtl() ?
         'right' : 'left';
   },
+
+  /**
+   * Ensures the settings-menu in the app-drawer is stamped.
+   * @param {!Event} e
+   * @private
+   */
+  onDrawerFirstShown_: function(e) {
+    this.$.drawerTemplate.if = true;
+  },
 });