Fix projection touch HUD in RTL

Fix projection touch HUD in RTL

The projection touch HUD was shown horizontally mirrored when in
right-to-left mode. Now, fixed.

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

[sadrul, 2013-07-15 21:44:06]

https://codereview.chromium.org/19271002/diff/1/ash/touch/touch_hud_projectio...
File ash/touch/touch_hud_projection.cc (left):

https://codereview.chromium.org/19271002/diff/1/ash/touch/touch_hud_projectio...
ash/touch/touch_hud_projection.cc:66: SetX(touch.root_location().x() -
kPointRadius - 1);
Can you use:
  SetX(parent()->GetMirroredXInView(touch.root_location().x()) - kPointRadius -
1);
instead?

[mohsen, 2013-07-15 22:10:52]

https://codereview.chromium.org/19271002/diff/1/ash/touch/touch_hud_projectio...
File ash/touch/touch_hud_projection.cc (left):

https://codereview.chromium.org/19271002/diff/1/ash/touch/touch_hud_projectio...
ash/touch/touch_hud_projection.cc:66: SetX(touch.root_location().x() -
kPointRadius - 1);
On 2013/07/15 21:44:06, sadrul wrote:
> Can you use:
>   SetX(parent()->GetMirroredXInView(touch.root_location().x()) - kPointRadius
-
> 1);
> instead?

Yes, much better. Done.

[sadrul, 2013-07-15 22:15:13]

LGTM

[mohsen, 2013-07-15 22:17:47]

sky@: Please take a look...

[sky, 2013-07-15 22:34:46]

LGTM

[commit-bot: I haz the power, 2013-07-15 22:35:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mohsen@chromium.org/19271002/5001

[commit-bot: I haz the power, 2013-07-16 02:29:21]

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

[commit-bot: I haz the power, 2013-07-16 04:04:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mohsen@chromium.org/19271002/5001

[commit-bot: I haz the power, 2013-07-16 05:01:22]

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

[commit-bot: I haz the power, 2013-07-16 13:58:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mohsen@chromium.org/19271002/5001

[commit-bot: I haz the power, 2013-07-16 13:59:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mohsen@chromium.org/19271002/5001

[commit-bot: I haz the power, 2013-07-16 15:08:40]

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

[commit-bot: I haz the power, 2013-07-16 17:51:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mohsen@chromium.org/19271002/5001

[commit-bot: I haz the power, 2013-07-16 19:28:43]

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

[commit-bot: I haz the power, 2013-07-17 04:46:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mohsen@chromium.org/19271002/5001

[commit-bot: I haz the power, 2013-07-17 07:00:18]

Change committed as 211989