Disable core keyboard device as well when entering touchview.

Disable core keyboard device as well when entering touchview.

Repeated keypress events seem to be coming from the core keyboard device for the internal keyboard. Amazingly, they seem to come from the correct device (for my external keyboard at least) so this patch may be all we need.

BUG=402868
TEST=Flip open device past 180 to enter touchview. Hold a key on keyboard. Key is not typed. External keyboard still works while in touchview.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=289524

[flackr, 2014-08-12 22:08:27]

Please take a look. It seems a little hacky but this seems to do exactly what we
want on device - prevent all internal keyboard events in touchview while still
allowing single and repeated external keyboard events. I have no idea why
repeated key events come from the core device only on the internal keyboard.

[sadrul, 2014-08-13 02:31:09]

+kpschoedel@ mind reviewing? :)

[flackr, 2014-08-13 18:05:02]

ping

[kpschoedel, 2014-08-13 18:08:55]

Sorry, missed the message. lgtm.

[flackr, 2014-08-13 18:10:42]

The CQ bit was checked by flackr@chromium.org

[commit-bot: I haz the power, 2014-08-13 18:11:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/flackr@chromium.org/460093003/1

[commit-bot: I haz the power, 2014-08-13 18:11:57]

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

[commit-bot: I haz the power, 2014-08-13 18:11:58]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[flackr, 2014-08-13 18:54:40]

On 2014/08/13 18:11:58, I haz the power (commit-bot) wrote:
> No LGTM from a valid reviewer yet. Only full committers are accepted.
> Even if an LGTM may have been provided, it was from a non-committer or
> a provisional committer, _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

sadrul, can you review as well?

[sadrul, 2014-08-13 18:58:11]

lgtm

[flackr, 2014-08-13 18:58:46]

The CQ bit was checked by flackr@chromium.org

[commit-bot: I haz the power, 2014-08-13 19:02:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/flackr@chromium.org/460093003/1

[commit-bot: I haz the power, 2014-08-14 07:36:27]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)

[commit-bot: I haz the power, 2014-08-14 11:21:15]

Committed patchset #1 (1) as 289524