64992 - windows timer handling is causing trouble

Status: VERIFIED FIXED

(Core Graveyard :: Embedding: APIs, defect)

Description

[Judson Valeski]

In the windows embedding case (mozilla/embedding/test/winEmbed), META refresh is 
broken because we're not processing low priority (idle) timers.

Adam Lock is providing a workaround in 
http://bugzilla.mozilla.org/show_bug.cgi?id=44120 , but what should really 
happen is that we handle this case internally (so embeddors don't have to 
explicitly service our timer queue on windows).

Comment 1

[Judson Valeski]

nominating 0.8. Also, http://bugzilla.mozilla.org/show_bug.cgi?id=58701, and 
http://bugzilla.mozilla.org/show_bug.cgi?id=44120 provide workarounds (though 
neither are optimal).

Comment 2

[Marek Z. Jeziorek]

Mozilla 0.8 builds started today. We would consider taking reviewed low risk 
fixes if they are available today (Wednesday, 7/Feb/01) or tomorrow (Thursday, 
8/Feb/01). Otherwise, please set the target milestone to Mozilla 0.9. Thanks.

Comment 3

[Michael Dunn]

Correction: Changing QA contact for the Embed API bugs to David Epstein.

Comment 4

[Stuart Parmenter]

i'm going to take this since it is fixed by my new timer impl.

Comment 5

[Blake Ross]

Doesn't look like this is getting fixed before the freeze tomorrow night.
Pushing out a milestone.  Please correct if I'm mistaken.

(Wasn't pav's timer stuff landed?)

Comment 6

[Stuart Parmenter]

.

Comment 7

[Stuart Parmenter]

since 78611 isn't going in for 0.9.7, over to 0.9.8

Comment 8

[Stuart Parmenter]

marking fixed now that 78611 has landed

Comment 9

[David Epstein]

OnIdle() no longer calls NS_DoIdleEmbeddingStuff() in mfcEmbed. TimerThread.cpp,
TimerThread.h, nsITimer.h, nsITimerCallback.h, nsTimerImpl.cpp, & nsTimerImpl.h
are included in /xpcom/threads in the 12/20/01 mozilla debug build.