Deprecate some macros in wtf/Assertions.h in favor of base/logging.h

third_party/WebKit/Source/wtf/HashTable.h

Index: third_party/WebKit/Source/wtf/HashTable.h
diff --git a/third_party/WebKit/Source/wtf/HashTable.h b/third_party/WebKit/Source/wtf/HashTable.h
index 373dbda75bb434f7c78da8374f61bc68e46cc688..aad25e7c29840c7cec0016c97746ae89fe419eee 100644
--- a/third_party/WebKit/Source/wtf/HashTable.h
+++ b/third_party/WebKit/Source/wtf/HashTable.h
@@ -287,7 +287,8 @@ template <typename HashTableType, typename ValueType> struct HashTableAddResult
         , m_containerModifications(container->modifications())
 #endif
     {
-        ASSERT_UNUSED(container, container);
+        ALLOW_UNUSED_LOCAL(container);

[Nico, 2016/03/18 14:34:25]

Is this needed? container is used (assigned to m_container), and it's just a
parameter, not a local

[Nico, 2016/03/22 02:56:21]

(Did you see this?)

[tkent, 2016/03/22 03:04:21]

It's used only if ENABLE(SECURITY_ASSERT).
I remember a unused parameter caused a build error in WebKit.  It might not be
same in Chromium/Blink.

+        DCHECK(container);
     }
 
     ValueType* storedValue;