[Webkit-unassigned] [Bug 64230] REGRESSION (r73385): Marquee with behavior="alternate" is not working

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=64230


Andy Estes <aestes at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #118783|review?                     |review-
               Flag|                            |




--- Comment #15 from Andy Estes <aestes at apple.com>  2011-12-15 21:37:26 PST ---
(From update of attachment 118783)
View in context: https://bugs.webkit.org/attachment.cgi?id=118783&action=review

I applied your patch on my machine and marquee-alternate.html fails when I run it. What guarantees that a cycle will complete in 4.6 seconds? Are you assuming the marquee is a certain width based on the text within it? That isn't a safe assumption.

> LayoutTests/fast/html/marquee-alternate.html:10
> +var fullCycle = 4600;

It's unfortunate that this test requires 4.6 seconds to run. You can shorten this time by using marquee's scrollAmount and scrollDelay properties.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.