[ozone/wayland] Implement basic keyboard handling support

ui/ozone/platform/wayland/BUILD.gn

Index: ui/ozone/platform/wayland/BUILD.gn
diff --git a/ui/ozone/platform/wayland/BUILD.gn b/ui/ozone/platform/wayland/BUILD.gn
index 494ab6005256fd07c39b6b5353e8c5c29bbdac1f..841a52e0434de57010e1ee46ddd8857957ef47ce 100644
--- a/ui/ozone/platform/wayland/BUILD.gn
+++ b/ui/ozone/platform/wayland/BUILD.gn
@@ -20,6 +20,8 @@ source_set("wayland") {
     "ozone_platform_wayland.h",
     "wayland_connection.cc",
     "wayland_connection.h",
+    "wayland_keyboard.cc",
+    "wayland_keyboard.h",
     "wayland_object.cc",
     "wayland_object.h",
     "wayland_output.cc",
@@ -37,8 +39,11 @@ source_set("wayland") {
     "//skia",
     "//third_party/wayland:wayland_client",
     "//third_party/wayland-protocols:xdg_shell_protocol",
+    "//ui/base:ui_features",
     "//ui/display",
     "//ui/events",
+    "//ui/events/ozone:events_ozone_evdev",
+    "//ui/events/ozone:events_ozone_layout",

[spang, 2017/01/24 20:41:50]

Remove. This component is for Chrome OS (or other desktopless) builds.

[tonikitoo, 2017/01/24 21:51:11]

'gn check <out>' actually complains (see below).
Basically, it is needed to include
"ui/events/ozone/layout/keyboard_layout_engine_manager.h", so that we call:

 36 #if BUILDFLAG(USE_XKBCOMMON)
 37   KeyboardLayoutEngineManager::SetKeyboardLayoutEngine(
 38       base::MakeUnique<XkbKeyboardLayoutEngine>(codes_));
 39 #else
 40   KeyboardLayoutEngineManager::SetKeyboardLayoutEngine(
 41       base::MakeUnique<StubKeyboardLayoutEngine>());
 42 #endif


Actual error:


Dependency chain (there may also be others):
  //ui/ozone/platform/wayland:wayland -->
  //ui/events:events --[private]-->
  //ui/events/ozone:events_ozone_layout

___________________
ERROR at //ui/ozone/platform/wayland/wayland_connection.cc:15:11: Can't include
this header from here.
#include "ui/events/ozone/layout/keyboard_layout_engine_manager.h"
          ^------------------------------------------------------
The target:
  //ui/ozone/platform/wayland:wayland
is including a file from the target:
  //ui/events/ozone:events_ozone_layout

[spang, 2017/01/24 22:24:07]

Er, I meant to comment on "//ui/events/ozone:events_ozone_evdev" not
"//ui/events/ozone:events_ozone_layout". You do need the latter.

     "//ui/events/platform",
     "//ui/gfx",
     "//ui/gfx/geometry",
@@ -63,6 +68,7 @@ source_set("wayland_unittests") {
     "fake_server.h",
     "mock_platform_window_delegate.cc",
     "wayland_connection_unittest.cc",
+    "wayland_keyboard_unittest.cc",
     "wayland_pointer_unittest.cc",
     "wayland_surface_factory_unittest.cc",
     "wayland_test.cc",