[MD History] Clear ripple on CBD press.

[MD History] Clear ripple on CBD press.

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
NO_DEPENDENCY_CHECKS=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/348a850f2f2999585d354ce729864b578b4c9a36
Cr-Commit-Position: refs/heads/master@{#418807}

[calamity, 2016-08-29 05:34:48]

Description was changed from

==========
[MD History] Clear ripple on CBD press.

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
==========

to

==========
[MD History] Clear ripple on CBD press.

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[calamity, 2016-09-02 06:32:48]

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

[Dan Beam, 2016-09-02 23:24:51]

https://codereview.chromium.org/2282403002/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_history/side_bar.html (right):

https://codereview.chromium.org/2282403002/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_history/side_bar.html:88:
<paper-ripple></paper-ripple>
you guys should construct these dynamically rather than declaratively if
performance is important

[tsergeant, 2016-09-05 23:52:52]

lgtm

Do you know why this is happening? Is it because a new tab opens and eats the up
action?

[calamity, 2016-09-06 04:05:24]

On 2016/09/05 23:52:52, tsergeant wrote:
> lgtm
> 
> Do you know why this is happening? Is it because a new tab opens and eats the
up
> action?

It clears the ripple on keyup which ends up happening on the new tab.

[calamity, 2016-09-06 04:05:28]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2016-09-06 04:05:33]

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

[commit-bot: I haz the power, 2016-09-06 04:05:34]

This CL has an open dependency (Issue 2278383002 Patch 20001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[calamity, 2016-09-06 06:36:48]

Description was changed from

==========
[MD History] Clear ripple on CBD press.

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD History] Clear ripple on CBD press.

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
NO_DEPENDENCY_CHECKS=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[calamity, 2016-09-06 07:02:33]

The CQ bit was checked by calamity@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-06 07:02:47]

Dry run: 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-09-06 07:05:01]

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

[commit-bot: I haz the power, 2016-09-06 07:05:02]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[calamity, 2016-09-13 03:50:58]

The CQ bit was checked by calamity@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-13 03:51:18]

Dry run: 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-09-13 04:01:02]

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

[commit-bot: I haz the power, 2016-09-13 04:01:03]

Dry run: Try jobs failed on following builders:
  closure_compilation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/closure_compila...)

[calamity, 2016-09-15 00:48:47]

The CQ bit was checked by calamity@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-15 00:50:06]

Dry run: 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-09-15 01:02:31]

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

[commit-bot: I haz the power, 2016-09-15 01:02:32]

Dry run: Try jobs failed on following builders:
  closure_compilation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/closure_compila...)

[calamity, 2016-09-15 01:49:14]

The CQ bit was checked by calamity@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-15 01:49:36]

Dry run: 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-09-15 01:58:43]

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

[commit-bot: I haz the power, 2016-09-15 01:58:44]

Dry run: Try jobs failed on following builders:
  closure_compilation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/closure_compila...)

[calamity, 2016-09-15 03:13:57]

Patchset #3 (id:30001) has been deleted

[calamity, 2016-09-15 07:42:35]

The CQ bit was checked by calamity@chromium.org

[calamity, 2016-09-15 07:42:37]

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/2282403002/#ps50001
(title: "rebase")

[commit-bot: I haz the power, 2016-09-15 07:42:51]

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-09-15 08:20:37]

Description was changed from

==========
[MD History] Clear ripple on CBD press.

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
NO_DEPENDENCY_CHECKS=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD History] Clear ripple on CBD press.

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
NO_DEPENDENCY_CHECKS=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-09-15 08:20:38]

Committed patchset #3 (id:50001)

[commit-bot: I haz the power, 2016-09-15 08:23:53]

Description was changed from

==========
[MD History] Clear ripple on CBD press.

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
NO_DEPENDENCY_CHECKS=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD History] Clear ripple on CBD press.

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
NO_DEPENDENCY_CHECKS=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/348a850f2f2999585d354ce729864b578b4c9a36
Cr-Commit-Position: refs/heads/master@{#418807}
==========

[commit-bot: I haz the power, 2016-09-15 08:23:54]

Patchset 3 (id:??) landed as
https://crrev.com/348a850f2f2999585d354ce729864b578b4c9a36
Cr-Commit-Position: refs/heads/master@{#418807}