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

Issue 2798933004: Restore mappings for older Xbox One S Bluetooth gamepads on OSX/Linux (Closed)

Created:
3 years, 8 months ago by mattreynolds
Modified:
3 years, 4 months ago
Reviewers:
bajones
CC:
chromium-reviews, mac-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Restore mappings for older Xbox One S Bluetooth gamepads on OSX/Linux Microsoft released a firmware update for Xbox One S gamepads that changed the button and axis assignments when connected over Bluetooth, requiring a new mapping in Chrome. In a previous CL, the mappings for the old firmware were removed under the assumption that soon all gamepads would use the new firmware mappings. However, this was incorrect as the new mappings are not exposed on all platforms. Furthermore, Xbox One S gamepads are still shipping with the old FW and users may not have the means to update. On OSX, gamepads with the old FW use device ID 045e:02e0 while those with the new FW use 045e:02fd. This CL re-adds the mapping for the old FW that was previously replaced. The new mapping will still be applied to gamepads with the new FW. On Linux, gamepads with either FW use device ID 045e:02e0 and need the old mapping which was previously replaced. This CL re-adds the mapping for the old FW. The new FW mapping is retained for now, but may be removed in the future as it should now be unused. On Windows, Xbox One S gamepads are handled by the OS and are exposed to Chrome as Xbox 360 compatible gamepads. No extra work is required to support the old FW. On Android, the default mapping is correct for the new FW but a custom mapping is required to support the old FW. This will be addressed in a future CL. BUG=705506 Review-Url: https://codereview.chromium.org/2798933004 Cr-Commit-Position: refs/heads/master@{#490505} Committed: https://chromium.googlesource.com/chromium/src/+/0901a39d9de30d0a98d483a79b7963cbcf1cfd90

Patch Set 1 #

Patch Set 2 : re-add old firmware mappings #

Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -4 lines) Patch
M device/gamepad/gamepad_standard_mappings_linux.cc View 1 2 chunks +28 lines, -2 lines 0 comments Download
M device/gamepad/gamepad_standard_mappings_mac.mm View 1 2 chunks +25 lines, -2 lines 0 comments Download

Messages

Total messages: 9 (5 generated)
mattreynolds
Hi Brandon, PTAL. This fixes Xbox One S gamepads on Linux and re-enables the mapping ...
3 years, 4 months ago (2017-07-28 01:42:12 UTC) #3
bajones
On 2017/07/28 01:42:12, mattreynolds wrote: > Hi Brandon, PTAL. This fixes Xbox One S gamepads ...
3 years, 4 months ago (2017-07-28 19:19:03 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2798933004/20001
3 years, 4 months ago (2017-07-28 19:32:04 UTC) #6
commit-bot: I haz the power
3 years, 4 months ago (2017-07-28 20:56:11 UTC) #9
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/0901a39d9de30d0a98d483a79b79...

Powered by Google App Engine
This is Rietveld 408576698