Bug 77350

Summary: All mouse events after a right click are ignored when they came from WebkitTestRunner
Product: WebKit Reporter: Hugo Parente Lima <hugo.lima>
Component: UI EventsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, cshu, hugo.lima, mjs, sam, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch none

Hugo Parente Lima
Reported 2012-01-30 12:04:38 PST
When a right click comes to WebCore and it realize that a context menu should pop up to the user it sends back a event telling the UI process to show the context menu, but as described in bug#58406, a mouse release event may already be in the CoreIPC pipeline because the UI sent it before receiving the "show context menu" event from WebProcess. To solve this issue the patch from bug#58406 added a flag to tell WebPage if it should ignore the mouse release event (or any other mouse events). WebKitTestRunner send events synchronized, so the issue reported on bug#58406 doesn't affect WebKitTestRunner, besides due to the sync nature of WebKitTestRunner it will only receive the showContextMenu (that was sent async) after it already sent all events, so all events sent after the right click event will be ignored by webcore. The proposed patch removes the check when using synchronized events making some tests that are Skipped on many webkit2 ports pass. Another possible approach would send the showContextMenu/contextMenuHidden events sync, but contextMenuHidden is only used to fix bug#58406. Please correct me if any assumptions I made are wrong. Test now passing: fast/events/mouse-click-events.html Found on following skip lists: qt-wk2/Skipped:57 efl/Skipped:1523 gtk/Skipped:839 mac-wk2/Skipped:146 win-wk2/Skipped:224 The patch only removes the test from qt-wk2 skip list, but it probably would make this test pass on other wk2 ports.
Attachments
Patch (2.36 KB, patch)
2012-01-30 12:15 PST, Hugo Parente Lima
no flags
Patch (2.99 KB, patch)
2012-02-24 05:43 PST, Hugo Parente Lima
no flags
Hugo Parente Lima
Comment 1 2012-01-30 12:15:15 PST
Chang Shu
Comment 2 2012-02-13 02:00:42 PST
Nice, have you tried to run all the tests to avoid regressions?
Hugo Parente Lima
Comment 3 2012-02-15 08:35:54 PST
I did it at the time I uploaded the patch, the results must be the same nowadays.
WebKit Review Bot
Comment 4 2012-02-15 11:22:20 PST
Comment on attachment 124580 [details] Patch Rejecting attachment 124580 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: ing file Source/WebKit2/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file Source/WebKit2/WebProcess/WebPage/WebPage.cpp Hunk #1 succeeded at 1297 (offset 15 lines). patching file LayoutTests/platform/qt-wk2/Skipped Hunk #1 FAILED at 54. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/qt-wk2/Skipped.rej Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Chang Shu']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue/ Full output: http://queues.webkit.org/results/11530252
Hugo Parente Lima
Comment 5 2012-02-16 20:29:01 PST
This reviewbot messge means I should rebase the patch myself and re-upload it?
Chang Shu
Comment 6 2012-02-17 00:55:41 PST
yes,simply replace oops with reviewer's name and cq+
Hugo Parente Lima
Comment 7 2012-02-24 05:43:50 PST
Hugo Parente Lima
Comment 8 2012-02-24 05:46:03 PST
Patch finally rebased.
WebKit Review Bot
Comment 9 2012-02-28 16:35:11 PST
The commit-queue encountered the following flaky tests while processing attachment 128720 [details]: css3/filters/effect-hue-rotate-hw.html bug 79845 (author: cmarrin@apple.com) The commit-queue is continuing to process your patch.
WebKit Review Bot
Comment 10 2012-02-28 16:37:21 PST
Comment on attachment 128720 [details] Patch Clearing flags on attachment: 128720 Committed r109169: <http://trac.webkit.org/changeset/109169>
WebKit Review Bot
Comment 11 2012-02-28 16:37:28 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.