Round Touch attributes back to int for backwards compatibility

Round Touch attributes back to int for backwards compatibility

Most of the attributes in the Touch interface (clientX/Y, screenX/Y, pageX/Y, webkitRadiusX/Y) have changed to use double instead of int.  Since we are generating dart:html from old idl, this immediately breaks for us as soon as these are used.  This CL wraps calls to these with a round to retain the old API.

BUG=http://dartbug.com/20587
R=blois@google.com

Committed: https://code.google.com/p/dart/source/detail?r=39426

[blois, 2014-08-20 20:11:44]

https://codereview.chromium.org/493843003/diff/1/tools/dom/templates/html/imp...
File tools/dom/templates/html/impl/impl_Touch.darttemplate (right):

https://codereview.chromium.org/493843003/diff/1/tools/dom/templates/html/imp...
tools/dom/templates/html/impl/impl_Touch.darttemplate:14: int get __clientX =>
JS('num', '#.clientX', this).round();
Why not suppress clientX etc completely, since it has a custom implementation?
Then can avoid the __ prefix.

[Leaf, 2014-08-20 20:21:02]

https://codereview.chromium.org/493843003/diff/1/tools/dom/templates/html/imp...
File tools/dom/templates/html/impl/impl_Touch.darttemplate (right):

https://codereview.chromium.org/493843003/diff/1/tools/dom/templates/html/imp...
tools/dom/templates/html/impl/impl_Touch.darttemplate:14: int get __clientX =>
JS('num', '#.clientX', this).round();
Unfortunately we're still generating dart:_blink from the same scripts, and I
need to dart:_blink entry points.  I don't know of good existing mechanism to
suppress something from dart:html and not dart:_blink (suggestions welcome if
you know of a way to do this).  I could implement something to do this, but
since we're moving blink out of those scripts Real Soon Now, and since I'd like
to get this fix in 1.6, I'd prefer to just use this approach for now.

On 2014/08/20 20:11:44, blois wrote:
> Why not suppress clientX etc completely, since it has a custom implementation?
> Then can avoid the __ prefix.

[blois, 2014-08-20 20:23:09]

lgtm

[Leaf, 2014-08-20 20:29:43]

Committed patchset #2 manually as 39426 (presubmit successful).