MD Settings: First iteration of searching within settings.

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 41 matching lines @@
       value: true,
     },
 
     /** @private */
     showAboutPage_: {
       type: Boolean,
       value: false,
     },
   },
 
+  /** @override */
   created: function() {
     /** @private {!PromiseResolver} */
     this.resolver_ = new PromiseResolver;
     settings.main.rendered = this.resolver_.promise;
   },
 
   attached: function() {
     document.addEventListener('toggle-advanced-page', function(e) {
       this.showAdvancedPage_ = e.detail;
       this.isAdvancedMenuOpen_ = e.detail;
@@ skipping 45 matching lines @@
         (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: ''};

[Dan Beam, 2016/07/01 21:32:35]

nit: can you just omit the URL? and maybe add a route for search?

[dpapad, 2016/07/01 21:59:42]

The 'url' field is needed only for the Compiler. I prefer not to deal with
routing in this CL, but the goal is to add routing for searching in follow up CL
(after the core logic is completed).

[Dan Beam, 2016/07/01 22:41:01]

fun fact: it's only required if you:
a) depend settings_router and actually get the @typedef (otherwise it's silently
ignored, oh yay!)
b) land before my CL to make it optional ;)[1]

[1] https://codereview.chromium.org/2115133002

[Dan Beam, 2016/07/01 22:48:27]

or dschuyler@'s CL that makes url: (string|undefined), that'd also help ;)

https://codereview.chromium.org/2111223002/

+    }
+  },
+
+  /**
+   * @param {string} query
+   */
+  searchContents: function(query) {
+    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);
+  },
 });