Bug 107592

Summary: [Chromium] Correct auto-zoom when using compositor scaling
Product: WebKit Reporter: Xianzhu Wang <wangxianzhu>
Component: Layout and RenderingAssignee: Xianzhu Wang <wangxianzhu>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, aelias, enne, extendermywifi, jamesr, klobag, trchen, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Updated patch none

Description Xianzhu Wang 2013-01-22 14:57:45 PST 
Need to correct auto-zoom when using compositor scaling (applyDeviceScaleFactorInCompositor() and applyPageScaleFactorInCompositor()).
Comment 1 Xianzhu Wang 2013-01-22 15:02:25 PST 
        1. The scale of auto-zoom should exclude deviceScaleFactor because the compositor will handle the scaling;
        2. Scroll location needs to be calculated differently when pageScaleFactor is handled by the compositor.
Comment 2 Xianzhu Wang 2013-01-22 15:04:03 PST 
Created attachment 184056 [details]
Patch
Comment 3 Xianzhu Wang 2013-01-22 15:38:06 PST 
The patch is now conflicting with aelias@'s change. Will update after that change is landed.
Comment 4 Xianzhu Wang 2013-01-22 17:27:22 PST 
Created attachment 184090 [details]
Updated patch
Comment 5 Xianzhu Wang 2013-01-22 17:28:24 PST 
The new patch no longer conflicts with the patch of bug 107424.
Comment 6 Adam Barth 2013-01-28 12:31:16 PST 
Comment on attachment 184090 [details]
Updated patch

Thanks for the tests.
Comment 7 WebKit Review Bot 2013-01-28 13:09:51 PST 
Comment on attachment 184090 [details]
Updated patch

Clearing flags on attachment: 184090

Committed r140994: <http://trac.webkit.org/changeset/140994>
Comment 8 WebKit Review Bot 2013-01-28 13:09:55 PST 
All reviewed patches have been landed.  Closing bug.