tree 2e6fbe85f38b41b8f5bda3e1428181982145dc1d
parent 10906a154860653a7a6ae7c94b30cd01af59c6c5
author Mao Huang <littlecvr@chromium.org> 1533073466 +0000
committer Commit Bot <commit-bot@chromium.org> 1533073466 +0000

tcmalloc: fix heap profiler to always append the process id to the heap
dump.

Original CLs:

- http://codereview.chromium.org/6532051

  Fix heap profiler to always append the process id to the heap dump.

  To run Chrome:
  HEAP_PROFILE_ALLOCATION_INTERVAL=1000000 HEAPPROFILE=heapprof
  out/Debug/chrome

  To analyze:
  pprof --text out/Debug/chrome  heapprof.21026.0039.heap
  pprof --gv out/Debug/chrome  heapprof.21026.0039.heap
  (pprof is slooow)
  Committed:
  http://src.chromium.org/viewvc/chrome?view=rev&revision=86273

- https://codereview.chromium.org/8957007

  Change the heap file names to be Cleanup'ed in
  third_party/tcmalloc/chromium.

  In spite of Chromium's changing heap file names dumped in
  heap-profiler.cc:DumpProfileLocked(), the file names cleaned-up in
  heap-profile-table.cc:HeapProfileTable::CleanupOldProfiles are not
  modified.

  (See the difference between
  third_party/tcmalloc/chromium/src/heap-profiler.cc and
  .../vendor/src/heap-profiler.cc)

  This patch is originally from http://codereview.chromium.org/7865021/.

  BUG=none
  TEST=none

  Committed:
  http://src.chromium.org/viewvc/chrome?view=rev&revision=114777

BUG=724399,b:70905156

Change-Id: Iab590a6a34f3e3922b0958460169e517accef7df
Reviewed-on: https://chromium-review.googlesource.com/1130782
Commit-Queue: Gabriel Marin <gmx@chromium.org>
Reviewed-by: Will Harris <wfh@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#579581}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: bf3721aec76107e6b45c766a025fbd0da99946dd