Bug 134286

Summary: [iOS][WK2] Rotating a zoomed PDF leads to weird scrolling behavior
Product: WebKit Reporter: Tim Horton <thorton>
Component: WebKit2Assignee: Tim Horton <thorton>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, bunhere, cdumez, commit-queue, gyuyoung.kim, sergio, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch benjamin: review+

Description Tim Horton 2014-06-24 18:49:43 PDT 
The root view bounds and contentSize need to be scaled :|
Comment 1 Radar WebKit Bug Importer 2014-06-24 18:50:41 PDT 
<rdar://problem/17445835>
Comment 2 Tim Horton 2014-06-24 18:56:43 PDT 
Created attachment 233775 [details]
patch
Comment 3 Benjamin Poulain 2014-06-24 19:49:37 PDT 
Comment on attachment 233775 [details]
patch

Updating the frame to account for the scale makes sense.

But updating this on scroll make no absolutely no sense to me.
Comment 4 Tim Horton 2014-06-24 21:09:50 PDT 
(In reply to comment #3)
> (From update of attachment 233775 [details])
> Updating the frame to account for the scale makes sense.
> 
> But updating this on scroll make no absolutely no sense to me.

It ... isn't updated on scroll. Only web_setContentProviderData and web_setMinimumSize call _computePageAndDocumentFrames. Maybe you're thinking of _revalidateViews? Or I'm missing something?
Comment 5 Tim Horton 2014-06-25 14:43:48 PDT 
http://trac.webkit.org/changeset/170439