Bug 1372851 - Disable TraceLogger by default in opt builds. r=bbouvier

This commit is contained in:
Jan de Mooij 2017-06-14 13:04:23 +02:00
parent dcff4df9e6
commit 3b0f76327e
2 changed files with 9 additions and 5 deletions

View File

@ -1504,13 +1504,17 @@ AC_SUBST(MOZ_OPTIMIZE_LDFLAGS)
AC_SUBST(MOZ_PGO_OPTIMIZE_FLAGS)
dnl ========================================================
dnl = Disable trace logging
dnl = Trace logging. Enabled by default in DEBUG builds.
dnl ========================================================
ENABLE_TRACE_LOGGING=1
MOZ_ARG_DISABLE_BOOL(trace-logging,
[ --disable-trace-logging Disable trace logging],
MOZ_ARG_ENABLE_BOOL(trace-logging,
[ --enable-trace-logging Enable trace logging],
ENABLE_TRACE_LOGGING=1,
ENABLE_TRACE_LOGGING= )
if test -n "$MOZ_DEBUG"; then
ENABLE_TRACE_LOGGING=1
fi
AC_SUBST(ENABLE_TRACE_LOGGING)
if test -n "$ENABLE_TRACE_LOGGING"; then

View File

@ -5031,6 +5031,7 @@ Debugger::makeGlobalObjectReference(JSContext* cx, unsigned argc, Value* vp)
return dbg->wrapDebuggeeValue(cx, args.rval());
}
#ifdef JS_TRACE_LOGGING
static bool
DefineProperty(JSContext* cx, HandleObject obj, HandleId id, const char* value, size_t n)
{
@ -5042,7 +5043,6 @@ DefineProperty(JSContext* cx, HandleObject obj, HandleId id, const char* value,
return JS_DefinePropertyById(cx, obj, id, str, JSPROP_ENUMERATE);
}
#ifdef JS_TRACE_LOGGING
# ifdef NIGHTLY_BUILD
bool
Debugger::setupTraceLogger(JSContext* cx, unsigned argc, Value* vp)