MD Bookmarks: Make shortcuts for select all/deselect all global

MD Bookmarks: Make shortcuts for select all/deselect all global

These shortcuts (Ctrl/Meta-A and Escape) can now be triggered while
the sidebar or document are active.

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

Review-Url: https://codereview.chromium.org/2929053004
Cr-Commit-Position: refs/heads/master@{#479975}
Committed: https://chromium.googlesource.com/chromium/src/+/679159f7fd567e7c1953a1c1548ee42ae55d2907

[tsergeant, 2017-06-09 05:48:22]

Description was changed from

==========
MD Bookmarks: Make shortcuts for select all/deselect all global

These shortcuts (Ctrl/Meta-A and Escape) can now be triggered while
the sidebar or document are active.

BUG=730405
==========

to

==========
MD Bookmarks: Make shortcuts for select all/deselect all global

These shortcuts (Ctrl/Meta-A and Escape) can now be triggered while
the sidebar or document are active.

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

[tsergeant, 2017-06-09 05:49:24]

Description was changed from

==========
MD Bookmarks: Make shortcuts for select all/deselect all global

These shortcuts (Ctrl/Meta-A and Escape) can now be triggered while
the sidebar or document are active.

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

to

==========
MD Bookmarks: Make shortcuts for select all/deselect all global

These shortcuts (Ctrl/Meta-A and Escape) can now be triggered while
the sidebar or document are active.

This CL also changes how shortcuts are registered to make it easier to
read.

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

[tsergeant, 2017-06-09 05:50:06]

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

[commit-bot: I haz the power, 2017-06-09 05:50:18]

Dry run: CQ is trying da patch.

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

[tsergeant, 2017-06-09 05:51:26]

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

[commit-bot: I haz the power, 2017-06-09 06:31:19]

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

[commit-bot: I haz the power, 2017-06-09 06:31:20]

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

[tsergeant, 2017-06-09 06:48:35]

On 2017/06/09 05:51:27, tsergeant wrote:

I'll look into the failing tests next week.

[tsergeant, 2017-06-14 03:47:48]

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

[tsergeant, 2017-06-14 03:47:58]

Alright, the tests are fixed, PTAL.

[commit-bot: I haz the power, 2017-06-14 03:48:03]

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, 2017-06-14 04:59:36]

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

[commit-bot: I haz the power, 2017-06-14 04:59:37]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[calamity, 2017-06-15 04:32:15]

lgtm, update CL description (refactor moved upstream?)

[tsergeant, 2017-06-15 07:00:19]

Description was changed from

==========
MD Bookmarks: Make shortcuts for select all/deselect all global

These shortcuts (Ctrl/Meta-A and Escape) can now be triggered while
the sidebar or document are active.

This CL also changes how shortcuts are registered to make it easier to
read.

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

to

==========
MD Bookmarks: Make shortcuts for select all/deselect all global

These shortcuts (Ctrl/Meta-A and Escape) can now be triggered while
the sidebar or document are active.

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

[tsergeant, 2017-06-16 03:56:01]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2017-06-16 03:56:16]

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, 2017-06-16 05:04:15]

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

[commit-bot: I haz the power, 2017-06-16 05:04:16]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[tsergeant, 2017-06-16 05:13:30]

The CQ bit was checked by tsergeant@chromium.org

[tsergeant, 2017-06-16 05:13:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
calamity@chromium.org
Link to the patchset: https://codereview.chromium.org/2929053004/#ps40001
(title: "Fix test flub")

[commit-bot: I haz the power, 2017-06-16 05:13:59]

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, 2017-06-16 06:58:43]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1497590010702730,
"parent_rev": "2696d5a95cc63cb1cf460afbe1f94d4005e366e4", "commit_rev":
"679159f7fd567e7c1953a1c1548ee42ae55d2907"}

[commit-bot: I haz the power, 2017-06-16 06:58:53]

Description was changed from

==========
MD Bookmarks: Make shortcuts for select all/deselect all global

These shortcuts (Ctrl/Meta-A and Escape) can now be triggered while
the sidebar or document are active.

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

to

==========
MD Bookmarks: Make shortcuts for select all/deselect all global

These shortcuts (Ctrl/Meta-A and Escape) can now be triggered while
the sidebar or document are active.

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

Review-Url: https://codereview.chromium.org/2929053004
Cr-Commit-Position: refs/heads/master@{#479975}
Committed:
https://chromium.googlesource.com/chromium/src/+/679159f7fd567e7c1953a1c1548e...
==========

[commit-bot: I haz the power, 2017-06-16 06:58:54]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/679159f7fd567e7c1953a1c1548e...