Bug 36461

Summary: No vertical scrollbar after the CSS class change
Product: WebKit Reporter: Nickolay <nickolay8>
Component: Layout and RenderingAssignee: Shinichiro Hamaji <hamaji>
Severity: Normal CC: eric, hamaji, hyatt, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Description Flags
test case
Patch v1
Patch v2
Patch v2 - rebased hyatt: review+

Description Nickolay 2010-03-22 13:27:24 PDT
The vertical scrollbar do not appears after the CSS class which suppress them gets removed.

Attaching the test case. It fails on latests Safari for Windows, and Chrome for Ubuntu.

The case contains a long text with vertical scrollbar, initially suppressed by css class. 
After pressing 'Unmask' the css class is being removed, but no scrollbars appears.

Regards, Nickolay
Comment 1 Nickolay 2010-03-22 13:27:52 PDT
Created attachment 51340 [details]
test case
Comment 2 Shinichiro Hamaji 2010-03-29 10:21:48 PDT
Created attachment 51925 [details]
Patch v1
Comment 3 Shinichiro Hamaji 2010-03-29 10:23:49 PDT
A few info in addition to WebCore/ChangeLog in my patch:

- This issue also happens with horizontal scrollbars
- I guess this was introduced in http://trac.webkit.org/changeset/5085 (so I'm CCing dhyatt)
- I've confirmed my testcase doesn't show scrollbars without this patch
Comment 4 Simon Fraser (smfr) 2010-03-29 10:47:21 PDT
Your testcase doesn't have to contain so much text. It could be just <div style="height: 1000px">
Comment 5 Shinichiro Hamaji 2010-03-29 11:00:58 PDT
Created attachment 51931 [details]
Patch v2
Comment 6 Shinichiro Hamaji 2010-03-29 11:07:13 PDT
Created attachment 51932 [details]
Patch v2 - rebased
Comment 7 Shinichiro Hamaji 2010-03-29 11:09:07 PDT
(In reply to comment #4)
> Your testcase doesn't have to contain so much text. It could be just <div
> style="height: 1000px">

Thanks for the comment! I've fixed my test case (and rebased to avoid purple bot status).
Comment 8 Dave Hyatt 2010-03-29 11:10:51 PDT
Comment on attachment 51931 [details]
Patch v2

Comment 9 Dave Hyatt 2010-03-29 11:11:11 PDT
Comment on attachment 51932 [details]
Patch v2 - rebased

Comment 10 Eric Seidel (no email) 2010-03-29 11:39:54 PDT
Attachment 51932 [details] was posted by a committer and has review+, assigning to Shinichiro Hamaji for commit.
Comment 11 Shinichiro Hamaji 2010-03-29 21:35:43 PDT
Thanks for your quick reviews!
Comment 12 Shinichiro Hamaji 2010-03-29 23:15:43 PDT
Committed r56764: <http://trac.webkit.org/changeset/56764>
Comment 13 Nickolay 2010-05-19 04:42:36 PDT
Please check also the closely related bug:

Seems this issue was fixed only on 50%.

Regards, Nickolay