Bug 877584: Add missing #ifdef DEBUG. r=me
authorKyle Huey <khuey@kylehuey.com>
Sun, 09 Jun 2013 16:08:19 -0700
changeset 134466 94fb66d82988800685cd2bb3c6d257aa8c1bfd63
parent 134465 5b76afd813299c484672027d9064b9f854ddd347
child 134467 8543100834216b417584be93d80b1f61eac213ef
push id29227
push userkhuey@mozilla.com
push dateSun, 09 Jun 2013 23:08:35 +0000
treeherdermozilla-inbound@94fb66d82988 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs877584
milestone24.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 877584: Add missing #ifdef DEBUG. r=me
js/xpconnect/src/XPCJSRuntime.cpp
--- a/js/xpconnect/src/XPCJSRuntime.cpp
+++ b/js/xpconnect/src/XPCJSRuntime.cpp
@@ -363,21 +363,23 @@ XPCJSRuntime::RemoveJSHolder(void* aHold
 #endif
     bool hadOne = mJSHolders.Count() == 1;
     mJSHolders.Remove(aHolder);
     if (hadOne && mJSHolders.Count() == 0) {
       nsLayoutStatics::Release();
     }
 }
 
+#ifdef DEBUG
 bool
 XPCJSRuntime::TestJSHolder(void* aHolder)
 {
     return mJSHolders.Get(aHolder, nullptr);
 }
+#endif
 
 // static
 void XPCJSRuntime::TraceBlackJS(JSTracer* trc, void* data)
 {
     XPCJSRuntime* self = (XPCJSRuntime*)data;
 
     // Skip this part if XPConnect is shutting down. We get into
     // bad locking problems with the thread iteration otherwise.