Bug 55417

Summary:

RunLoop::Timer fires 1000x too early on Windows

Product:

WebKit

Reporter:

Adam Roben (:aroben) <aroben>

Component:

Platform

Assignee:

Adam Roben (:aroben) <aroben>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

andersca

Priority:

Keywords:

InRadar, PlatformOnly

Version:

528+ (Nightly build)

Hardware:

OS:

Windows XP

Bug Depends on:

Bug Blocks:

51262

Attachments:

Description	Flags
Convert seconds to milliseconds before calling ::SetTimer	andersca: review+

Description Adam Roben (:aroben) 2011-02-28 15:18:51 PST

RunLoop::Timer fires 1000x too early on Windows. We're making a seconds vs. milliseconds conversion error. Right now this only affects the responsiveness timer (all other timers are 0-delay).

Comment 1 Adam Roben (:aroben) 2011-02-28 15:20:07 PST

This is blocking bug 51262, which wants to add a new non-0-delay timer.

Comment 2 Adam Roben (:aroben) 2011-02-28 15:20:39 PST

<rdar://problem/9065287>

Comment 3 Adam Roben (:aroben) 2011-03-01 07:25:38 PST

Created attachment 84228 [details]
Convert seconds to milliseconds before calling ::SetTimer

Comment 4 Adam Roben (:aroben) 2011-03-01 07:30:27 PST

Committed r80001: <http://trac.webkit.org/changeset/80001>