[tcmalloc]: Remove calls to __system_property_get in tcmalloc.

third_party/tcmalloc/chromium/src/base/commandlineflags.h

Index: third_party/tcmalloc/chromium/src/base/commandlineflags.h
diff --git a/third_party/tcmalloc/chromium/src/base/commandlineflags.h b/third_party/tcmalloc/chromium/src/base/commandlineflags.h
index 614be3a1a6bca90ebd621793f0c8c4033a5f7727..cfb03769082cb91ee09ce63f83de3c130d3bd7ae 100644
--- a/third_party/tcmalloc/chromium/src/base/commandlineflags.h
+++ b/third_party/tcmalloc/chromium/src/base/commandlineflags.h
@@ -54,10 +54,6 @@
 #include <stdlib.h>               // for getenv
 #include "base/basictypes.h"
 
-#if defined(__ANDROID__) || defined(ANDROID)
-#include <sys/system_properties.h>
-#endif
-
 #define DECLARE_VARIABLE(type, name)                                          \
   namespace FLAG__namespace_do_not_use_directly_use_DECLARE_##type##_instead {\
   extern PERFTOOLS_DLL_DECL type FLAGS_##name;                                \
@@ -115,52 +111,11 @@
 
 // These macros (could be functions, but I don't want to bother with a .cc
 // file), make it easier to initialize flags from the environment.
-// They are functions in Android because __system_property_get() doesn't
-// return a string.
 
 #if defined(ENABLE_PROFILING)
 
 #if defined(__ANDROID__) || defined(ANDROID)
-
-// Returns a pointer to a static variable.  The string pointed by the returned
-// pointer must not be modified.
-inline const char* const EnvToString(const char* envname, const char* dflt) {
-  static char system_property_value[PROP_VALUE_MAX];
-  if (__system_property_get(envname, system_property_value) > 0)
-    return system_property_value;
-  return dflt;
-}
-
-inline bool EnvToBool(const char* envname, bool dflt) {
-  static const char kTrueValues[] = "tTyY1";
-  char system_property_value[PROP_VALUE_MAX];
-  if (__system_property_get(envname, system_property_value) > 0)
-    return memchr(kTrueValues, system_property_value[0], sizeof(kTrueValues));
-  return dflt;
-}
-
-inline int EnvToInt(const char* envname, int dflt) {
-  char system_property_value[PROP_VALUE_MAX];
-  if (__system_property_get(envname, system_property_value) > 0)
-    return strtol(system_property_value, NULL, 10);
-  return dflt;
-}
-
-inline int64 EnvToInt64(const char* envname, int64 dflt) {
-  char system_property_value[PROP_VALUE_MAX];
-  if (__system_property_get(envname, system_property_value) > 0)
-    return strtoll(system_property_value, NULL, 10);
-  return dflt;
-}
-
-inline double EnvToDouble(const char* envname, double dflt) {
-  char system_property_value[PROP_VALUE_MAX];
-  if (__system_property_get(envname, system_property_value) > 0)
-    return strtod(system_property_value, NULL);
-  return dflt;
-}
-
-#else  // defined(__ANDROID__) || defined(ANDROID)
+#error TCMalloc does not support profiling on Android

[jar (doing other things), 2015/01/07 19:39:57]

I'm missing the connection to TCMalloc.

You are removing calls to __system_property_get(), presumably because it may not
be available at some point.

You have changed the code to not compile this file for Android (in fact, you
can't compile it on Android).

Can you clarify the connection to TCMalloc, and explain why blocking compilation
of this file on Android is a good thing?

[jar (doing other things), 2015/01/07 20:28:20]

<DOH!>  I misread the file name (I stopped looking up the path after base...
src... chromium (which is about where I have the root of my tree!?!).  I now see
it was under third_party/tcmalloc.

One obvious question: If Android is not using TCMalloc, why is this even trying
to get compiled?  If it is not, why are you conditioning to preclude compilation
at this point??  Why not pull all mentions of Android?   .... and if you want it
to "sort of compile" (?) then why are you putting in a #error?

 
 #define EnvToString(envname, dflt)   \
   (!getenv(envname) ? (dflt) : getenv(envname))