To match WebKit1 behavior, we need to send unhandled wheel events to the parent window on Windows to allow the application to deal with them.
Created attachment 92334 [details] Patch
Comment on attachment 92334 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=92334&action=review > Source/WebKit2/UIProcess/WebPageProxy.cpp:2633 > +#if PLATFORM(WIN) > + if (!handled && m_currentlyProcessedWheelEvent) > + wheelEventNotHandled(*m_currentlyProcessedWheelEvent); > +#endif Does this need to be Windows-only? > Source/WebKit2/UIProcess/WebPageProxy.h:697 > +#if PLATFORM(WIN) > + void wheelEventNotHandled(NativeWebWheelEvent&) const; > +#endif Does this need to be Windows-only? > Source/WebKit2/UIProcess/API/qt/qwkpage.cpp:343 > - WebWheelEvent wheelEvent = WebEventFactory::createWebWheelEvent(ev); > + NativeWebWheelEvent wheelEvent = NativeWebWheelEvent(ev); > page->handleWheelEvent(wheelEvent); Can we write this more simply, taking advantage of the default constructor? > Source/WebKit2/UIProcess/win/WebView.cpp:438 > + NativeWebWheelEvent wheelEvent = NativeWebWheelEvent(hWnd, message, wParam, lParam); We should write this with construction syntax, not construct an object and then assign it.
Comment on attachment 92334 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=92334&action=review >> Source/WebKit2/UIProcess/WebPageProxy.cpp:2633 >> +#endif > > Does this need to be Windows-only? I'm certain this is not needed on the Mac, although I'm not sure about other platforms. I'm assuming this is Windows-only for now. >> Source/WebKit2/UIProcess/WebPageProxy.h:697 >> +#endif > > Does this need to be Windows-only? I'm certain this is not needed on the Mac, although I'm not sure about other platforms. I'm assuming this is Windows-only for now. >> Source/WebKit2/UIProcess/API/qt/qwkpage.cpp:343 >> page->handleWheelEvent(wheelEvent); > > Can we write this more simply, taking advantage of the default constructor? I will clean this up before landing (hopefully without breaking the qt build, since I relied on the ews bots to catch my previous mistakes). >> Source/WebKit2/UIProcess/win/WebView.cpp:438 >> + NativeWebWheelEvent wheelEvent = NativeWebWheelEvent(hWnd, message, wParam, lParam); > > We should write this with construction syntax, not construct an object and then assign it. Fixed.
Committed r85808: <http://trac.webkit.org/changeset/85808>