Bug 22096 - REGRESSION (r35879): scrolldelay is counted in seconds instead of miliseconds
Summary: REGRESSION (r35879): scrolldelay is counted in seconds instead of miliseconds
Status: VERIFIED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P1 Normal
Assignee: Nobody
URL: http://www.anamosa.k12.ia.us
Keywords: HasReduction, Regression
Depends on:
Blocks:
 
Reported: 2008-11-05 20:24 PST by Roy B. Carter
Modified: 2009-01-13 14:59 PST (History)
3 users (show)

See Also:

Attachments
minimal test case (93 bytes, text/html)
2008-12-27 12:02 PST, Robert Blaut 		no flags Details
fix for the issue (4.28 KB, patch)
2009-01-07 04:53 PST, Robert Blaut 		darin: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Roy B. Carter 2008-11-05 20:24:00 PST 
The scrolling banner headline for web page does not scroll using webkit r38068 or previous, does scroll using regular Safari v3.1.2 browser.
Comment 1 Robert Blaut 2008-12-27 12:02:11 PST 
I can confirm this bug as reggresion bug. 

Scrolldelay sets the amount of delay in milliseconds (a millisecond is 1/1000th of a second). Actually WebKit uses seconds instead of miliseconds.

Check attached test case. You can inspect the test case. Inspector shows:
 -webkit-marquee-speed: 10s; 
instead of 
-webkit-marquee-speed: 10ms;
Comment 2 Robert Blaut 2008-12-27 12:02:55 PST 
Created attachment 26271 [details]
minimal test case
Comment 3 Robert Blaut 2009-01-05 01:21:17 PST 
This changeset <http://trac.webkit.org/changeset/35879> caused regression.
Comment 4 Robert Blaut 2009-01-07 04:53:57 PST 
Created attachment 26490 [details]
fix for the issue
Comment 5 Darin Adler 2009-01-07 07:38:39 PST 
Comment on attachment 26490 [details]
fix for the issue

r=me
Comment 6 Brent Fulgham 2009-01-12 16:25:17 PST 
Landed in r39848.
Comment 7 Roy B. Carter 2009-01-13 14:59:11 PST 
It works on our home page! Thank you!