Chromium Code Reviews
Help | Chromium Project | Sign in
(1)

Issue 5687003: New heap profiler: add support for progress reporting and control. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 2 months ago by mnaganov (cr)
Modified:
3 years, 9 months ago
CC:
v8-dev
Visibility:
Public.

Description

New heap profiler: add support for progress reporting and control. As taking a snapshot of a large heap takes noticeable time, it's good to be able to monitor and control it. The change itself is small, big code deletes and additions are in fact moves. The only significant change is simplification of approximated retained sizes calculation algorithm. Committed: http://code.google.com/p/v8/source/detail?r=5978

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+347 lines, -205 lines) Patch
M include/v8.h View 1 chunk +18 lines, -0 lines 0 comments Download
M include/v8-profiler.h View 2 chunks +2 lines, -2 lines 0 comments Download
M src/api.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M src/heap-profiler.h View 2 chunks +12 lines, -4 lines 0 comments Download
M src/heap-profiler.cc View 2 chunks +23 lines, -10 lines 0 comments Download
M src/profile-generator.h View 10 chunks +21 lines, -20 lines 0 comments Download
M src/profile-generator.cc View 7 chunks +207 lines, -166 lines 0 comments Download
M src/profile-generator-inl.h View 2 chunks +13 lines, -1 line 0 comments Download
M test/cctest/test-heap-profiler.cc View 1 chunk +47 lines, -0 lines 0 comments Download
Trybot results:
Commit:

Messages

Total messages: 2 (0 generated)
mnaganov (cr)
4 years, 2 months ago (2010-12-10 14:51:32 UTC) #1
Søren Thygesen Gjesse
4 years, 2 months ago (2010-12-13 07:52:23 UTC) #2
LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld dd99357-tainted