MD Downloads: fix focus issue on downloads page by combining pause/play

MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOPRESUBMIT=true

Committed: https://crrev.com/9c17549d24c067ceaab8303f829fd090da0ae7d4
Cr-Commit-Position: refs/heads/master@{#406209}

[Dan Beam, 2016-07-16 00:22:48]

Description was changed from

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=groby@chromium.org
BUG=605093
==========

to

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=groby@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-07-16 00:24:05]

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

[commit-bot: I haz the power, 2016-07-16 00:24:34]

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-07-16 02:12:33]

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

[commit-bot: I haz the power, 2016-07-16 02:12:34]

Dry run: This issue passed the CQ dry run.

[Dan Beam, 2016-07-18 19:31:27]

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

[commit-bot: I haz the power, 2016-07-18 19:32:49]

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-07-18 21:43:33]

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

[commit-bot: I haz the power, 2016-07-18 21:43:34]

Dry run: This issue passed the CQ dry run.

[Dan Beam, 2016-07-19 01:40:34]

dbeam@chromium.org changed reviewers:
+ tsergeant@chromium.org
- groby@chromium.org

[Dan Beam, 2016-07-19 01:40:35]

-groby@, +tsergeant@

[Dan Beam, 2016-07-19 01:41:21]

https://codereview.chromium.org/2154033002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/md_downloads/crisper.js (right):

https://codereview.chromium.org/2154033002/diff/20001/chrome/browser/resource...
chrome/browser/resources/md_downloads/crisper.js:11418: if
(menu.items.indexOf(e.relatedTarget) >= 0)
totallllly remembered to re-vulcanize after that last downloads focus CL you
reviewed

*looks nervously for escape routes*

[tsergeant, 2016-07-19 03:39:53]

lgtm

[Dan Beam, 2016-07-19 03:55:03]

Description was changed from

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=groby@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-07-19 03:58:56]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2016-07-19 03:59:22]

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

[Dan Beam, 2016-07-19 04:00:55]

Description was changed from

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true
==========

[Dan Beam, 2016-07-19 04:01:17]

Description was changed from

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true
==========

to

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOPRESUBMIT=true
==========

[Dan Beam, 2016-07-19 04:01:55]

The CQ bit was unchecked by dbeam@chromium.org

[Dan Beam, 2016-07-19 04:02:05]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2016-07-19 04:02:40]

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-19 04:03:35]

Description was changed from

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOPRESUBMIT=true
==========

to

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-07-19 04:03:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-19 04:05:29]

Description was changed from

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOPRESUBMIT=true
==========

to

==========
MD Downloads: fix focus issue on downloads page by combining pause/play

R=tsergeant@chromium.org
BUG=605093
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOPRESUBMIT=true

Committed: https://crrev.com/9c17549d24c067ceaab8303f829fd090da0ae7d4
Cr-Commit-Position: refs/heads/master@{#406209}
==========

[commit-bot: I haz the power, 2016-07-19 04:05:30]

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