Bug 130301 - element.click does synchronous IPC to get the screenRect.
Summary: element.click does synchronous IPC to get the screenRect.
Status: RESOLVED DUPLICATE of bug 163648
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Darin Adler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-03-15 23:54 PDT by Andreas Kling
Modified: 2016-10-19 10:27 PDT (History)
3 users (show)

See Also:

Attachments
First cut (4.84 KB, patch)
2014-03-16 01:07 PDT, Darin Adler 		buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2 (484.83 KB, application/zip)
2014-03-16 01:57 PDT, Build Bot 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Kling 2014-03-15 23:54:30 PDT 
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.
Comment 1 Darin Adler 2014-03-15 23:55:40 PDT 
Should be pretty easy to reverse this so UI process sends the mapping to the web process.
Comment 2 Darin Adler 2014-03-16 01:07:24 PDT 
Created attachment 226836 [details]
First cut
Comment 3 Build Bot 2014-03-16 01:57:30 PDT 
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
Comment 4 Build Bot 2014-03-16 01:57:31 PDT 
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
Comment 5 Darin Adler 2016-10-19 10:27:00 PDT 
The changes in bug 163648 are the practical solution to this problem.

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