Bug 129487

Summary: [CSS Regions] The height of a scrollable element flowed inside a scrollable region is not computed correctly
Product: WebKit Reporter: Radu Stavila <stavila>
Component: CSSAssignee: Radu Stavila <stavila>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, glenn, kondapallykalyan, WebkitBugTracker
Priority: P2 Keywords: AdobeTracked
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 57312    
Attachments:
Description Flags
Test-case
none
Test-case 2
none
Patch
none
Patch v2 none

Radu Stavila
Reported 2014-02-28 08:21:29 PST
Created attachment 225467 [details] Test-case If the scrollable element flowed into a region is large enough to not fit the region and generate scrollbars, the region cannot be scrolled all the way down to see the entire flowed element. See test-case.
Attachments
Test-case (2.82 KB, text/html)
2014-02-28 08:21 PST, Radu Stavila
no flags
Test-case 2 (2.79 KB, text/html)
2014-03-07 07:12 PST, Radu Stavila
no flags
Patch (66.70 KB, patch)
2014-03-20 05:22 PDT, Radu Stavila
no flags
Patch v2 (66.21 KB, patch)
2014-03-20 06:43 PDT, Radu Stavila
no flags
Radu Stavila
Comment 1 2014-03-07 07:12:02 PST
Created attachment 226125 [details] Test-case 2 Apparently this issue also happens when flowing a div with a not-specified height or a specified height which is greater than the height of its contents. In that situation, the region cannot be scrolled all the way down to see the bottom border of the flowed div.
Radu Stavila
Comment 2 2014-03-20 05:22:10 PDT
Radu Stavila
Comment 3 2014-03-20 06:43:45 PDT
Created attachment 227286 [details] Patch v2
Andrei Bucur
Comment 4 2014-03-20 06:46:45 PDT
Comment on attachment 227286 [details] Patch v2 r=me
WebKit Commit Bot
Comment 5 2014-03-20 08:01:41 PDT
Comment on attachment 227286 [details] Patch v2 Clearing flags on attachment: 227286 Committed r165964: <http://trac.webkit.org/changeset/165964>
WebKit Commit Bot
Comment 6 2014-03-20 08:01:44 PDT
All reviewed patches have been landed. Closing bug.
Radu Stavila
Comment 7 2014-06-03 09:08:21 PDT
Note You need to log in before you can comment on or make changes to this bug.