<AWT Dev> [8] Review Request: JDK-8025585 Win: Popups in JFXPanel do not receive MouseWheel events

[Petr Pchelko]

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.ananiev at 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.
>>>