Move touchscreen_util.cc/h from ash to ui.

Move touchscreen_util.cc/h from ash to ui.

AssociatedTouchscreens() is used by DisplayChangeObserver/DisplayManager
and needed for mustash. Remove the minimal ash dependencies in the
tests, they weren't actually needed, and move the tests too.

BUG=667344
Committed: https://crrev.com/3ce955f57b5440e5f75424beb896a4d8c04ce09d
Cr-Commit-Position: refs/heads/master@{#434210}

[kylechar, 2016-11-21 22:07:50]

Description was changed from

==========
Move touchscreen_util.cc/h from ash to ui.

This is used by DisplayChangeObserver/DisplayManager to associated
displays with touchscreen devices.

BUG=667344
==========

to

==========
Move touchscreen_util.cc/h from ash to ui.

AssociatedTouchscreens() is used by DisplayChangeObserver/DisplayManager
and needed for mustash. Remove the minimal ash dependencies in the
tests, they weren't actually needed, and move the tests too.

BUG=667344
==========

[kylechar, 2016-11-21 22:08:41]

kylechar@chromium.org changed reviewers:
+ oshima@chromium.org, sadrul@chromium.org

[kylechar, 2016-11-21 22:08:41]

+oshima for ash/display* and ui/display/* changes.
+sadrul for ui/display/manager/DEPS

[sadrul, 2016-11-22 02:03:05]

https://codereview.chromium.org/2520143002/diff/1/ui/display/BUILD.gn
File ui/display/BUILD.gn (right):

https://codereview.chromium.org/2520143002/diff/1/ui/display/BUILD.gn#newcode87
ui/display/BUILD.gn:87: "//ui/events/devices",
Interesting. This doesn't introduce a cyclic dependency?

[sadrul, 2016-11-22 02:29:56]

The CQ bit was checked by sadrul@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-22 02:30:46]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sadrul, 2016-11-22 03:00:02]

lgtm

[commit-bot: I haz the power, 2016-11-22 03:13:37]

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

[commit-bot: I haz the power, 2016-11-22 03:13:38]

Dry run: Try jobs failed on following builders:
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[kylechar, 2016-11-22 13:32:36]

https://codereview.chromium.org/2520143002/diff/1/ui/display/BUILD.gn
File ui/display/BUILD.gn (right):

https://codereview.chromium.org/2520143002/diff/1/ui/display/BUILD.gn#newcode87
ui/display/BUILD.gn:87: "//ui/events/devices",
On 2016/11/22 02:03:05, sadrul wrote:
> Interesting. This doesn't introduce a cyclic dependency?

Not after crrev.com/2519993002!

[oshima, 2016-11-22 21:48:32]

lgtm

[kylechar, 2016-11-23 14:43:10]

Thanks!

[kylechar, 2016-11-23 14:45:37]

The CQ bit was checked by kylechar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-23 14:45:50]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-23 16:11:55]

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

[commit-bot: I haz the power, 2016-11-23 16:11:56]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kylechar, 2016-11-23 16:14:50]

The CQ bit was checked by kylechar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-23 16:15:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-23 18:29:00]

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

[commit-bot: I haz the power, 2016-11-23 18:29:02]

Dry run: This issue passed the CQ dry run.

[kylechar, 2016-11-23 18:29:25]

The CQ bit was checked by kylechar@chromium.org

[kylechar, 2016-11-23 18:29:26]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org,
oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/2520143002/#ps40001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-11-23 18:29:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-23 18:39:48]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1479925765971170,
"parent_rev": "2624e59169353ba129915f486a24e24fd4dba705", "commit_rev":
"2ad452df3e1ebecdc04b8e0c61e8770058866747"}

[commit-bot: I haz the power, 2016-11-23 18:40:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-23 18:45:52]

Description was changed from

==========
Move touchscreen_util.cc/h from ash to ui.

AssociatedTouchscreens() is used by DisplayChangeObserver/DisplayManager
and needed for mustash. Remove the minimal ash dependencies in the
tests, they weren't actually needed, and move the tests too.

BUG=667344
==========

to

==========
Move touchscreen_util.cc/h from ash to ui.

AssociatedTouchscreens() is used by DisplayChangeObserver/DisplayManager
and needed for mustash. Remove the minimal ash dependencies in the
tests, they weren't actually needed, and move the tests too.

BUG=667344

Committed: https://crrev.com/3ce955f57b5440e5f75424beb896a4d8c04ce09d
Cr-Commit-Position: refs/heads/master@{#434210}
==========

[commit-bot: I haz the power, 2016-11-23 18:45:53]

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