Split //ui/events/devices into two targets.

Split //ui/events/devices into two targets.

Split /ui/events/devices into //ui/events/devices +
//ui/events/devices/x11 to keep X11 code in a different component. Also
udpate GYP targets in a similar fashion.

Update some dependent targets and fix a few GN check problems that crop
up because GN check doesn't fully understand #if defined(USE_X11) and
includes.

Originally commited at r388472 and reverted at r388474. There were
missing deps in the GN Chrome OS X11 non-Ozone component build that I
missed. Nothing in the CQ runs that configuration but it made a
waterfall bot fail.

TBR=sky@chromium.org,sievers@chromium.org
BUG=588026
Committed: https://crrev.com/ad7148777cc4714145ef66b1b60df7e000ca9f22
Cr-Commit-Position: refs/heads/master@{#388510}

[kylechar, 2016-04-20 13:44:40]

Description was changed from

==========
Split //ui/events/devices into two targets.

Split /ui/events/devices into //ui/events/devices +
//ui/events/devices/x11 to keep X11 code in a different component. Also
udpate GYP targets in a similar fashion.

Update some dependent targets and fix a few GN check problems that crop
up because GN check doesn't fully understand #if defined(USE_X11) and
includes.

Retrying patch with additional changes. Build failures weren't caught by
CQ last time so adding the new //ui/events/devices/x11 target as a dep
in more places.

BUG=588026
==========

to

==========
Split //ui/events/devices into two targets.

Split /ui/events/devices into //ui/events/devices +
//ui/events/devices/x11 to keep X11 code in a different component. Also
udpate GYP targets in a similar fashion.

Update some dependent targets and fix a few GN check problems that crop
up because GN check doesn't fully understand #if defined(USE_X11) and
includes.

Retrying patch with additional changes after it got reverted. There were
missing deps in the GN chromeos X11 non-Ozone component build that I
missed. Nothing in the CQ runs that configuration but it made a
waterfall bot fail.

BUG=588026
==========

[kylechar, 2016-04-20 13:51:31]

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

[commit-bot: I haz the power, 2016-04-20 13:51:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1908473002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1908473002/20001

[commit-bot: I haz the power, 2016-04-20 14:54:20]

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

[commit-bot: I haz the power, 2016-04-20 14:54:21]

Dry run: This issue passed the CQ dry run.

[kylechar, 2016-04-20 15:28:24]

Description was changed from

==========
Split //ui/events/devices into two targets.

Split /ui/events/devices into //ui/events/devices +
//ui/events/devices/x11 to keep X11 code in a different component. Also
udpate GYP targets in a similar fashion.

Update some dependent targets and fix a few GN check problems that crop
up because GN check doesn't fully understand #if defined(USE_X11) and
includes.

Retrying patch with additional changes after it got reverted. There were
missing deps in the GN chromeos X11 non-Ozone component build that I
missed. Nothing in the CQ runs that configuration but it made a
waterfall bot fail.

BUG=588026
==========

to

==========
Split //ui/events/devices into two targets.

Split /ui/events/devices into //ui/events/devices +
//ui/events/devices/x11 to keep X11 code in a different component. Also
udpate GYP targets in a similar fashion.

Update some dependent targets and fix a few GN check problems that crop
up because GN check doesn't fully understand #if defined(USE_X11) and
includes.

Originally commited at r388472 and reverted at r388474. There were
missing deps in the GN Chrome OS X11 non-Ozone component build that I
missed. Nothing in the CQ runs that configuration but it made a
waterfall bot fail.

TBR=sky@chromium.org,sievers@chromium.org
BUG=588026
==========

[kylechar, 2016-04-20 15:29:27]

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

[kylechar, 2016-04-20 15:29:28]

Second try with fixed deps for GN Chrome OS X11 non-Ozone component build.

[sadrul, 2016-04-20 15:59:26]

I assume patchset-1 is the original CL that landed, and patchset-2 includes the
fixes? lgtm

[kylechar, 2016-04-20 16:03:34]

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

[kylechar, 2016-04-20 16:03:35]

Yep, that's correct. Added TBR sky@chromium.org and sievers@chromium.org for the
same parts as last time.

[kylechar, 2016-04-20 16:04:05]

The CQ bit was checked by kylechar@chromium.org

[commit-bot: I haz the power, 2016-04-20 16:04:21]

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

[commit-bot: I haz the power, 2016-04-20 16:10:01]

Committed patchset #2 (id:20001)

[sky, 2016-04-20 19:39:56]

SLGTM

[commit-bot: I haz the power, 2016-04-22 19:23:57]

Description was changed from

==========
Split //ui/events/devices into two targets.

Split /ui/events/devices into //ui/events/devices +
//ui/events/devices/x11 to keep X11 code in a different component. Also
udpate GYP targets in a similar fashion.

Update some dependent targets and fix a few GN check problems that crop
up because GN check doesn't fully understand #if defined(USE_X11) and
includes.

Originally commited at r388472 and reverted at r388474. There were
missing deps in the GN Chrome OS X11 non-Ozone component build that I
missed. Nothing in the CQ runs that configuration but it made a
waterfall bot fail.

TBR=sky@chromium.org,sievers@chromium.org
BUG=588026
==========

to

==========
Split //ui/events/devices into two targets.

Split /ui/events/devices into //ui/events/devices +
//ui/events/devices/x11 to keep X11 code in a different component. Also
udpate GYP targets in a similar fashion.

Update some dependent targets and fix a few GN check problems that crop
up because GN check doesn't fully understand #if defined(USE_X11) and
includes.

Originally commited at r388472 and reverted at r388474. There were
missing deps in the GN Chrome OS X11 non-Ozone component build that I
missed. Nothing in the CQ runs that configuration but it made a
waterfall bot fail.

TBR=sky@chromium.org,sievers@chromium.org
BUG=588026

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

[commit-bot: I haz the power, 2016-04-22 19:23:59]

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