[Webkit-unassigned] [Bug 21462] Overflow auto in div element shows obsolete scrollbars when content size is reduced

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon May 11 05:34:13 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=21462





------- Comment #10 from eric at webkit.org  2009-05-11 05:34 PDT -------
(From update of attachment 26424)
This patch uses tabs (which will cause the commit to fail.  This patch also
adds extra { } which violate the style guidelines:
http://webkit.org/coding/coding-style.html

I cleaned up the test case slightly to use the "blue means you need to look at
the test case to understand the results; green means pass and red means fail"
system.

Otherwise, the patch looks sane to me.  Hyatt really needs to weigh in here
though.  Ideally you'd update the patch to fix the style issues.  Also I would
probably have used // for the clientHeight/clientWidth comment and would have
broken it into two lines.  But I think /* is OK too, I'd have to check the
style guidelines to be sure.


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list