Added Touched attribute to GamepadButton

Added Touched attribute to GamepadButton

BUG=628417
Committed: https://crrev.com/8392d79b4a34c1f9dd6dbe9fa9875c37c66e46c5
Cr-Commit-Position: refs/heads/master@{#416564}

[bajones, 2016-07-15 05:28:43]

The CQ bit was checked by bajones@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-15 05:29:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-15 06:46:58]

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

[commit-bot: I haz the power, 2016-07-15 06:46:59]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[bajones, 2016-07-15 23:03:07]

bajones@chromium.org changed reviewers:
+ bbudge@chromium.org, mkwst@chromium.org, scottmg@chromium.org

[bajones, 2016-07-15 23:03:07]

This attribute was recently added to the gamepad spec.

mkwst@: Please review public/ and take a look at modules/ if you don't mind.

bbudge@: Could you take a look at the ppapi/ change? It's pretty small.

scottmg@: device/gamepad/ if you're around?

[bbudge, 2016-07-16 00:18:19]

ppapi lgtm

[Mike West, 2016-07-18 08:34:30]

On 2016/07/16 at 00:18:19, bbudge wrote:
> ppapi lgtm

Code change LGTM.

Is this part of an existing intent to implement/ship? You'll need sign-off for
the API change visible in `//LayoutTests/virtual/stable/webexposed` from someone
in `//API_OWNERS`, so it would be helpful to link to the intent thread in the CL
description.

[bajones, 2016-07-18 15:15:00]

On 2016/07/18 08:34:30, Mike West wrote:
> Is this part of an existing intent to implement/ship? You'll need sign-off for
> the API change visible in `//LayoutTests/virtual/stable/webexposed` from
someone
> in `//API_OWNERS`, so it would be helpful to link to the intent thread in the
CL
> description.

I had a conversation about this on Friday, because I wasn't clear if additions
to an existing API like this required an intent to implement. The webexposed bit
is a fairly nice guideline. Thanks!

I'll get out an intent to implement today and make sure it's approved before I
land this.

[Mike West, 2016-07-20 07:32:20]

On 2016/07/18 at 15:15:00, bajones wrote:
> On 2016/07/18 08:34:30, Mike West wrote:
> > Is this part of an existing intent to implement/ship? You'll need sign-off
for
> > the API change visible in `//LayoutTests/virtual/stable/webexposed` from
someone
> > in `//API_OWNERS`, so it would be helpful to link to the intent thread in
the CL
> > description.
> 
> I had a conversation about this on Friday, because I wasn't clear if additions
to an existing API like this required an intent to implement. The webexposed bit
is a fairly nice guideline. Thanks!
> 
> I'll get out an intent to implement today and make sure it's approved before I
land this.

Thank you!

[bajones, 2016-09-05 21:16:56]

The CQ bit was checked by bajones@chromium.org

[bajones, 2016-09-05 21:16:57]

The patchset sent to the CQ was uploaded after l-g-t-m from bbudge@chromium.org,
mkwst@chromium.org
Link to the patchset: https://codereview.chromium.org/2150173002/#ps120001
(title: "Actually remove unnecessary test. Not sure what happened last time.")

[commit-bot: I haz the power, 2016-09-05 21:17:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-05 22:35:26]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-09-05 22:37:50]

Description was changed from

==========
Added Touched attribute to GamepadButton

BUG=628417
==========

to

==========
Added Touched attribute to GamepadButton

BUG=628417

Committed: https://crrev.com/8392d79b4a34c1f9dd6dbe9fa9875c37c66e46c5
Cr-Commit-Position: refs/heads/master@{#416564}
==========

[commit-bot: I haz the power, 2016-09-05 22:37:51]

Patchset 7 (id:??) landed as
https://crrev.com/8392d79b4a34c1f9dd6dbe9fa9875c37c66e46c5
Cr-Commit-Position: refs/heads/master@{#416564}