TouchTransformer for software mirroring mode

TouchTransformer for software mirroring mode

In software mirroring mode, there is one X root window and it should
be associated with both displays (touchscreens).

BUG=chrome-os-partner:31466
TEST=tested on Big with clone mode.

Committed: https://crrev.com/b03278a7385427ab34ed856df09c36f59f6f66ad
Cr-Commit-Position: refs/heads/master@{#294407}

[Yufeng Shen (Slow to review), 2014-09-09 21:17:50]

miletus@chromium.org changed reviewers:
+ oshima@chromium.org

[oshima, 2014-09-10 22:34:57]

https://codereview.chromium.org/557073002/diff/1/ash/touch/touch_transformer_...
File ash/touch/touch_transformer_controller.cc (right):

https://codereview.chromium.org/557073002/diff/1/ash/touch/touch_transformer_...
ash/touch/touch_transformer_controller.cc:243: display1_id, display2_id);
Does this work even if the touch screen is on external display side?
If so, how events on the external display mapped to the root window?

[Yufeng Shen (Slow to review), 2014-09-11 01:14:31]

https://codereview.chromium.org/557073002/diff/1/ash/touch/touch_transformer_...
File ash/touch/touch_transformer_controller.cc (right):

https://codereview.chromium.org/557073002/diff/1/ash/touch/touch_transformer_...
ash/touch/touch_transformer_controller.cc:243: display1_id, display2_id);
On 2014/09/10 22:34:56, oshima wrote:
> Does this work even if the touch screen is on external display side?
> If so, how events on the external display mapped to the root window?

right, in the external touch display case, the event should be mapped from
framebuffer to the source display (internal display)'s size, not the external
display's size. This is different than the true extended mode.

Fixed it in the new patchset. Tested on Big with external touchscreen.

[oshima, 2014-09-11 08:13:44]

lgtm

[Yufeng Shen (Slow to review), 2014-09-11 15:58:35]

The CQ bit was checked by miletus@chromium.org

[commit-bot: I haz the power, 2014-09-11 15:59:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/557073002/20001

[commit-bot: I haz the power, 2014-09-11 17:03:58]

Committed patchset #2 (id:20001) as 99d1c4c595c4a312e1ad05ff7fc6ef74e8af1678

[commit-bot: I haz the power, 2014-09-11 17:15:08]

Patchset 2 (id:??) landed as
https://crrev.com/b03278a7385427ab34ed856df09c36f59f6f66ad
Cr-Commit-Position: refs/heads/master@{#294407}