Bug 126531

Summary: On iOS, zooming in with a TileController-backed main frame makes hundreds of tiles
Product: WebKit Reporter: Tim Horton <thorton>
Component: Layout and RenderingAssignee: Tim Horton <thorton>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, commit-queue, sam, simon.fraser
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 126536    
Bug Blocks:    
Attachments:
Description Flags
patch none

Tim Horton
Reported 2014-01-06 11:40:43 PST
<rdar://problem/15745862> The exposed rect that's handed through to TileController is in view coordinates. If we delegate page scale to the view (like iOS), this causes trouble, and we need to transform it appropriately.
Attachments
patch (7.68 KB, patch)
2014-01-15 13:41 PST, Tim Horton 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tim Horton
Comment 1 2014-01-15 13:33:34 PST
Not quite right. TileController's exposedRect is in FrameView coordinate space, but the TileController's visibleRect is in 'document' space. We need to scale the exposedRect. This also makes exposedRect stuff work correctly on desktop if you flip it on for Safari/MiniBrowser and zoom. Also, WKContentView was not sending the exposedRect in FrameView coordinate space, but it should be.
Tim Horton
Comment 2 2014-01-15 13:41:26 PST
Created attachment 221305 [details] patch
WebKit Commit Bot
Comment 3 2014-01-16 23:27:17 PST
Comment on attachment 221305 [details] patch Clearing flags on attachment: 221305 Committed r162189: <http://trac.webkit.org/changeset/162189>
WebKit Commit Bot
Comment 4 2014-01-16 23:27:19 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.