Allow assertions to be enabled in Blink Release builds.

Source/bindings/core/v8/V8PerIsolateData.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 25 matching lines @@
 #include "bindings/core/v8/V8RecursionScope.h"
 #include "bindings/core/v8/V8ScriptRunner.h"
 #include "core/frame/UseCounter.h"
 #include "public/platform/Platform.h"
 #include "wtf/MainThread.h"
 
 namespace WebCore {
 
 static V8PerIsolateData* mainThreadPerIsolateData = 0;
 
-#ifndef NDEBUG
+#if ENABLE(ASSERT)
 static void assertV8RecursionScope()
 {
     ASSERT(V8RecursionScope::properlyUsed(v8::Isolate::GetCurrent()));
 }
 #endif
 
 static void useCounterCallback(v8::Isolate* isolate, v8::Isolate::UseCounterFeature feature)
 {
     switch (feature) {
     case v8::Isolate::kUseAsm:
         UseCounter::count(currentExecutionContext(isolate), UseCounter::UseAsm);
         break;
     default:
         ASSERT_NOT_REACHED();
     }
 }
 
 V8PerIsolateData::V8PerIsolateData(v8::Isolate* isolate)
     : m_isolate(isolate)
     , m_isolateHolder(adoptPtr(new gin::IsolateHolder(m_isolate, v8ArrayBufferAllocator())))
     , m_stringCache(adoptPtr(new StringCache(m_isolate)))
     , m_hiddenValue(adoptPtr(new V8HiddenValue()))
     , m_constructorMode(ConstructorMode::CreateNewObject)
     , m_recursionLevel(0)
     , m_isHandlingRecursionLevelError(false)
-#ifndef NDEBUG
+#if ENABLE(ASSERT)
     , m_internalScriptRecursionLevel(0)
 #endif
     , m_gcEventData(adoptPtr(new GCEventData()))
     , m_performingMicrotaskCheckpoint(false)
 {
-#ifndef NDEBUG
+#if ENABLE(ASSERT)
     // currentThread will always be non-null in production, but can be null in Chromium unit tests.
     if (blink::Platform::current()->currentThread())
         isolate->AddCallCompletedCallback(&assertV8RecursionScope);
 #endif
     if (isMainThread()) {
         mainThreadPerIsolateData = this;
         PageScriptDebugServer::setMainThreadIsolate(isolate);
     }
     isolate->SetUseCounterCallback(&useCounterCallback);
 }
@@ skipping 24 matching lines @@
 
 v8::Persistent<v8::Value>& V8PerIsolateData::ensureLiveRoot()
 {
     if (m_liveRoot.isEmpty())
         m_liveRoot.set(m_isolate, v8::Null(m_isolate));
     return m_liveRoot.getUnsafe();
 }
 
 void V8PerIsolateData::dispose(v8::Isolate* isolate)
 {
-#ifndef NDEBUG
+#if ENABLE(ASSERT)
     if (blink::Platform::current()->currentThread())
         isolate->RemoveCallCompletedCallback(&assertV8RecursionScope);
 #endif
     void* data = isolate->GetData(gin::kEmbedderBlink);
     delete static_cast<V8PerIsolateData*>(data);
     isolate->SetData(gin::kEmbedderBlink, 0);
 }
 
 V8PerIsolateData::DOMTemplateMap& V8PerIsolateData::currentDOMTemplateMap()
 {
@@ skipping 90 matching lines @@
 }
 
 v8::Handle<v8::FunctionTemplate> V8PerIsolateData::toStringTemplate()
 {
     if (m_toStringTemplate.isEmpty())
         m_toStringTemplate.set(m_isolate, v8::FunctionTemplate::New(m_isolate, constructorOfToString));
     return m_toStringTemplate.newLocal(m_isolate);
 }
 
 } // namespace WebCore