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

[Artem Ananiev]

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