Bug 835686: Mark debug-only variable as DebugOnly in transportlayer.h. r=ekr
authorDaniel Holbert <dholbert@cs.stanford.edu>
Tue, 29 Jan 2013 09:01:11 -0800
changeset 120244 e182be8b7b8a18656e0fc777ec3c3d8b7bef9553
parent 120243 bf9886e0d35620c31ed2586a4a2fc8c316ddaa5e
child 120245 cd02b8d5653c459b1adedcfa916feba47485e8b6
push id24243
push userryanvm@gmail.com
push dateWed, 30 Jan 2013 00:49:21 +0000
treeherdermozilla-central@5c248ef0fe62 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersekr
bugs835686
milestone21.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 835686: Mark debug-only variable as DebugOnly in transportlayer.h. r=ekr
media/mtransport/transportlayer.h
--- a/media/mtransport/transportlayer.h
+++ b/media/mtransport/transportlayer.h
@@ -6,16 +6,17 @@
 
 // Original author: ekr@rtfm.com
 
 #ifndef transportlayer_h__
 #define transportlayer_h__
 
 #include "sigslot.h"
 
+#include "mozilla/DebugOnly.h"
 #include "mozilla/RefPtr.h"
 #include "nsCOMPtr.h"
 #include "nsIEventTarget.h"
 #include "nsThreadUtils.h"
 
 #include "m_cpp_utils.h"
 
 namespace mozilla {
@@ -60,17 +61,17 @@ class TransportLayer : public sigslot::h
   TransportLayer *downward() { return downward_; }
 
   // Dispatch a call onto our thread (or run on the same thread if
   // thread is not set). This is always synchronous.
   nsresult RunOnThread(nsIRunnable *event) {
     if (target_) {
       nsIThread *thr;
 
-      nsresult rv = NS_GetCurrentThread(&thr);
+      DebugOnly<nsresult> rv = NS_GetCurrentThread(&thr);
       MOZ_ASSERT(NS_SUCCEEDED(rv));
 
       if (target_ != thr) {
         return target_->Dispatch(event, NS_DISPATCH_SYNC);
       }
     }
 
     return event->Run();