Warn about extractTypeNameFromFunctionName usage

Source/wtf/InstanceCounter.h

 /*
  * Copyright (C) 2013 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
@@ skipping 16 matching lines @@
 #define InstanceCounter_h
 
 #include "wtf/Compiler.h"
 #include "wtf/WTFExport.h"
 
 namespace WTF {
 
 class String;
 WTF_EXPORT String dumpRefCountedInstanceCounts();
 
+// Warning: Using extractTypeNameFromFunctionName bloats binary size.

[tkent, 2015/09/01 01:36:15]

Can we warp the function with ENABLE(ASSERT) || ENABLE(GC_PROFILING)?

+// This function should be only used for debug and profiling purposes,
+// and should be never enabled on release builds.
 WTF_EXPORT String extractTypeNameFromFunctionName(const char* funcName);
 
 template<typename T>
 inline const char* extractNameFunction()
 {
     return WTF_PRETTY_FUNCTION;
 }
 
 #if ENABLE(INSTANCE_COUNTER) || ENABLE(GC_PROFILING)
 WTF_EXPORT void incrementInstanceCount(const char* extractNameFunctionName, void* ptr);
 WTF_EXPORT void decrementInstanceCount(const char* extractNameFunctionName, void* ptr);
 
 template<typename T>
 inline void incrementInstanceCount(T* p)
 {
     incrementInstanceCount(extractNameFunction<T>(), p);
 }
 
 template<typename T>
 inline void decrementInstanceCount(T* p)
 {
     decrementInstanceCount(extractNameFunction<T>(), p);
 }
 
 #endif // ENABLE(INSTANCE_COUNTER) || ENABLE(GC_PROFILING)
 
 } // namespace WTF
 
 #endif