Add new XBox 360 Wireless USB ID.

Add new XBox 360 Wireless USB ID.

Newer 360 controllers have the product ID of 0719. Adding this to
chromium enables standard mapping to be performed. Without it, buttons
will behave differently than defined in the GamePad API standard
mapping.

Committed: https://crrev.com/21a97db383613682082fb9b1c5860888cc8b2cfc
Cr-Commit-Position: refs/heads/master@{#356244}

[mjbauer, 2015-10-24 20:49:06]

mjbauer95@gmail.com changed reviewers:
+ scottmg@chromium.org

[mjbauer, 2015-10-24 20:49:06]

This is a fairly simple change. Just adds another product ID to the list for Mac
and Linux standard mappings.

[scottmg, 2015-10-25 02:40:07]

lgtm

https://codereview.chromium.org/1418513011/diff/1/content/browser/gamepad/gam...
File content/browser/gamepad/gamepad_standard_mappings_linux.cc (right):

https://codereview.chromium.org/1418513011/diff/1/content/browser/gamepad/gam...
content/browser/gamepad/gamepad_standard_mappings_linux.cc:236: {"045e", "028f",
MapperXInputStyleGamepad},  // Xbox 360 Wireless Controller
Please either re-wrap this, or alternatively drop "Controller" to fit 80 col.

You could also dedent all the initializers by 2.

[mjbauer, 2015-10-25 20:19:01]

On 2015/10/25 02:40:07, scottmg wrote:
> lgtm
> 
>
https://codereview.chromium.org/1418513011/diff/1/content/browser/gamepad/gam...
> File content/browser/gamepad/gamepad_standard_mappings_linux.cc (right):
> 
>
https://codereview.chromium.org/1418513011/diff/1/content/browser/gamepad/gam...
> content/browser/gamepad/gamepad_standard_mappings_linux.cc:236: {"045e",
"028f",
> MapperXInputStyleGamepad},  // Xbox 360 Wireless Controller
> Please either re-wrap this, or alternatively drop "Controller" to fit 80 col.
> 
> You could also dedent all the initializers by 2.

Okay take a look at the revised changes. Can squash the two commits if needed.

[mjbauer, 2015-10-25 22:05:44]

The CQ bit was checked by mjbauer95@gmail.com

[mjbauer, 2015-10-25 22:05:45]

The patchset sent to the CQ was uploaded after l-g-t-m from scottmg@chromium.org
Link to the patchset: https://codereview.chromium.org/1418513011/#ps20001
(title: "fix 80 chars issue")

[mjbauer, 2015-10-25 22:05:45]

The CQ bit was unchecked by mjbauer95@gmail.com

[scottmg, 2015-10-26 16:19:29]

On 2015/10/25 20:19:01, mjbauer95 wrote:
> On 2015/10/25 02:40:07, scottmg wrote:
> > lgtm
> > 
> >
>
https://codereview.chromium.org/1418513011/diff/1/content/browser/gamepad/gam...
> > File content/browser/gamepad/gamepad_standard_mappings_linux.cc (right):
> > 
> >
>
https://codereview.chromium.org/1418513011/diff/1/content/browser/gamepad/gam...
> > content/browser/gamepad/gamepad_standard_mappings_linux.cc:236: {"045e",
> "028f",
> > MapperXInputStyleGamepad},  // Xbox 360 Wireless Controller
> > Please either re-wrap this, or alternatively drop "Controller" to fit 80
col.
> > 
> > You could also dedent all the initializers by 2.
> 
> Okay take a look at the revised changes. Can squash the two commits if needed.

LGTM. CLs are always landed "squashed", it's just handled as a patch.

[scottmg, 2015-10-26 16:19:33]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2015-10-26 16:20:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1418513011/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1418513011/20001

[commit-bot: I haz the power, 2015-10-26 16:29:01]

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

[commit-bot: I haz the power, 2015-10-26 16:29:02]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[scottmg, 2015-10-26 16:30:07]

On 2015/10/26 16:29:02, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

"mjbauer95@gmail.com is not in AUTHORS file. If you are a new contributor,
please visit
http://www.chromium.org/developers/contributing-code and read the "Legal"
section
If you are a chromite, verify the contributor signed the CLA."

[mjbauer, 2015-10-27 06:01:24]

The CQ bit was checked by mjbauer95@gmail.com

[mjbauer, 2015-10-27 06:01:24]

The patchset sent to the CQ was uploaded after l-g-t-m from scottmg@chromium.org
Link to the patchset: https://codereview.chromium.org/1418513011/#ps40001
(title: "Add to AUTHORS")

[commit-bot: I haz the power, 2015-10-27 06:01:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1418513011/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1418513011/40001

[mjbauer, 2015-10-27 06:01:35]

On 2015/10/26 16:30:07, scottmg wrote:
> On 2015/10/26 16:29:02, commit-bot: I haz the power wrote:
> > Try jobs failed on following builders:
> >   chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
> >
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
> 
> mailto:"mjbauer95@gmail.com is not in AUTHORS file. If you are a new
contributor,
> please visit
> http://www.chromium.org/developers/contributing-code and read the "Legal"
> section
> If you are a chromite, verify the contributor signed the CLA."

Ok, think I've got it.

[commit-bot: I haz the power, 2015-10-27 06:35:37]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-10-27 06:36:25]

Patchset 3 (id:??) landed as
https://crrev.com/21a97db383613682082fb9b1c5860888cc8b2cfc
Cr-Commit-Position: refs/heads/master@{#356244}

[scottmg, 2015-10-27 16:29:31]

Thanks!