Get helper_hittest_overscroll_contextmenu.html to pass on Windows
Categories
(Core :: Panning and Zooming, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox106 | --- | fixed |
People
(Reporter: botond, Assigned: hiro)
References
(Blocks 2 open bugs)
Details
Attachments
(2 files)
In bug 1704070 we fixed a bug, where clicking in the root scroll frame's overscroll gutter activated a browser chrome context menu, for Linux and Mac.
The test for that, helper_hittest_overscroll_contextmenu.html, is currently skipped on Windows, and would fail if it were run. This bug tracks fixing the issue and enabling the test on Windows.
Reporter | ||
Updated•3 years ago
|
Assignee | ||
Comment 1•2 years ago
|
||
I figured out a way to solve this issue. IIUC on Windows WM_CONTEXTMENU message is sent from OS and we can tell whether the message was triggered by mouse or not just like this code. So with telling whether the last mouse up event was in overscroll gutter just like we did in bug 1704070 for Mac/Linux we can prevent opening context menu.
Assignee | ||
Comment 2•2 years ago
|
||
There's another case we need to care to prevent opening context menu in the case of touch sceen (with touch pen?) here. I have a local fix to the case, but hasn't finished writing tests. I will handle it in another bug later.
Assignee | ||
Comment 3•2 years ago
|
||
The comment about NS_IF_RELEASE() is now useless since there's no
NS_IF_RELEASE() call.
Assignee | ||
Comment 4•2 years ago
|
||
Depends on D155038
Pushed by hikezoe.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c6f17233ceac Remove a comment about NS_IF_RELEASE() use in nsWindow::DispatchMouseEvent. r=botond https://hg.mozilla.org/integration/autoland/rev/550108db5022 Prevent opening context menu in the case the user did right click in the overscroll gutter on Windows. r=botond,handyman
Comment 6•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c6f17233ceac
https://hg.mozilla.org/mozilla-central/rev/550108db5022
Description
•