Bug 126261

Summary: [WK2][CoordinatedGraphics] Adding callback to let WebView know that WebPageProxy has changed its pageScaleFactor
Product: WebKit Reporter: Thiago de Barros Lacerda <thiago.lacerda>
Component: WebKit2Assignee: Thiago de Barros Lacerda <thiago.lacerda>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: benjamin, cmarcelo, commit-queue, gyuyoung.kim, kling, luiz, nick.diego, noam, simon.fraser, thorton, zeno
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 126960    
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Thiago de Barros Lacerda
Reported 2013-12-27 06:43:54 PST
After r161087, the scale factor is being set directly in WebPageProxy, avoiding information duplication of the scale factor. However, it takes a while to the scale factor gets applied in WebProcess and get back to UIProcess (WebPageProxy) due do IPC. So, during this interval, if the user queries the contentScaleFactor to the WebView he will get the old scale factor. This callback is useful to let the user know when it can rely on the contentScaleFactor. It is also important for API tests that might use contentScaleFactor
Attachments
Patch (6.98 KB, patch)
2013-12-27 06:51 PST, Thiago de Barros Lacerda 		no flags
DetailsFormatted DiffDiff
Patch (10.95 KB, patch)
2013-12-27 09:52 PST, Thiago de Barros Lacerda 		no flags
DetailsFormatted DiffDiff
Patch (11.22 KB, patch)
2014-01-02 16:01 PST, Thiago de Barros Lacerda 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Thiago de Barros Lacerda
Comment 1 2013-12-27 06:51:31 PST
Created attachment 220058 [details] Patch
Thiago de Barros Lacerda
Comment 2 2013-12-27 09:52:06 PST
Created attachment 220063 [details] Patch
Thiago de Barros Lacerda
Comment 3 2013-12-27 09:54:02 PST
(In reply to comment #2) > Created an attachment (id=220063) [details] > Patch Forgot to fix the test
Thiago de Barros Lacerda
Comment 4 2014-01-02 16:01:27 PST
Created attachment 220263 [details] Patch
Thiago de Barros Lacerda
Comment 5 2014-01-16 02:19:35 PST
*** This bug has been marked as a duplicate of bug 127090 ***
Note You need to log in before you can comment on or make changes to this bug.