[MD History] Add 2-way binding to query state.

[MD History] Add 2-way binding to query state.

This CL fixes an issue where the query state was not correctly bound to
the list container. This caused the loading spinner and message to break.

BUG=619799
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/279f2470f520e8d7dbdd90066d06eac1a25f647d
Cr-Commit-Position: refs/heads/master@{#405068}

[calamity, 2016-07-07 08:12:36]

Description was changed from

==========
[MD History] Add 2-way binding to query state.

This CL fixes an issue where the query state was not correctly bound to
the list container.

BUG=619799
==========

to

==========
[MD History] Add 2-way binding to query state.

This CL fixes an issue where the query state was not correctly bound to
the list container.

BUG=619799
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[calamity, 2016-07-08 01:44:07]

Patchset #1 (id:1) has been deleted

[calamity, 2016-07-08 01:44:50]

Description was changed from

==========
[MD History] Add 2-way binding to query state.

This CL fixes an issue where the query state was not correctly bound to
the list container.

BUG=619799
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD History] Add 2-way binding to query state.

This CL fixes an issue where the query state was not correctly bound to
the list container. This caused the loading spinner and message to break.

BUG=619799
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[calamity, 2016-07-08 01:45:02]

calamity@chromium.org changed reviewers:
+ tsergeant@chromium.org

[tsergeant, 2016-07-08 02:24:45]

{{lgtm}}

[tsergeant, 2016-07-08 02:25:21]

On 2016/07/08 02:24:45, tsergeant wrote:
> {{lgtm}}

(and woo-hoo, increasing test coverage)

[calamity, 2016-07-08 05:46:28]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2016-07-08 05:47:00]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-08 14:22:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-08 14:22:35]

Failed to apply patch for
chrome/test/data/webui/md_history/history_toolbar_test.js:
While running git apply --index -3 -p1;
  error: patch failed:
chrome/test/data/webui/md_history/history_toolbar_test.js:45
  Falling back to three-way merge...
  Applied patch to 'chrome/test/data/webui/md_history/history_toolbar_test.js'
with conflicts.
  U chrome/test/data/webui/md_history/history_toolbar_test.js

Patch:       chrome/test/data/webui/md_history/history_toolbar_test.js
Index: chrome/test/data/webui/md_history/history_toolbar_test.js
diff --git a/chrome/test/data/webui/md_history/history_toolbar_test.js
b/chrome/test/data/webui/md_history/history_toolbar_test.js
index
86f6f02b4b08ee39cda66844f1a1a2f62825b14a..efd43ebc56051a229b1299df7b64fc4895cfc5a4
100644
--- a/chrome/test/data/webui/md_history/history_toolbar_test.js
+++ b/chrome/test/data/webui/md_history/history_toolbar_test.js
@@ -45,13 +45,25 @@ cr.define('md_history.history_toolbar_test', function() {
       test('search term gathered correctly from toolbar', function(done) {
         app.queryState_.queryingDisabled = false;
         registerMessageCallback('queryHistory', this, function (info) {
-          assertEquals(info[0], 'Test');
+          assertEquals('Test', info[0]);
           done();
         });
 
         toolbar.$$('cr-toolbar').fire('search-changed', 'Test');
       });
 
+      test('spinner is active on search' , function(done) {
+        app.queryState_.queryingDisabled = false;
+        registerMessageCallback('queryHistory', this, function (info) {
+          assertTrue(toolbar.spinnerActive);
+          app.historyResult(createHistoryInfo(), TEST_HISTORY_RESULTS);
+          assertFalse(toolbar.spinnerActive);
+          done();
+        });
+
+        toolbar.$$('cr-toolbar').fire('search-changed', 'Test2');
+      });
+
       teardown(function() {
         element.historyData_ = [];
         element.searchedTerm = '';

[commit-bot: I haz the power, 2016-07-08 14:22:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-08 14:22:48]

CQ bit was unchecked.

[calamity, 2016-07-13 05:42:12]

The CQ bit was checked by calamity@chromium.org

[calamity, 2016-07-13 05:42:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
tsergeant@chromium.org
Link to the patchset: https://codereview.chromium.org/2126173002/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2016-07-13 05:42:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-13 07:27:09]

Description was changed from

==========
[MD History] Add 2-way binding to query state.

This CL fixes an issue where the query state was not correctly bound to
the list container. This caused the loading spinner and message to break.

BUG=619799
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD History] Add 2-way binding to query state.

This CL fixes an issue where the query state was not correctly bound to
the list container. This caused the loading spinner and message to break.

BUG=619799
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-07-13 07:27:10]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-07-13 07:28:45]

Description was changed from

==========
[MD History] Add 2-way binding to query state.

This CL fixes an issue where the query state was not correctly bound to
the list container. This caused the loading spinner and message to break.

BUG=619799
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD History] Add 2-way binding to query state.

This CL fixes an issue where the query state was not correctly bound to
the list container. This caused the loading spinner and message to break.

BUG=619799
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/279f2470f520e8d7dbdd90066d06eac1a25f647d
Cr-Commit-Position: refs/heads/master@{#405068}
==========

[commit-bot: I haz the power, 2016-07-13 07:28:47]

Patchset 2 (id:??) landed as
https://crrev.com/279f2470f520e8d7dbdd90066d06eac1a25f647d
Cr-Commit-Position: refs/heads/master@{#405068}