WebKit: make #if DCHECK_IS_ON consistent in third_party/WebKit/Source/platform/

third_party/WebKit/Source/platform/heap/PersistentNode.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef PersistentNode_h
 #define PersistentNode_h
 
 #include "platform/PlatformExport.h"
 #include "platform/heap/ThreadState.h"
 #include "wtf/Allocator.h"
@@ skipping 112 matching lines @@
       ensurePersistentNodeSlots(self, trace);
     ASSERT(m_freeListHead);
     PersistentNode* node = m_freeListHead;
     m_freeListHead = m_freeListHead->freeListNext();
     node->initialize(self, trace);
     ASSERT(!node->isUnused());
     return node;
   }
 
   void freePersistentNode(PersistentNode* persistentNode) {
-    ASSERT(m_persistentCount > 0);
+#if DCHECK_IS_ON()
+    DCHECK_GT(m_persistentCount, 0);

[Alexander Alekseev, 2016/11/23 07:53:07]

This variable exists only if DCHECK_IS_ON() evaluates to true.

+#endif
     persistentNode->setFreeListNext(m_freeListHead);
     m_freeListHead = persistentNode;
 #if DCHECK_IS_ON()
     --m_persistentCount;
 #endif
   }
 
   static bool shouldTracePersistentNode(Visitor*, PersistentNode*) {
     return true;
   }
@@ skipping 99 matching lines @@
   // Recursive as prepareForThreadStateTermination() clears a PersistentNode's
   // associated Persistent<> -- it in turn freeing the PersistentNode. And both
   // CrossThreadPersistentRegion operations need a lock on the region before
   // mutating.
   RecursiveMutex m_mutex;
 };
 
 }  // namespace blink
 
 #endif