We can look it up in minimumDelay() instead and lose the parseAttribute() overload.
Created attachment 126753 [details] Patçh
Comment on attachment 126753 [details] Patçh stuff that matters indeed!
Committed r107579: <http://trac.webkit.org/changeset/107579>
Comment on attachment 126753 [details] Patçh View in context: https://bugs.webkit.org/attachment.cgi?id=126753&action=review > Source/WebCore/html/HTMLMarqueeElement.cpp:-39 > -const int defaultMinimumDelay = 60; Was it really an improvement to remove the named constant?
(In reply to comment #4) > (From update of attachment 126753 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=126753&action=review > > > Source/WebCore/html/HTMLMarqueeElement.cpp:-39 > > -const int defaultMinimumDelay = 60; > > Was it really an improvement to remove the named constant? In retrospect maybe not. I'll put it back if you like.