Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(132)

Issue 1950883002: Fix button 1 being stuck down on certain Logitech gamepads on Windows (Closed)

Created:
4 years, 7 months ago by cgutman
Modified:
4 years, 7 months ago
Reviewers:
bajones
CC:
chromium-reviews, darin-cc_chromium.org, jam
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Fix button 1 being stuck down on certain Logitech gamepads on Windows There are HID elements in these gamepads classified as buttons which don't use the usage page value for buttons. These elements don't actually correspond to real buttons on the controller, and are stuck in the pressed position. On OSX, we skip these "fake" buttons, but we didn't on Windows. As a result, a HID element would overlap with button 1 and make it appear to be constantly pressed. R=bajones@chromium.org BUG=600230 Committed: https://crrev.com/a8d9d22865f445b8f6f2b9814fb919b8008fbc5c Cr-Commit-Position: refs/heads/master@{#391616}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -2 lines) Patch
M content/browser/gamepad/raw_input_data_fetcher_win.cc View 3 chunks +5 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
cgutman
I ran into this issue a few weeks ago and finally took a look. A ...
4 years, 7 months ago (2016-05-04 03:07:40 UTC) #1
bajones
Works great on the gamepads I've seen this behavior from in the past! Thank you ...
4 years, 7 months ago (2016-05-04 16:51:49 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1950883002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1950883002/1
4 years, 7 months ago (2016-05-04 16:52:13 UTC) #4
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 7 months ago (2016-05-04 20:08:37 UTC) #5
commit-bot: I haz the power
4 years, 7 months ago (2016-05-04 20:10:16 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/a8d9d22865f445b8f6f2b9814fb919b8008fbc5c
Cr-Commit-Position: refs/heads/master@{#391616}

Powered by Google App Engine
This is Rietveld 408576698