Make Touch.radiusX/Y float and make Touch.webkit* aliases in the IDL

Make Touch.radiusX/Y float and make Touch.webkit* aliases in the IDL

BUG=460099
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=195637

[philipj_slow, 2015-05-20 15:27:21]

philipj@opera.com changed reviewers:
+ rbyers@chromium.org

[philipj_slow, 2015-05-20 15:27:22]

PTAL. This doesn't change any observable behavior, it just eliminates a few
float->double->float conversions and makes it more obvious that the prefixed
attributes are pure aliases.

[Rick Byers, 2015-05-20 15:33:41]

LGTM, thanks

https://codereview.chromium.org/1149563003/diff/1/Source/core/dom/Touch.idl
File Source/core/dom/Touch.idl (right):

https://codereview.chromium.org/1149563003/diff/1/Source/core/dom/Touch.idl#n...
Source/core/dom/Touch.idl:45: // readonly attribute float rotationAngle;
FYI I think we can unprefixing this too and deprecated all the prefix versions -
see crbug.com/460099.

[Rick Byers, 2015-05-20 15:34:06]

You may want to mention BUG=460099 since it's really part of that larger issue.

[philipj_slow, 2015-05-20 18:08:37]

https://codereview.chromium.org/1149563003/diff/1/Source/core/dom/Touch.idl
File Source/core/dom/Touch.idl (right):

https://codereview.chromium.org/1149563003/diff/1/Source/core/dom/Touch.idl#n...
Source/core/dom/Touch.idl:45: // readonly attribute float rotationAngle;
On 2015/05/20 15:33:41, Rick Byers wrote:
> FYI I think we can unprefixing this too and deprecated all the prefix versions
-
> see crbug.com/460099.

That looks very quick to do indeed, and I'd be very quick to LGTM both shipping
rotationAngle and deprecating the prefixed attributes :)

[philipj_slow, 2015-05-20 18:08:43]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-05-20 18:09:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1149563003/1

[Rick Byers, 2015-05-20 18:59:09]

rbyers@chromium.org changed reviewers:
+ mustaq@chromium.org

[Rick Byers, 2015-05-20 18:59:48]

On 2015/05/20 18:08:37, philipj_UTC2 wrote:
> https://codereview.chromium.org/1149563003/diff/1/Source/core/dom/Touch.idl
> File Source/core/dom/Touch.idl (right):
> 
>
https://codereview.chromium.org/1149563003/diff/1/Source/core/dom/Touch.idl#n...
> Source/core/dom/Touch.idl:45: // readonly attribute float rotationAngle;
> On 2015/05/20 15:33:41, Rick Byers wrote:
> > FYI I think we can unprefixing this too and deprecated all the prefix
versions
> -
> > see crbug.com/460099.
> 
> That looks very quick to do indeed, and I'd be very quick to LGTM both
shipping
> rotationAngle and deprecating the prefixed attributes :)

Yep, mustaq@ was planning on putting up a CL for that at some point but hasn't
had a chance.  Fine with me if you want to.

[commit-bot: I haz the power, 2015-05-20 20:32:12]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=195637