aura: Remove WindowTreeHost::PostNativeEvent().

aura: Remove WindowTreeHost::PostNativeEvent().

WindowTreeHost::PostNativeEvent() is only used in tests. So remove this from
the public API. Instead:
 . For Win32, directly use PostMessage (this is used in only two places).
 . For Ozone, use EventSourceTestApi to inject the event (this is equivalent to
   current code, in terms of not doing the right thing when menus are open).
 . For X11, introduce aura::test::PostEventToWindowTreeHost() instead, and use
   it in tests.

BUG=none
R=sky@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/c332de7c1838b85e27685bed6c376c36ccb17287

[sadrul, 2014-11-07 01:23:10]

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

[sky, 2014-11-07 15:50:15]

https://codereview.chromium.org/710553002/diff/1/ui/aura/test/ui_controls_fac...
File ui/aura/test/ui_controls_factory_ozone.cc (right):

https://codereview.chromium.org/710553002/diff/1/ui/aura/test/ui_controls_fac...
ui/aura/test/ui_controls_factory_ozone.cc:169: void
SendEventToProcessor(ui::Event* event) {
Is there a reason this needs to be a pointer? I briefly looked at why
SendEventToProcessor, but it isn't clear why that needs a pointer too.

https://codereview.chromium.org/710553002/diff/1/ui/aura/test/x11_event_sender.h
File ui/aura/test/x11_event_sender.h (right):

https://codereview.chromium.org/710553002/diff/1/ui/aura/test/x11_event_sende...
ui/aura/test/x11_event_sender.h:10: typedef union _XEvent XEvent;
using?

[sadrul, 2014-11-07 17:03:15]

https://codereview.chromium.org/710553002/diff/1/ui/aura/test/ui_controls_fac...
File ui/aura/test/ui_controls_factory_ozone.cc (right):

https://codereview.chromium.org/710553002/diff/1/ui/aura/test/ui_controls_fac...
ui/aura/test/ui_controls_factory_ozone.cc:169: void
SendEventToProcessor(ui::Event* event) {
On 2014/11/07 15:50:14, sky wrote:
> Is there a reason this needs to be a pointer? I briefly looked at why
> SendEventToProcessor, but it isn't clear why that needs a pointer too.

This is mostly so the consumers can do SetHandled()/StopPropagation() on the
event.

https://codereview.chromium.org/710553002/diff/1/ui/aura/test/x11_event_sender.h
File ui/aura/test/x11_event_sender.h (right):

https://codereview.chromium.org/710553002/diff/1/ui/aura/test/x11_event_sende...
ui/aura/test/x11_event_sender.h:10: typedef union _XEvent XEvent;
On 2014/11/07 15:50:14, sky wrote:
> using?

Done.

[sky, 2014-11-07 22:49:46]

Ok, LGTM

[commit-bot: I haz the power, 2014-11-07 22:58:44]

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

[sadrul, 2014-11-07 22:58:50]

Committed patchset #3 (id:40001) manually as
c332de7c1838b85e27685bed6c376c36ccb17287 (presubmit successful).