846067 - Assertion: QueryInterface needed

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Fix

Steps to reproduce: Run a debug build.

Breakpoint 1, NS_DebugBreak_P (aSeverity=1, aStr=0x418d75dd "QueryInterface needed", aExpr=<value optimized out>, aFile=<value optimized out>, aLine=503) at /build/b2g18/mozilla/xpcom/base/nsDebugImpl.cpp:294
294          sevString = "###!!! ASSERTION";
(gdb) up
#1  0x411b5a10 in nsCOMPtr<nsITimerCallback>::Assert_NoQueryNeeded (this=0x43d15a28, rhs=<value optimized out>) at ../../dist/include/nsCOMPtr.h:503
503                   NS_ASSERTION(query_result.get() == mRawPtr, "QueryInterface needed");
(gdb) 
#2  nsCOMPtr<nsITimerCallback>::operator= (this=0x43d15a28, rhs=<value optimized out>) at ../../dist/include/nsCOMPtr.h:623
623               NSCAP_ASSERT_NO_QUERY_NEEDED();
(gdb) 
#3  0x411b5cee in nsTimerImpl::Fire (this=0x43d15a10) at /build/b2g18/mozilla/xpcom/threads/nsTimerImpl.cpp:458
458         mTimerCallbackWhileFiring = mCallback.i;
(gdb) p mCallback.i
$1 = (class nsITimerCallback *) 0x4326aa38
(gdb) x/a $
0x4326aa38:     0x41ea0bec <_ZTVN7mozilla3dom3ipc12_GLOBAL__N_122ProcessPriorityManagerE+76>

Comment 1

[Justin Lebar (not reading bugmail)]

Comment on attachment 719224 [details] [diff] [review]
Fix

Sigh, and thanks.  See also bug 845886.

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/534e5d2520fe

I'm not going to bother trying to get this on b2g18 per jst.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/534e5d2520fe