REGRESSION: (r13028) Scrolling causes incomplete drawing of <ul> bullets

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Steven Fisher]

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.

[Steven Fisher]

Also reproduced using scroll bars instead of scroll wheel.

[Steven Fisher]

Created attachment 6795 [details]
System profile

[Steven Fisher]

Created attachment 6796 [details]
Screenshot

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

Created attachment 6803 [details]
Broken bullets; some are "thinner"

Looks like "the regression version" of bug 4855.

[Steven Fisher]

#6: Well, I can't recreate this one (7542) on the shipping Safari. Doesn't that make this a regression, or have I erred?

(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]

Thanks for the explanation. Sorry if I seemed offended or something, I wasn't. I was just trying not to ramble. :)

[Justin Garcia]

Here it is:
+    IntRect box(_tx + m_x, _ty + m_y, m_height, m_width);
Oops!

[Justin Garcia]

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 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]

landing with a layout test that mitz wrote