WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
118439
Fix
r152265
: FrameView's pagination mode is only one of two, and the logic was totally wrong
https://bugs.webkit.org/show_bug.cgi?id=118439
Summary
Fix r152265: FrameView's pagination mode is only one of two, and the logic wa...
Tim Horton
Reported
2013-07-05 17:30:19 PDT
The test was only passing by sheer happenstance (FrameView's pagination is the CSS one, so it was set to Unpaginated, so the totally-broken logic happened to cause the clamping not to happen). So, use both FrameView's and Page's. And fix the logic (we wanted to NOT clamp in the pagination case). Also, I found ways (with RTL text) that even left-to-right and top-to-bottom pagination can be broken here, so expand to all "pagination enabled" non-0-scrollOrigin states. Also, the test was broken because DrawingAreaImpl doesn't know what to do with a 40000 px wide view (and just asserts in SharedMemory code and thus makes the test time out). And DidFirstPaintAfterSuppressedIncrementalRendering doesn't fire if the window is offscreen. And EXPECT_JS_EQ is pretty cool. Basically, rewrite the patch. It deserves it.
Attachments
patch
(5.70 KB, patch)
2013-07-05 17:48 PDT
,
Tim Horton
andersca
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2013-07-05 17:30:42 PDT
<
rdar://problem/14366120
>
Tim Horton
Comment 2
2013-07-05 17:48:53 PDT
Created
attachment 206179
[details]
patch
Tim Horton
Comment 3
2013-07-05 19:20:39 PDT
http://trac.webkit.org/changeset/152433
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug