Bug 130301 – element.click does synchronous IPC to get the screenRect.

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: WebKit2

[Andreas Kling]

Actual IPC happens here:

IntRect WebPage::rootViewToScreen(const IntRect& rect)
{
    IntRect screenRect;
    sendSync(Messages::WebPageProxy::RootViewToScreen(rect), Messages::WebPageProxy::RootViewToScreen::Reply(screenRect));
    return screenRect;
}

This is showing up as idle stalls in DoYouEvenBench occasionally.

[Darin Adler]

Should be pretty easy to reverse this so UI process sends the mapping to the web process.

[Darin Adler]

Created attachment 226836 [details]
First cut

[Build Bot]

Comment on attachment 226836 [details]
First cut

Attachment 226836 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5497521262559232

New failing tests:
platform/mac/accessibility/visible-position-crash-for-text-node.html
platform/mac/accessibility/text-marker-for-bounds.html
platform/mac/accessibility/range-for-position.html
platform/mac/accessibility/webkit-scrollarea-position.html
platform/mac/accessibility/scroll-to-visible-action.html

[Build Bot]

Created attachment 226838 [details]
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-14  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5

[Darin Adler]

The changes in bug 163648 are the practical solution to this problem.

*** This bug has been marked as a duplicate of bug 163648 ***