Created attachment 64431 [details] patch

Attachment 64431 [details] did not pass style-queue: Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1 WebKit2/WebProcess/InjectedBundle/API/c/WKBundleMouseEvent.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] WebKit2/WebProcess/InjectedBundle/InjectedBundleMouseEvent.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] WebKit2/WebProcess/InjectedBundle/API/c/WKBundleWheelEvent.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] WebKit2/WebProcess/InjectedBundle/InjectedBundleKeyboardEvent.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] WebKit2/WebProcess/InjectedBundle/InjectedBundleWheelEvent.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] WebKit2/WebProcess/InjectedBundle/API/c/WKBundleAPICast.h:63: More than one command on the same line [whitespace/newline] [4] WebKit2/WebProcess/InjectedBundle/API/c/WKBundleAPICast.h:64: More than one command on the same line [whitespace/newline] [4] WebKit2/WebProcess/InjectedBundle/API/c/WKBundleAPICast.h:65: More than one command on the same line [whitespace/newline] [4] WebKit2/WebProcess/InjectedBundle/API/c/WKBundleAPICast.h:75: More than one command on the same line [whitespace/newline] [4] WebKit2/WebProcess/InjectedBundle/API/c/WKBundleAPICast.h:76: More than one command on the same line [whitespace/newline] [4] WebKit2/WebProcess/InjectedBundle/API/c/WKBundleAPICast.h:77: More than one command on the same line [whitespace/newline] [4] WebKit2/WebProcess/InjectedBundle/API/c/WKBundleKeyboardEvent.cpp:26: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] Total errors found: 12 in 27 files If any of these errors are false positives, please file a bug against check-webkit-style.

I was working on a similar approach when I became unsure if it made sense. It seems that WebCore really needs an NSEvent for input methods - or we need a much deeper refactoring for them to work. I'm not sure if adding injected bundle interfaces is helpful at this point.

One thing I'd like to preserve in eventSender is an ability to create native events (such as NSEvent), to test how they are handled. Testing only WebCore seems particularly insufficient in this case.

(In reply to comment #4) > One thing I'd like to preserve in eventSender is an ability to create native events (such as NSEvent), to test how they are handled. Testing only WebCore seems particularly insufficient in this case. My plan was to add a function for each platform that takes a platform specific event (NSEvent on mac) and creates a WKBundleFooEvent from it, this way we could test the conversion (and we wouldn't have to do the manual conversion ourselves in the test runner.

What's the status of this patch? It's been in the review queue for a while.

(In reply to comment #6) > What's the status of this patch? It's been in the review queue for a while. Apparently not a lot of love :(. I am taking it out of the queue for now, and will come back to when I am more focused on EventSender again.