Bug 7542

Summary:

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

Product:

WebKit

Reporter:

Steven Fisher <sdfisher>

Component:

Layout and Rendering

Assignee:

Justin Garcia <justin.garcia>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

mitz

Priority:

Keywords:

Regression

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

URL:

http://en.wikipedia.org/w/index.php?title=Quicksilver_%28software%29&oldid=40652040

Attachments:

Description	Flags
System profile	none
Screenshot	none
Broken bullets; some are "thinner"	none
patch	sullivan: review+

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	Details
Screenshot (355.80 KB, image/png) 2006-03-01 14:34 PST, Steven Fisher	no flags	Details
Broken bullets; some are "thinner" (39.37 KB, image/png) 2006-03-01 22:18 PST, crastinator	no flags	Details
patch (506 bytes, patch) 2006-03-02 12:21 PST, Justin Garcia	sullivan: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

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.