Commit 73a70e0a authored by Harry Cutts's avatar Harry Cutts Committed by Commit Bot

ozone: evdev: block various mice from being keyboards

These mice's HID descriptors claim that they have full keyboards, when
actually they're just normal mice. On tablets, this meant that the
on-screen keyboard didn't show up with one of the mice connected,
preventing text from being entered. Adding them to the blocklist fixes
the issue.

Tested with all the Microsoft and Logitech mice except the M500s and
M185/M187. (The others had been previously identified by Works with
Chromebook testing.)

(Yes, the mouse is actually just called "Microsoft Bluetooth Mouse", I'm
not just being vague:
https://microsoft.com/en-us/p/microsoft-bluetooth-mouse/8qbx2w3xsp2h )

      appears.

Bug: none
Test: Connect mouse, touch a text box, and check the on-screen keyboard
Change-Id: I7913aebf685334dedafaef90943e9e62d630cde8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2298327Reviewed-by: default avatarMichael Spang <spang@chromium.org>
Commit-Queue: Harry Cutts <hcutts@chromium.org>
Cr-Commit-Position: refs/heads/master@{#789127}
parent 8b0154bd
......@@ -32,7 +32,21 @@ constexpr struct {
uint16_t vendor;
uint16_t product_id;
} kKeyboardBlocklist[] = {
{0x045e, 0x0b05}, // Xbox One Elite Series 2 gamepad
{0x03f0, 0xa407}, // HP X4000 Wireless Mouse
{0x045e, 0x0745}, // Microsoft Wireless Mobile Mouse 6000
{0x045e, 0x0821}, // Microsoft Surface Precision Mouse
{0x045e, 0x082a}, // Microsoft Pro IntelliMouse
{0x045e, 0x082f}, // Microsoft Bluetooth Mouse
{0x045e, 0x0b05}, // Xbox One Elite Series 2 gamepad
{0x046d, 0x4069}, // Logitech MX Master 2S (Unifying)
{0x046d, 0xb016}, // Logitech M535
{0x046d, 0xb019}, // Logitech MX Master 2S (Bluetooth)
{0x046d, 0xc093}, // Logitech M500s
{0x046d, 0xc534}, // Logitech M185/M187
{0x056e, 0x0134}, // Elecom Enelo IR LED Mouse 350
{0x056e, 0x0141}, // Elecom EPRIM Blue LED 5 button mouse 228
{0x056e, 0x0159}, // Elecom Blue LED Mouse 203
{0x1bcf, 0x08a0}, // Kensington Pro Fit Full-size
};
constexpr struct {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment