Use system properties for heap profiler in Android instead of environment variables.

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 54bf94faf94833a2b50c5d4023467e0ae4be38a4..e12b6687ee1085b7069b7293f95b6502b8b8b433 100644
--- a/third_party/tcmalloc/chromium/src/base/commandlineflags.h
+++ b/third_party/tcmalloc/chromium/src/base/commandlineflags.h
@@ -54,6 +54,10 @@
 #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;                                \
@@ -112,6 +116,47 @@
 
 // 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(__ANDROID__) || defined(ANDROID)
+
+inline const char* EnvToString(const char* envname, const char* dflt) {
+  static char system_property_value[PROP_VALUE_MAX];

[bulach, 2013/05/01 17:55:30]

nit: s/static//

[Dai Mikurube (NOT FULLTIME), 2013/05/02 16:26:40]

It actually needs static since it returns the string.  (Returning a pointer to a
static variable is not so good, but it's a behavior similar to getenv.)

Added a comment.

+  if (__system_property_get(envname, system_property_value) > 0)
+    return system_property_value;
+  return dflt;
+}
+
+inline bool EnvToBool(const char* envname, bool dflt) {
+  char system_property_value[PROP_VALUE_MAX];

[bulach, 2013/05/01 17:55:30]

nit: const char kTrueValues[] = "tTyY1"; then sizeof(kTrueValues);

[Dai Mikurube (NOT FULLTIME), 2013/05/02 16:26:40]

Done.

+  if (__system_property_get(envname, system_property_value) > 0)
+    return memchr("tTyY1\0", system_property_value[0], 6);
+  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)
 
 #define EnvToString(envname, dflt)   \
   (!getenv(envname) ? (dflt) : getenv(envname))
@@ -128,4 +173,6 @@
 #define EnvToDouble(envname, dflt)  \
   (!getenv(envname) ? (dflt) : strtod(getenv(envname), NULL))
 
+#endif  // defined(__ANDROID__) || defined(ANDROID)
+
 #endif  // BASE_COMMANDLINEFLAGS_H_