[Webkit-unassigned] [Bug 7619] New: TimerBase::setNextFireTime removes a timer from the list of firing timers even when not changing its heap position

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=7619

           Summary: TimerBase::setNextFireTime removes a timer from the list
                    of firing timers even when not changing its heap
                    position
           Product: WebKit
           Version: 420+ (nightly)
          Platform: Macintosh
        OS/Version: Mac OS X 10.0
            Status: NEW
          Severity: normal
          Priority: P2
         Component: New Bugs
        AssignedTo: webkit-unassigned at opendarwin.org
        ReportedBy: ggaren at apple.com


If you set the fire time of a timer that's ready to fire, and the new time
matches the old time, doesn't this code *prevent* that timer from firing?

    if (timersReadyToFire)
        timersReadyToFire->remove(this);

    double oldTime = m_nextFireTime;
    if (oldTime != newTime) {

I think this is a question for Darin.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.