events: Introduce PlatformEventDispatcher and PlatformEventSource.

ui/events/platform/platform_event_types.h

Index: ui/events/platform/platform_event_types.h
diff --git a/ui/events/platform/platform_event_types.h b/ui/events/platform/platform_event_types.h
new file mode 100644
index 0000000000000000000000000000000000000000..a6444668bde389a468312bdceff7a399b08925c3
--- /dev/null
+++ b/ui/events/platform/platform_event_types.h
@@ -0,0 +1,24 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef UI_EVENTS_PLATFORM_PLATFORM_EVENT_TYPES_H_
+#define UI_EVENTS_PLATFORM_PLATFORM_EVENT_TYPES_H_
+
+#if defined(USE_X11)
+typedef union _XEvent XEvent;
+#elif defined(OS_WIN)
+#include <windows.h>
+#endif
+
+namespace ui {
+#if defined(USE_X11)
+typedef XEvent* PlatformEvent;

[sky, 2014/03/25 15:51:15]

Aren't you duplicating NativeEvent here?

[sadrul, 2014/03/25 18:33:15]

I am, yes. My plan is to get rid of base::NativeEvent altogether (it's currently
a bit confusing with base::NativeEvent and gfx::NativeEvent), since
MessagePumpDispatcher/MessagePumpObservers are the only users of this in base/.

I have changed this block to simply 'typedef base::NativeEvent PlatformEvent;'
Does this look more reasonable?

+#elif defined(OS_WIN)
+typedef MSG PlatformEvent;
+#else
+typedef void* PlatformEvent;

[sky, 2014/03/25 15:51:15]

Maybe this should be an error?

+#endif
+}  // namespace ui
+
+#endif  // UI_EVENTS_PLATFORM_PLATFORM_EVENT_TYPES_H_