It looks fine now. Please, let Anthony also confirm this.
Thanks,
Artem
On 10/2/2013 5:33 PM, Petr Pchelko wrote:
Hello, Thank you for the review.
It really seems to be an issue. We need another check that hWndForWheel belongs
to the current process, and doesn't sent the event to this window, if it
doesn't.
That was indeed an issue. I've got used to Mac behaviour and did not notice the
problem.
The new version of the fix is available here:
http://cr.openjdk.java.net/~pchelko/8025585/webrev.00/
Now we redispatch the wheel event only if the window is in the same process.
With best regards. Petr.
On Oct 1, 2013, at 9:54 PM, Artem Ananiev <artem.anan...@oracle.com> wrote:
On 10/1/2013 9:17 PM, Anthony Petrov wrote:
Hi Petr,
MS Windows always sends WHEEL events to the focused window. And my
testing shows that when you scroll the wheel outside of an AWT window,
the scroll events are consumed (by AWT, supposedly).
With your fix you seem to always redirect the events to the window under
the mouse. Now suppose you have a 3rd-party window with a scroll bar in
background (e.g. Windows Explorer). If an AWT window is currently
focused, but you move the mouse pointer outside of it and above the
Explorer window, and start scrolling, will the Explorer window scroll
its contents? Currently it doesn't. What about if we apply your fix?
It really seems to be an issue. We need another check that hWndForWheel belongs
to the current process, and doesn't sent the event to this window, if it
doesn't.
Thanks,
Artem
--
best regards,
Anthony
On 09/27/2013 07:24 PM, Petr Pchelko wrote:
Hello, AWT Team.
Please review the fix for the issue:
https://bugs.openjdk.java.net/browse/JDK-8025585
The fix is available at:
http://cr.openjdk.java.net/~pchelko/8025585/webrev.00/
The fix is needed for JFXPanel support. We need to redispatch
MOUSEWHEEL messages to the window under mouse. In case of Popups in a
JFXPanel the HWND belongs to a different tollkit, so we need to use
::SendMessage to redispatch the message.
With best regards. Petr.
