Add GN Build file for ui/events/ozone.

Add GN Build file for ui/events/ozone.

This was tested with:

$ gn gen out/Debug_gn --args='use_ozone=true'
$ ninja -C out/Debug_gn events_ozone events_ozone_evdev

BUG=None
TEST=see above
R=brettw@chromium.org
TBR=ben@chromium # for ui/ changes
NOTRY=true

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

[tfarina, 2014-06-10 03:42:54]

Certainly not the easiest. But it was fun to do.

https://codereview.chromium.org/322983002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/322983002/diff/1/BUILD.gn#newcode21
BUILD.gn:21: "//ui/events/ozone:events_ozone",
I tried //ui/events/ozone:*, but that didn't work. Is there another way?

https://codereview.chromium.org/322983002/diff/1/BUILD.gn#newcode21
BUILD.gn:21: "//ui/events/ozone:events_ozone",
Do I need to hide this behind an:

if (use_ozone) {
 ..
}

here?

https://codereview.chromium.org/322983002/diff/1/ui/events/ozone/BUILD.gn
File ui/events/ozone/BUILD.gn (right):

https://codereview.chromium.org/322983002/diff/1/ui/events/ozone/BUILD.gn#new...
ui/events/ozone/BUILD.gn:33: sources_assignment_filter += [
I think I will learn something about this.

[brettw, 2014-06-10 05:13:19]

Thanks for looking at this one!

https://codereview.chromium.org/322983002/diff/20001/ui/events/ozone/BUILD.gn
File ui/events/ozone/BUILD.gn (right):

https://codereview.chromium.org/322983002/diff/20001/ui/events/ozone/BUILD.gn...
ui/events/ozone/BUILD.gn:33: sources_assignment_filter += [
This won't actually work since the filter is only applied when sources is
assigned to, and this is a more complicated regexp that is supported anyway.

I think this only applies to two files anyway so:
sources -= [
  "device/udev/device_manager_udev.cc",
  "device/udev/device_manager_udev.h",
]

https://codereview.chromium.org/322983002/diff/20001/ui/events/ozone/BUILD.gn...
ui/events/ozone/BUILD.gn:97: sources_assignment_filter += [
Ditto here that this doesn't work.

Since this is a bunch of files, I'd actually delete this and move all of the
libgestures_glue files into the
  if (use_ozone_evdev && use_evdev_gestures) {
block (so the're never added in this else case).

[tfarina, 2014-06-10 05:24:37]

https://codereview.chromium.org/322983002/diff/20001/ui/events/ozone/BUILD.gn
File ui/events/ozone/BUILD.gn (right):

https://codereview.chromium.org/322983002/diff/20001/ui/events/ozone/BUILD.gn...
ui/events/ozone/BUILD.gn:33: sources_assignment_filter += [
On 2014/06/10 05:13:18, brettw wrote:
> This won't actually work since the filter is only applied when sources is
> assigned to, and this is a more complicated regexp that is supported anyway.
> 
> I think this only applies to two files anyway so:
> sources -= [
>   "device/udev/device_manager_udev.cc",
>   "device/udev/device_manager_udev.h",
> ]

Done.

https://codereview.chromium.org/322983002/diff/20001/ui/events/ozone/BUILD.gn...
ui/events/ozone/BUILD.gn:97: sources_assignment_filter += [
On 2014/06/10 05:13:18, brettw wrote:
> Ditto here that this doesn't work.
> 
> Since this is a bunch of files, I'd actually delete this and move all of the
> libgestures_glue files into the
>   if (use_ozone_evdev && use_evdev_gestures) {
> block (so the're never added in this else case).

Done.

[brettw, 2014-06-10 05:45:29]

LGTM!

https://codereview.chromium.org/322983002/diff/40001/ui/events/ozone/BUILD.gn
File ui/events/ozone/BUILD.gn (right):

https://codereview.chromium.org/322983002/diff/40001/ui/events/ozone/BUILD.gn...
ui/events/ozone/BUILD.gn:71: ":events_ozone",
This goes at the top of the list of the deps here.

[tfarina, 2014-06-10 15:44:41]

The CQ bit was checked by tfarina@chromium.org

[tfarina, 2014-06-10 15:44:55]

https://codereview.chromium.org/322983002/diff/40001/ui/events/ozone/BUILD.gn
File ui/events/ozone/BUILD.gn (right):

https://codereview.chromium.org/322983002/diff/40001/ui/events/ozone/BUILD.gn...
ui/events/ozone/BUILD.gn:71: ":events_ozone",
On 2014/06/10 05:45:29, brettw wrote:
> This goes at the top of the list of the deps here.

Done.

[commit-bot: I haz the power, 2014-06-10 15:49:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tfarina@chromium.org/322983002/60001

[commit-bot: I haz the power, 2014-06-10 15:58:49]

Change committed as 276044