Correct scale for touch radius & scroll offset ordinal

Correct scale for touch radius & scroll offset ordinal    

In https://codereview.chromium.org/12983010/

reversed_root_transform is passed into event::UpdateForRootTransform.

Then for TouchEvent::UpdateForRootTransform(), the radius_x/y now
should times the inverted device scale factor instead of being
divided by it. The same in ScrollEvent::UpdateForRootTransform() for
scroll offset ordinal.

BUG=chromium:226169
TEST=On Link, visit http://www.rbyers.net/paint.html, make sure
     touch radius has normal range.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192490

[sadrul, 2013-04-03 23:09:04]

https://codereview.chromium.org/13594009/diff/1/ui/base/events/event.cc
File ui/base/events/event.cc (right):

https://codereview.chromium.org/13594009/diff/1/ui/base/events/event.cc#newco...
ui/base/events/event.cc:700: x_offset_ordinal_ /= decomp.scale[0];
Please fix these too?

[Yufeng Shen (Slow to review), 2013-04-03 23:19:15]

https://codereview.chromium.org/13594009/diff/1/ui/base/events/event.cc
File ui/base/events/event.cc (right):

https://codereview.chromium.org/13594009/diff/1/ui/base/events/event.cc#newco...
ui/base/events/event.cc:700: x_offset_ordinal_ /= decomp.scale[0];
On 2013/04/03 23:09:04, sadrul wrote:
> Please fix these too?

Done.

[sadrul, 2013-04-03 23:25:32]

Awesome. Thanks! Mind updating the CL description too?

LGTM (FWIW)

[oshima, 2013-04-03 23:27:26]

lgtm

Sorry for not updating them, and thank you for fixing!

[oshima, 2013-04-03 23:29:25]

On 2013/04/03 23:27:26, oshima wrote:
> lgtm
> 
> Sorry for not updating them, and thank you for fixing!

Actually, can you add test for this? It shouldn't be hard
if you do it in ash/.

[Yufeng Shen (Slow to review), 2013-04-04 17:17:53]

tests added. PTAL

[sadrul, 2013-04-04 17:21:06]

Please make sure to update the CL description before landing.

[Yufeng Shen (Slow to review), 2013-04-04 17:46:14]

CL title and message updated.

[oshima, 2013-04-04 17:57:08]

lgtm with nit

https://codereview.chromium.org/13594009/diff/15002/ash/display/display_contr...
File ash/display/display_controller_unittest.cc (right):

https://codereview.chromium.org/13594009/diff/15002/ash/display/display_contr...
ash/display/display_controller_unittest.cc:125: if (target->name() !=
"DesktopBackgroundView")
since this is now used in 3 places, can you define constant in anonymous
namespace?

[Yufeng Shen (Slow to review), 2013-04-04 18:13:12]

add sky@ for OWNERS

https://codereview.chromium.org/13594009/diff/15002/ash/display/display_contr...
File ash/display/display_controller_unittest.cc (right):

https://codereview.chromium.org/13594009/diff/15002/ash/display/display_contr...
ash/display/display_controller_unittest.cc:125: if (target->name() !=
"DesktopBackgroundView")
On 2013/04/04 17:57:08, oshima wrote:
> since this is now used in 3 places, can you define constant in anonymous
> namespace?

Done.

[sky, 2013-04-04 20:21:31]

LGTM

[commit-bot: I haz the power, 2013-04-04 20:52:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/miletus@chromium.org/13594009/21001

[commit-bot: I haz the power, 2013-04-04 21:21:19]

Retried try job too often on win_x64_rel for step(s) compile
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_x64_re...

[commit-bot: I haz the power, 2013-04-04 22:44:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/miletus@chromium.org/13594009/21001

[commit-bot: I haz the power, 2013-04-05 04:02:38]

Change committed as 192490