diff options
| author | dank@chromium.org <dank@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-03 21:07:47 +0000 |
|---|---|---|
| committer | dank@chromium.org <dank@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-03 21:07:47 +0000 |
| commit | 7e0d664aad3baa2c8af840addc75e1cebae75702 (patch) | |
| tree | 24d17986c6796be6b3f5c522df0c58311d96bb82 /chrome/browser | |
| parent | 2947cdcd14aa838dafc8c248e91e594fa4b15520 (diff) | |
| download | chromium_src-7e0d664aad3baa2c8af840addc75e1cebae75702.zip chromium_src-7e0d664aad3baa2c8af840addc75e1cebae75702.tar.gz chromium_src-7e0d664aad3baa2c8af840addc75e1cebae75702.tar.bz2 | |
Make no-tcmalloc (really, non-base/allocator) builds work again,
(i.e. this is a plain vanilla build used when layers like base/allocator
are getting in the way of debugging)
and make sure they use msvcrt rather than libcmt
(libcmt is used to help shim malloc/free, but it gets
in the way of valgrind doing the same thing).
Sadly, this is now a gyp-time operation rather than a Configuration
option.
Had to remove hardcoded C prototype for _set_new_mode,
as that caused link errors.
Also add variables win_{release,debug}_{Optimization,RuntimeLibrary}
to let the valgrind build override those settings.
Fix calling convention on _set_new_mode to match the
one in <new.h>
BUG=none
TEST=build with ~/.gyp/include.gypi set as described in comment in common.gypi, gclient runhooks, do release build, verify all exe's and dll's linked against msvcrt dll
Review URL: http://codereview.chromium.org/455037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33719 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/memory_purger.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/memory_purger.cc b/chrome/browser/memory_purger.cc index b66f043..d230e45 100644 --- a/chrome/browser/memory_purger.cc +++ b/chrome/browser/memory_purger.cc @@ -128,7 +128,7 @@ void MemoryPurger::PurgeBrowser() { // * Purge AppCache memory. Not yet implemented sufficiently. // * Browser-side DatabaseTracker. Not implemented sufficiently. -#if defined(OS_WIN) +#if defined(OS_WIN) && defined(USE_TCMALLOC) // Tell tcmalloc to release any free pages it's still holding. // // TODO(pkasting): A lot of the above calls kick off actions on other threads. |