On Thu, 24 Feb 2022 23:59:29 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Basically, buttons property is a mask which represents the button/buttons 
>> clicked on the mouse.
>>     It is observed that event.buttons property is set to 0 when there is 
>> mouse press or drag event.This behaviour is observed only with javafx 
>> webView.Other browsers set the buttons property to 1, when there is mouse 
>> press or drag.
>>      The issue happens because the buttons property is not updated in the 
>> framework.
>>      Added implementation to update and propagate the buttons property from 
>> javafx platform to native webkit.Added a robot test case for the same.
>>    Performed sanity testing with the added implementation and the buttons 
>> property is compliant with the specification mentioned in 
>> https://w3c.github.io/pointerevents/#the-buttons-property.
>
> modules/javafx.web/src/main/native/Source/WebCore/platform/PlatformMouseEvent.h
>  line 76:
> 
>> 74:        PlatformMouseEvent(const IntPoint& position, const IntPoint& 
>> globalPosition, MouseButton button, PlatformEvent::Type type,
>> 75:                            int clickCount, bool shiftKey, bool ctrlKey, 
>> bool altKey, bool metaKey, WallTime timestamp, double force,
>> 76:                            SyntheticClickType syntheticClickType, 
>> PointerID pointerId = mousePointerID)
> 
> I recommend reverting this change, since this is in WebKit shared code and 
> the only change you made is in formatting. It will help avoid future merge 
> conflicts.

GitHub is showing more context than it should have, so my comment might be 
confusing. I only meant to suggest that you revert the reformatting of the 
existing constructor. Everything inside the `#if` looks fine.

> move the runAndWait outside the list...

I meant "loop"

-------------

PR: https://git.openjdk.java.net/jfx/pull/742

Reply via email to