MD Bookmarks: Update URL router to work in new data binding system

chrome/browser/resources/md_bookmarks/toolbar.js

Index: chrome/browser/resources/md_bookmarks/toolbar.js
diff --git a/chrome/browser/resources/md_bookmarks/toolbar.js b/chrome/browser/resources/md_bookmarks/toolbar.js
index 6b89342cf60b2116c08b802315b13bf06ec394c7..4dac0cd7f727f6396e27674118bbf3e049179c91 100644
--- a/chrome/browser/resources/md_bookmarks/toolbar.js
+++ b/chrome/browser/resources/md_bookmarks/toolbar.js
@@ -77,7 +77,8 @@ Polymer({
    */
   onSearchChanged_: function(e) {
     var searchTerm = /** @type {string} */ (e.detail);
-    this.dispatch(bookmarks.actions.setSearchTerm(searchTerm));
+    if (searchTerm != this.searchTerm_)

[calamity, 2017/03/20 06:34:28]

What makes this necessary?

[tsergeant, 2017/03/21 00:27:07]

This is just a small optimisation to avoid firing off an action which does
nothing.

The flow is:

router fires a search action
StoreClient picks up a change to search term
onSearchTermChanged_ updates the search field
onSearchChanged_ sees that change to the search field, fires off an identical
search action.

This check avoids that second action.

+      this.dispatch(bookmarks.actions.setSearchTerm(searchTerm));
   },
 
   /** @private */