1. Enable large object pointer offset check in release build.

base/allocator/allocator.gyp

 # 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.
 
 {
   'variables': {
     'jemalloc_dir': '../../third_party/jemalloc/chromium',
     'tcmalloc_dir': '../../third_party/tcmalloc/chromium',
   },
   'targets': [
@@ skipping 183 matching lines @@
         # jemalloc files
         '<(jemalloc_dir)/jemalloc.c',
         '<(jemalloc_dir)/jemalloc.h',
         '<(jemalloc_dir)/ql.h',
         '<(jemalloc_dir)/qr.h',
         '<(jemalloc_dir)/rb.h',
 
         'allocator_shim.cc',
         'allocator_shim.h',
         'generic_allocators.cc',
-        'win_allocator.cc',        
+        'win_allocator.cc',
       ],
       # sources! means that these are not compiled directly.
       'sources!': [
         # Included by allocator_shim.cc for maximal inlining.
         'generic_allocators.cc',
         'win_allocator.cc',
 
         # We simply don't use these, but list them above so that IDE
         # users can view the full available source for reference, etc.
         '<(tcmalloc_dir)/src/addressmap-inl.h',
@@ skipping 206 matching lines @@
             '<(tcmalloc_dir)/src/heap-checker.cc',
           ],
           # Disable the heap checker in tcmalloc.
           'defines': [
             'NO_HEAP_CHECK',
           ],
         }],
       ],
     },
     {
-      # This library is linked in to libbase and allocator_unittests.
-      # It can't depend on either and nothing else should depend on it -
-      # all other code should use the interfaced provided by libbase.
+      # This library is linked in to src/base.gypi:base, allocator_unittests and
+      # tcmalloc_unittests. It can't depend on either and nothing else should
+      # depend on it - all other code should use the interfaced provided by
+      # base.
       'target_name': 'allocator_extension_thunks',
       'type': 'static_library',
       'sources': [
         'allocator_extension_thunks.cc',
         'allocator_extension_thunks.h',
       ],
       'toolsets': ['host', 'target'],
       'include_dirs': [
         '../../'
       ],
@@ skipping 52 matching lines @@
           'toolsets': ['host', 'target'],
           'include_dirs': [
             '../../'
           ],
           'configurations': {
             'Common_Base': {
               'msvs_target_platform': 'x64',
             },
           },
         },
+      {
+        'target_name': 'tcmalloc_unittest',
+        'type': 'executable',
+        'sources': [
+          'tcmalloc_unittest.cc',
+
+          # Part of chromium code(instead of TCMalloc). We don't include the

[gpike, 2012/05/21 19:04:58]

space before open paren

+          # TCMalloc version(atomicops-internals-x86.cc) in allocator library,
+          # so need this for replacement.
+          '../atomicops_internals_x86_gcc.cc',

[gpike, 2012/05/21 19:04:58]

This is gcc specific?  What happens on Windows/Mac?

[kaiwang, 2012/05/21 22:21:42]

This is removed since this is now depending on base

+
+          # Code allocator_shim.cc depends on.
+          '../profiler/alternate_timer.cc',
+          '../profiler/alternate_timer.h',
+        ],
+        'include_dirs': [
+          # This is tricky.. We add this include dir so alternate_timer.cc will

[gpike, 2012/05/21 19:04:58]

.. should be .

[kaiwang, 2012/05/21 22:21:42]

? which ..?

+          # use TCMalloc's logging.h, instead of the chromium base/logging.h as
+          # in other build targets.
+          '<(tcmalloc_dir)/src',
+
+          '../..',
+        ],
+        'dependencies': [
+          'allocator',
+          'allocator_extension_thunks',
+          '../../testing/gtest.gyp:gtest',
+        ],
+      },
       ],
     }],
   ],
 }