Use contentsToRootView when converting the mouse coordinate for the context menu key event
Created attachment 117026 [details] Patch
Comment on attachment 117026 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117026&action=review > Source/WebCore/ChangeLog:8 > + No new tests. (OOPS!) Why not?
What is the symptom this patch fixes? Your bug title should describe it.
(In reply to comment #2) > (From update of attachment 117026 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=117026&action=review > > > Source/WebCore/ChangeLog:8 > > + No new tests. (OOPS!) > > Why not? This should already be covered by existing tests. (In reply to comment #3) > What is the symptom this patch fixes? Your bug title should describe it. Another part of https://bugs.webkit.org/show_bug.cgi?id=71945 - I'll relate the bugs and put the info in the change log.
Created attachment 117033 [details] Patch
Comment on attachment 117033 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117033&action=review > Source/WebCore/ChangeLog:12 > + Use contentsToRootView when converting the mouse coordinates for the context menu key event > + https://bugs.webkit.org/show_bug.cgi?id=73352 > + > + Reviewed by NOBODY (OOPS!). > + > + This is another step towards fixing https://bugs.webkit.org/show_bug.cgi?id=71945, by getting > + rid of a call to ScrollView::contentsToWindow. > + > + * page/EventHandler.cpp: > + (WebCore::EventHandler::sendContextMenuEventForKey): You should mention why you haven't modified/added any tests.
Committed r101416: <http://trac.webkit.org/changeset/101416>