Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(684)

Issue 10855010: Reverting this to see if it fixes the redness on the ChromiumOS bots. (Closed)

Created:
8 years, 4 months ago by ananta
Modified:
8 years, 4 months ago
CC:
chromium-reviews, mihaip-chromium-reviews_chromium.org, jam, joi+watch-content_chromium.org, Aaron Boodman, pam+watch_chromium.org, darin-cc_chromium.org
Visibility:
Public.

Description

Reverting this to see if it fixes the redness on the ChromiumOS bots. Revert 150135 - Enable memory profiler on Linux when TC_MALLOC is enabled. Added a tc_malloc memory extension to get bytes allocated on current thread (GetBytesAllocatedOnCurrentThread API call). R=jam@chromium.org,jar@chromium.org BUG=139667 TEST=test about:profiler after setting the env variable CHROME_PROFILER_TIME=1. In about:profiler, it will show the memory allocated per thread. This change impacts linux only. Review URL: https://chromiumcodereview.appspot.com/10820063 TBR=rtenneti@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=150142

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -25 lines) Patch
M content/app/content_main_runner.cc View 3 chunks +0 lines, -14 lines 0 comments Download
M third_party/tcmalloc/chromium/src/gperftools/malloc_extension.h View 2 chunks +1 line, -5 lines 0 comments Download
M third_party/tcmalloc/chromium/src/malloc_extension.cc View 3 chunks +1 line, -6 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
ananta
8 years, 4 months ago (2012-08-06 20:34:38 UTC) #1
ramant (doing other things)
8 years, 4 months ago (2012-08-06 20:58:51 UTC) #2
LGTM.

Powered by Google App Engine
This is Rietveld 408576698