Refactor: Extract a KeyboardHandler and CommandHandler from Background

Refactor: Extract a KeyboardHandler and CommandHandler from Background

Besides the mechanical changes needed, some simplification of the mode change logic was done. These changes should have no functional effects.

In addition, many commands that required a range in CommandHandler.onCommand were moved to the switch block not requiring a current range. This should be the only behavioral change introduced.

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

Committed: https://crrev.com/e451a40f0789251b6bebd1df69a00bc49dcac873
Cr-Commit-Position: refs/heads/master@{#405799}

[David Tseng, 2016-07-13 17:14:32]

Description was changed from

==========
Group commands that don't require a current range.
==========

to

==========
Group commands that don't require a current range.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-07-13 17:16:29]

Description was changed from

==========
Group commands that don't require a current range.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Group commands that don't require a current range.


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

[David Tseng, 2016-07-13 19:27:03]

Description was changed from

==========
Group commands that don't require a current range.


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

to

==========
Refactor: Extract a KeyboardHandler and CommandHandler from Background

Besides the mechanical changes needed, some simplification of the mode change
logic was done. These changes should have no functional effects.

In addition, many commands that required a range in CommandHandler.onCommand
were moved to the switch block not requiring a current range. This should be the
only behavioral change introduced.

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

[David Tseng, 2016-07-13 19:27:04]

dtseng@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[David Tseng, 2016-07-13 19:27:15]

Patchset #1 (id:1) has been deleted

[dmazzoni, 2016-07-13 23:46:18]

lgtm

[David Tseng, 2016-07-15 16:30:27]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-07-15 16:31:03]

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-15 16:31:04]

Your CL relies on deprecated CQ feature(s):
 * Specifying master names in CQ_INCLUDE_TRYBOTS part of description without
"master." prefix is deprecated:
     tryserver.chromium.linux
   For more details, see http://crbug.com/617627.

[commit-bot: I haz the power, 2016-07-15 18:36:51]

Description was changed from

==========
Refactor: Extract a KeyboardHandler and CommandHandler from Background

Besides the mechanical changes needed, some simplification of the mode change
logic was done. These changes should have no functional effects.

In addition, many commands that required a range in CommandHandler.onCommand
were moved to the switch block not requiring a current range. This should be the
only behavioral change introduced.

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

to

==========
Refactor: Extract a KeyboardHandler and CommandHandler from Background

Besides the mechanical changes needed, some simplification of the mode change
logic was done. These changes should have no functional effects.

In addition, many commands that required a range in CommandHandler.onCommand
were moved to the switch block not requiring a current range. This should be the
only behavioral change introduced.

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

[commit-bot: I haz the power, 2016-07-15 18:36:52]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-07-15 18:37:13]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-15 18:38:31]

Description was changed from

==========
Refactor: Extract a KeyboardHandler and CommandHandler from Background

Besides the mechanical changes needed, some simplification of the mode change
logic was done. These changes should have no functional effects.

In addition, many commands that required a range in CommandHandler.onCommand
were moved to the switch block not requiring a current range. This should be the
only behavioral change introduced.

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

to

==========
Refactor: Extract a KeyboardHandler and CommandHandler from Background

Besides the mechanical changes needed, some simplification of the mode change
logic was done. These changes should have no functional effects.

In addition, many commands that required a range in CommandHandler.onCommand
were moved to the switch block not requiring a current range. This should be the
only behavioral change introduced.

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

Committed: https://crrev.com/e451a40f0789251b6bebd1df69a00bc49dcac873
Cr-Commit-Position: refs/heads/master@{#405799}
==========

[commit-bot: I haz the power, 2016-07-15 18:38:32]

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