RESOLVED FIXED 7542
REGRESSION: (r13028) Scrolling causes incomplete drawing of <ul> bullets
https://bugs.webkit.org/show_bug.cgi?id=7542
Summary REGRESSION: (r13028) Scrolling causes incomplete drawing of <ul> bullets
Steven Fisher
Reported 2006-03-01 14:24:28 PST
Nightshift build 2006-02-28; recreated on new user account. Unable to recreate in Safari 2.0.3. Scrolling causes incomplete drawing of <ul> bullets. 1. Visit URL above: http://en.wikipedia.org/w/index.php?title=Quicksilver_%28software%29&oldid=40652040 2. Using the mouse wheel, scroll down to the long bullet lists in "Interfaces with other applications." Some of the bullets will be missing or partially drawn.
Attachments
System profile (944.48 KB, text/plain)
2006-03-01 14:34 PST, Steven Fisher
no flags
Screenshot (355.80 KB, image/png)
2006-03-01 14:34 PST, Steven Fisher
no flags
Broken bullets; some are "thinner" (39.37 KB, image/png)
2006-03-01 22:18 PST, crastinator
no flags
patch (506 bytes, patch)
2006-03-02 12:21 PST, Justin Garcia
sullivan: review+
Steven Fisher
Comment 1 2006-03-01 14:32:59 PST
Also reproduced using scroll bars instead of scroll wheel.
Steven Fisher
Comment 2 2006-03-01 14:34:10 PST
Created attachment 6795 [details] System profile
Steven Fisher
Comment 3 2006-03-01 14:34:55 PST
Created attachment 6796 [details] Screenshot
crastinator
Comment 4 2006-03-01 22:16:29 PST
Got the same thing here, they also become vertically "thinner" sometimes. It comes up when using directional keys, doesn't come up when using page up/down. Using build r13060, 10.4.5
crastinator
Comment 5 2006-03-01 22:18:10 PST
Created attachment 6803 [details] Broken bullets; some are "thinner"
mitz
Comment 6 2006-03-01 22:57:19 PST
Looks like "the regression version" of bug 4855.
Steven Fisher
Comment 7 2006-03-02 09:45:41 PST
#6: Well, I can't recreate this one (7542) on the shipping Safari. Doesn't that make this a regression, or have I erred?
mitz
Comment 8 2006-03-02 10:23:42 PST
(In reply to comment #7) > #6: Well, I can't recreate this one (7542) on the shipping Safari. Doesn't that > make this a regression, or have I erred? > Sorry, my previous comment was poorly-phrased and incorrect. This bug is a regression, and based on testing with nightly builds, it is probably a regression from the fix for bug 6844.
Steven Fisher
Comment 9 2006-03-02 11:19:14 PST
Thanks for the explanation. Sorry if I seemed offended or something, I wasn't. I was just trying not to ramble. :)
Justin Garcia
Comment 10 2006-03-02 12:12:32 PST
Here it is: + IntRect box(_tx + m_x, _ty + m_y, m_height, m_width); Oops!
Justin Garcia
Comment 11 2006-03-02 12:21:05 PST
Created attachment 6810 [details] patch I also looked through the rest of my checkin to make sure I didn't make the same mistake more than once. I'm not sure how I would create an automated test for this.
John Sullivan
Comment 12 2006-03-02 13:00:15 PST
Comment on attachment 6810 [details] patch Seems OK to me to check in without a layout test, but maybe someone else will have an idea about how to create one.
Justin Garcia
Comment 13 2006-03-02 13:40:14 PST
landing with a layout test that mitz wrote
Note You need to log in before you can comment on or make changes to this bug.