Implement rudimentary touchscreen support in mus.

Implement rudimentary touchscreen support in mus.

Add TouchController class that implements rudimentary touchscreen
support in mus. When there is 1 display and 1 touchscreen input-device
it matches them up and populates the relevant transform.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.

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

[kylechar, 2016-06-21 17:46:49]

Description was changed from

==========
Implement rudimentary touchscreen support to mus.

Add TouchController class that adds rudimentary touchscreen support to
mus. When there is 1 display and 1 touchscreen input device it matches
them up and populates the relevant transforms.

BUG=601074
TESTED=Touch works on Pixel.
==========

to

==========
Implement rudimentary touchscreen support to mus.

Add TouchController class that adds rudimentary touchscreen support to
mus. When there is 1 display and 1 touchscreen input device it matches
them up and populates the relevant transforms.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.
==========

[kylechar, 2016-06-21 17:49:18]

Description was changed from

==========
Implement rudimentary touchscreen support to mus.

Add TouchController class that adds rudimentary touchscreen support to
mus. When there is 1 display and 1 touchscreen input device it matches
them up and populates the relevant transforms.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.
==========

to

==========
Implement rudimentary touchscreen support in mus.

Add TouchController class that implements rudimentary touchscreen
support in mus. When there is 1 display and 1 touchscreen input-device
it matches them up and populates the relevant transform.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.
==========

[kylechar, 2016-06-21 17:49:29]

kylechar@chromium.org changed reviewers:
+ sky@chromium.org

[kylechar, 2016-06-21 17:49:42]

Description was changed from

==========
Implement rudimentary touchscreen support in mus.

Add TouchController class that implements rudimentary touchscreen
support in mus. When there is 1 display and 1 touchscreen input-device
it matches them up and populates the relevant transform.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.
==========

to

==========
Implement rudimentary touchscreen support in mus.

Add TouchController class that implements rudimentary touchscreen
support in mus. When there is 1 display and 1 touchscreen input-device
it matches them up and populates the relevant transform.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.
==========

[sky, 2016-06-21 20:09:22]

LGTM

https://codereview.chromium.org/2081253002/diff/1/components/mus/ws/touch_con...
File components/mus/ws/touch_controller.cc (right):

https://codereview.chromium.org/2081253002/diff/1/components/mus/ws/touch_con...
components/mus/ws/touch_controller.cc:80: if (touch_displays.size() == 1 &&
touch_devices.size() == 1) {
Please add a TODO for the multi-display case.

[kylechar, 2016-06-22 12:47:45]

Thanks sky!

https://codereview.chromium.org/2081253002/diff/1/components/mus/ws/touch_con...
File components/mus/ws/touch_controller.cc (right):

https://codereview.chromium.org/2081253002/diff/1/components/mus/ws/touch_con...
components/mus/ws/touch_controller.cc:80: if (touch_displays.size() == 1 &&
touch_devices.size() == 1) {
On 2016/06/21 20:09:22, sky wrote:
> Please add a TODO for the multi-display case.

Done.

[kylechar, 2016-06-22 12:47:50]

The CQ bit was checked by kylechar@chromium.org

[kylechar, 2016-06-22 12:47:50]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2081253002/#ps20001
(title: "Add TODO.")

[commit-bot: I haz the power, 2016-06-22 12:48:07]

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

[commit-bot: I haz the power, 2016-06-22 13:15:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-22 13:15:39]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[kylechar, 2016-06-22 13:37:13]

The CQ bit was checked by kylechar@chromium.org

[kylechar, 2016-06-22 13:37:13]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2081253002/#ps40001
(title: "Add missing pure virtual.")

[commit-bot: I haz the power, 2016-06-22 13:37:27]

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

[commit-bot: I haz the power, 2016-06-22 15:33:46]

Description was changed from

==========
Implement rudimentary touchscreen support in mus.

Add TouchController class that implements rudimentary touchscreen
support in mus. When there is 1 display and 1 touchscreen input-device
it matches them up and populates the relevant transform.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.
==========

to

==========
Implement rudimentary touchscreen support in mus.

Add TouchController class that implements rudimentary touchscreen
support in mus. When there is 1 display and 1 touchscreen input-device
it matches them up and populates the relevant transform.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.
==========

[commit-bot: I haz the power, 2016-06-22 15:33:47]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-22 15:35:34]

Description was changed from

==========
Implement rudimentary touchscreen support in mus.

Add TouchController class that implements rudimentary touchscreen
support in mus. When there is 1 display and 1 touchscreen input-device
it matches them up and populates the relevant transform.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.
==========

to

==========
Implement rudimentary touchscreen support in mus.

Add TouchController class that implements rudimentary touchscreen
support in mus. When there is 1 display and 1 touchscreen input-device
it matches them up and populates the relevant transform.

Small modifications to mus::Display and mus::PlatformDisplay to store
the display id and plumb display id and display size back out.

Change PlatformScreen implementation to call
NativeDisplayDelegate::GetDisplays() after a new display is added. If
this doesn't happen the list of DisplaySnapshots stored in Ozone isn't
updated and incomplete information is included with touch events.

BUG=601074
TEST=Touch works on Pixel.

Committed: https://crrev.com/c97cdcc4d02b8823790798f2e812d933a4045a03
Cr-Commit-Position: refs/heads/master@{#401285}
==========

[commit-bot: I haz the power, 2016-06-22 15:35:35]

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