components/metrics: Add runtime memory leak detector

build/common.gypi

Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 70d2ec571f5181e5207aa9f2df232e9a5445eab8..ca4037d91231cdbe50a1ead9bb2e484f8bf46063 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -1364,6 +1364,9 @@
     # Profile without optimizing out stack frames when profiling==1.
     'profiling_full_stack_frames%': '0',
 
+    # Enable automatic leak detector.
+    'leak_detector%': 0,
+
     # And if we want to dump symbols for Breakpad-enabled builds.
     'linux_dump_symbols%': 0,
     # And if we want to strip the binary after dumping symbols.
@@ -2707,6 +2710,9 @@
       ['profiling==1', {
         'defines': ['ENABLE_PROFILING=1'],
       }],
+      ['leak_detector==1 and chromeos==1', {
+        'defines': ['ENABLE_LEAK_DETECTOR=1'],
+      }],
       ['remoting==1', {
         'defines': ['ENABLE_REMOTING=1'],
       }],
@@ -3815,6 +3821,12 @@
                   'release_optimize%': 's',
                 },
               }],
+              ['leak_detector==1 and chromeos==1', {
+                'cflags': [
+                  '-fno-omit-frame-pointer',
+                  '-g',

[Nico, 2015/07/16 19:31:24]

Including full debug symbols in all chromeos builds seems…ambitious to me. I
guess this is off by default so it's ok?

[Simon Que, 2015/07/16 21:15:08]

The '-g' isn't necessary; I'm getting rid of it.

+                ],
+              }],
               ['profiling==1', {
                 'cflags': [
                   '-fno-omit-frame-pointer',