Created attachment 377952 [details] test case [Win] Add support for MouseEvent.buttons https://www.w3.org/TR/uievents/#dom-mouseevent-buttons Bug 178214 – [Mac] Add support for MouseEvent.buttons Bug 191481 – REGRESSION(r223264): [GTK] Unable to drag documents on Google Drive
another test case: https://jsfiddle.net/fujihiro/vy7hkpLw/10/embedded/result/
Created attachment 377954 [details] Patch
Comment on attachment 377954 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=377954&action=review > Source/WebCore/platform/win/PlatformMouseEventWin.cpp:92 > +} Could we put this helper somewhere like GDIUtilities.h?
Comment on attachment 377954 [details] Patch Attachment 377954 [details] did not pass win-ews (win): Output: https://webkit-queues.webkit.org/results/12995131 Number of test failures exceeded the failure limit.
Created attachment 377956 [details] Archive of layout-test-results from ews210 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews210 Port: win-future Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment on attachment 377954 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=377954&action=review > Source/WebCore/platform/win/PlatformMouseEventWin.cpp:121 > + m_button = NoButton; This change seems to break AppleWin (editing and drag/drop operations). Do those tests need to be rebased? Is WebKit relying on behavior that is counter to the spec? The tests that are failing related to editing/selection and dragging. I know that dragging involves mouse moves with the left button held down; I suspect this change breaks those tests because MouseMove (after this change) always has NoButton. I don't think this change is correct.
Oh.. Filed: Bug 201496 – MouseEvent.button should be 0 for mousemove events Bug 201497 – WheelEvent should set buttons property properly because it is a subclass of MouseEvent
Created attachment 378054 [details] Patch * Addressed the review feedback.
Comment on attachment 378054 [details] Patch r=me
Comment on attachment 378054 [details] Patch Clearing flags on attachment: 378054 Committed r249560: <https://trac.webkit.org/changeset/249560>
All reviewed patches have been landed. Closing bug.
<rdar://problem/55096934>