Avoid to call KeyboardController::ResetInstance twice

Avoid to call KeyboardController::ResetInstance twice

Removed KeyboardController::ResetInstance(nullptr) in ~Shell
because it should have been called already in DeactivateKeyboard.

BUG=None
TEST=try

Review-Url: https://codereview.chromium.org/2758573002
Cr-Commit-Position: refs/heads/master@{#458240}
Committed: https://chromium.googlesource.com/chromium/src/+/70c46acd7f5117902acf57801960abc64e314825

[oka, 2017-03-16 15:31:10]

Description was changed from

==========
Avoid to call KeyboardController.reset(nullptr) twice.

Removed KeyboardController::ResetInstance(nullptr) in ~Shell
because it should have been called already in DeactivateKeyboard.

BUG=None
TEST=try

Fix typo.
==========

to

==========
Avoid to call KeyboardController.reset(nullptr) twice.

Removed KeyboardController::ResetInstance(nullptr) in ~Shell
because it should have been called already in DeactivateKeyboard.

BUG=None
TEST=try
==========

[oka, 2017-03-16 15:33:14]

Description was changed from

==========
Avoid to call KeyboardController.reset(nullptr) twice.

Removed KeyboardController::ResetInstance(nullptr) in ~Shell
because it should have been called already in DeactivateKeyboard.

BUG=None
TEST=try
==========

to

==========
Avoid to call KeyboardController::ResetInstance twice

Removed KeyboardController::ResetInstance(nullptr) in ~Shell
because it should have been called already in DeactivateKeyboard.

BUG=None
TEST=try
==========

[oka, 2017-03-16 15:33:42]

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

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

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

[oka, 2017-03-16 15:40:25]

oka@chromium.org changed reviewers:
+ oshima@chromium.org

[oka, 2017-03-16 15:40:26]

PTAL.

[commit-bot: I haz the power, 2017-03-16 16:14:31]

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

[commit-bot: I haz the power, 2017-03-16 16:14:32]

Dry run: This issue passed the CQ dry run.

[oshima, 2017-03-16 18:54:39]

lgtm

[oka, 2017-03-17 08:07:24]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2017-03-17 08:07:39]

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

[commit-bot: I haz the power, 2017-03-17 08:07:40]

This CL has an open dependency (Issue 2747373006 Patch 1). 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.

[oka, 2017-03-20 22:51:43]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2017-03-20 22:52:14]

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-03-20 23:39:33]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490050303633880,
"parent_rev": "4b4a1b0b950b12e6c754e9af87e627019b87dd70", "commit_rev":
"70c46acd7f5117902acf57801960abc64e314825"}

[commit-bot: I haz the power, 2017-03-20 23:40:25]

Description was changed from

==========
Avoid to call KeyboardController::ResetInstance twice

Removed KeyboardController::ResetInstance(nullptr) in ~Shell
because it should have been called already in DeactivateKeyboard.

BUG=None
TEST=try
==========

to

==========
Avoid to call KeyboardController::ResetInstance twice

Removed KeyboardController::ResetInstance(nullptr) in ~Shell
because it should have been called already in DeactivateKeyboard.

BUG=None
TEST=try

Review-Url: https://codereview.chromium.org/2758573002
Cr-Commit-Position: refs/heads/master@{#458240}
Committed:
https://chromium.googlesource.com/chromium/src/+/70c46acd7f5117902acf57801960...
==========

[commit-bot: I haz the power, 2017-03-20 23:40:26]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/70c46acd7f5117902acf57801960...