Stop hiding soft input from CompositorViewHolder

Stop hiding soft input from CompositorViewHolder

CompositorViewHolder hides keyboard on switching tabs. We found that it
is causing a StrictMode violation error in KK (due to a flaw in Android
framework).

But CVH removes ContentView, which eventually calls
ImeAdapter#onViewFocusChanged(false) and then we hide the keyboard anyways.

Therefore, removing the redundant code from CVH and adding a test to
future-proof the behavior.

BUG=623256
TBR=dtrainor@chromium.org

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

[Changwan Ryu, 2016-06-30 08:45:07]

changwan@chromium.org changed reviewers:
+ aelias@chromium.org

[Changwan Ryu, 2016-06-30 08:45:08]

aelias@, could you take a look in IME's regard?

[aelias_OOO_until_Jul13, 2016-06-30 19:21:56]

aelias@chromium.org changed reviewers:
+ dfalcantara@chromium.org
- dtrainor@chromium.org

[aelias_OOO_until_Jul13, 2016-06-30 19:22:13]

lgtm.  Replacing OWNERS reviewer with dfalcantara@ since dtrainor@ is OOO.

[aelias_OOO_until_Jul13, 2016-06-30 19:23:00]

Description was changed from

==========
Stop hiding soft input from CompositorViewHolder

CompositorViewHolder hides keyboard on switching tabs. We found that it
is causing a StrictMode violation error in KK (which is an Android
framework error).

But CVH removes ContentView, which eventually calls
ImeAdapter#onViewFocusChanged(false) and then we hide the keyboard anyways.

Therefore, removing the redundant code from CVH and adding a test to
future-proof the behavior.

BUG=623256
TBR=dtrainor@chromium.org
==========

to

==========
Stop hiding soft input from CompositorViewHolder

CompositorViewHolder hides keyboard on switching tabs. We found that it
is causing a StrictMode violation error in KK (which is a flaw in Android
framework).

But CVH removes ContentView, which eventually calls
ImeAdapter#onViewFocusChanged(false) and then we hide the keyboard anyways.

Therefore, removing the redundant code from CVH and adding a test to
future-proof the behavior.

BUG=623256
TBR=dtrainor@chromium.org
==========

[aelias_OOO_until_Jul13, 2016-06-30 19:23:15]

Description was changed from

==========
Stop hiding soft input from CompositorViewHolder

CompositorViewHolder hides keyboard on switching tabs. We found that it
is causing a StrictMode violation error in KK (which is a flaw in Android
framework).

But CVH removes ContentView, which eventually calls
ImeAdapter#onViewFocusChanged(false) and then we hide the keyboard anyways.

Therefore, removing the redundant code from CVH and adding a test to
future-proof the behavior.

BUG=623256
TBR=dtrainor@chromium.org
==========

to

==========
Stop hiding soft input from CompositorViewHolder

CompositorViewHolder hides keyboard on switching tabs. We found that it
is causing a StrictMode violation error in KK (due to a flaw in Android
framework).

But CVH removes ContentView, which eventually calls
ImeAdapter#onViewFocusChanged(false) and then we hide the keyboard anyways.

Therefore, removing the redundant code from CVH and adding a test to
future-proof the behavior.

BUG=623256
TBR=dtrainor@chromium.org
==========

[gone, 2016-06-30 20:08:34]

OWNERs lgtm

[Changwan Ryu, 2016-06-30 22:27:27]

The CQ bit was checked by changwan@chromium.org

[commit-bot: I haz the power, 2016-06-30 22:27:47]

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-06-30 23:08:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-30 23:08:39]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 23:11:40]

Description was changed from

==========
Stop hiding soft input from CompositorViewHolder

CompositorViewHolder hides keyboard on switching tabs. We found that it
is causing a StrictMode violation error in KK (due to a flaw in Android
framework).

But CVH removes ContentView, which eventually calls
ImeAdapter#onViewFocusChanged(false) and then we hide the keyboard anyways.

Therefore, removing the redundant code from CVH and adding a test to
future-proof the behavior.

BUG=623256
TBR=dtrainor@chromium.org
==========

to

==========
Stop hiding soft input from CompositorViewHolder

CompositorViewHolder hides keyboard on switching tabs. We found that it
is causing a StrictMode violation error in KK (due to a flaw in Android
framework).

But CVH removes ContentView, which eventually calls
ImeAdapter#onViewFocusChanged(false) and then we hide the keyboard anyways.

Therefore, removing the redundant code from CVH and adding a test to
future-proof the behavior.

BUG=623256
TBR=dtrainor@chromium.org

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

[commit-bot: I haz the power, 2016-06-30 23:11:41]

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