Bug 22096 - REGRESSION (r35879): scrolldelay is counted in seconds instead of miliseconds
Summary: REGRESSION (r35879): scrolldelay is counted in seconds instead of miliseconds
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:
Reported: 2008-11-05 20:24 PST by Roy B. Carter
Modified: 2009-01-13 14:59 PST (History)
3 users (show)

See Also:

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

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

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!