[Webkit-unassigned] [Bug 37308] [Chromium] Support HTML5 <progress> element on Windows.

[bugzilla-daemon at webkit.org]

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


TAMURA, Kent <tkent at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tkent at chromium.org




--- Comment #8 from TAMURA, Kent <tkent at chromium.org>  2010-04-20 18:24:44 PST ---
(In reply to comment #7)
> Hi, thank you for reviewing. I'm sorry for my slow response.
> 
> > > +#if ENABLE(PROGRESS_TAG)
> > > +
> > > +static const double progressAnimationFrameRate = 0.033;
> > > +static const double progressAnimationNumFrames = 60;
> > 
> > How did you decide these values?  Do they match to Windows' behavior?
> I couldn't found "standard" behaviour for windows.
> The guideline doesn't mention marquee speed,
> (http://msdn.microsoft.com/en-us/library/aa511258.aspx)
> Nor default value of ProgressBar class on .NET framework looks unrealistic. 

So, please add a comment about why you chose 0.033 and 60, and add FIXME if you
think we need to improve them.

BTW,
http://msdn.microsoft.com/en-us/library/bb760842(v=VS.85).aspx
It seems the default interval is 30 msec.  So progressAnimationFrameRate=0.033
is reasonable.

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