components/metrics: Add runtime memory leak detector

build/common.gypi

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # IMPORTANT:
 # Please don't directly include this file if you are building via gyp_chromium,
 # since gyp_chromium is automatically forcing its inclusion.
 {
   # Variables expected to be overriden on the GYP command line (-D) or by
   # ~/.gyp/include.gypi.
@@ skipping 1346 matching lines @@
 
     # The default type of gtest.
     'gtest_target_type%': 'executable',
 
     # Enable sampling based profiler.
     # See http://google-perftools.googlecode.com/svn/trunk/doc/cpuprofile.html
     'profiling%': '0',
     # 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.
     'linux_strip_binary%': 0,
     # If we want stack unwind support for backtrace().
     'debug_unwind_tables%': 1,
     'release_unwind_tables%': 1,
 
     # Override where to find binutils
     'binutils_version%': 0,
@@ skipping 1323 matching lines @@
       }],
       ['enable_pre_sync_backup==1', {
         'defines': ['ENABLE_PRE_SYNC_BACKUP'],
       }],
       ['image_loader_extension==1', {
         'defines': ['IMAGE_LOADER_EXTENSION=1'],
       }],
       ['profiling==1', {
         'defines': ['ENABLE_PROFILING=1'],
       }],
+      ['leak_detector==1 and chromeos==1', {
+        'defines': ['ENABLE_LEAK_DETECTOR=1'],
+      }],
       ['remoting==1', {
         'defines': ['ENABLE_REMOTING=1'],
       }],
       ['enable_webrtc==1', {
         'defines': ['ENABLE_WEBRTC=1'],
       }],
       ['enable_media_router==1', {
         'defines': ['ENABLE_MEDIA_ROUTER=1'],
       }],
       ['proprietary_codecs==1', {
@@ skipping 1088 matching lines @@
                 'ldflags': [
                   # Warn in case of text relocations.
                   '-Wl,--warn-shared-textrel',
                 ],
               }],
               ['OS=="android"', {
                 'variables': {
                   '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',
                   '-g',
                 ],
                 'conditions' : [
                   ['profiling_full_stack_frames==1', {
                     'cflags': [
                       '-fno-inline',
                       '-fno-optimize-sibling-calls',
@@ skipping 2369 matching lines @@
     # settings in target dicts.  SYMROOT is a special case, because many other
     # Xcode variables depend on it, including variables such as
     # PROJECT_DERIVED_FILE_DIR.  When a source group corresponding to something
     # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
     # files to appear (when present) in the UI as actual files and not red
     # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
     # and therefore SYMROOT, needs to be set at the project level.
     'SYMROOT': '<(DEPTH)/xcodebuild',
   },
 }