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

Issue 2650863003: New heap dump format.

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months ago by DmitrySkiba
Modified:
1 week, 6 days ago
Reviewers:
Primiano Tucci, lpy, hjd, ssid
CC:
chromium-reviews, oilpan-reviews, Mads Ager (chromium), vmpstr+watch_chromium.org, wfh+watch_chromium.org, haraken, blink-reviews, tracing+reviews_chromium.org, kinuko+watch, kouhei+heap_chromium.org, erikchen
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

New heap dump format. TODO: unittests BUG=

Patch Set 1 #

Patch Set 2 : Add 'version' field. #

Total comments: 17

Patch Set 3 : Rebase #

Patch Set 4 : Address comments #

Patch Set 5 : Add StringDeduplicator, consolidate exporters #

Patch Set 6 : Fix unittests; describe format #

Patch Set 7 : DCHECK for continuous mode #

Total comments: 50

Patch Set 8 : Fix StringDeduplicator::Insert #

Total comments: 13

Patch Set 9 : Address comments #

Total comments: 7

Patch Set 10 : Address comments (heaps_v2) #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1179 lines, -1113 lines) Patch
M base/BUILD.gn View 1 2 3 4 5 6 7 2 chunks +6 lines, -3 lines 0 comments Download
A base/trace_event/heap_profiler_event_writer.h View 1 2 3 4 5 6 7 8 1 chunk +104 lines, -0 lines 0 comments Download
A base/trace_event/heap_profiler_event_writer.cc View 1 2 3 4 5 6 7 8 1 chunk +133 lines, -0 lines 0 comments Download
A base/trace_event/heap_profiler_event_writer_unittest.cc View 1 2 3 4 5 6 7 8 1 chunk +291 lines, -0 lines 0 comments Download
M base/trace_event/heap_profiler_heap_dump_writer.h View 1 2 3 4 1 chunk +0 lines, -113 lines 0 comments Download
M base/trace_event/heap_profiler_heap_dump_writer.cc View 1 2 3 4 1 chunk +0 lines, -322 lines 0 comments Download
M base/trace_event/heap_profiler_heap_dump_writer_unittest.cc View 1 2 3 4 1 chunk +0 lines, -333 lines 0 comments Download
M base/trace_event/heap_profiler_stack_frame_deduplicator.h View 1 2 3 4 5 6 7 8 5 chunks +17 lines, -10 lines 0 comments Download
M base/trace_event/heap_profiler_stack_frame_deduplicator.cc View 1 2 3 4 5 6 7 8 3 chunks +31 lines, -32 lines 0 comments Download
M base/trace_event/heap_profiler_stack_frame_deduplicator_unittest.cc View 1 2 3 4 5 6 7 8 5 chunks +163 lines, -47 lines 0 comments Download
A base/trace_event/heap_profiler_string_deduplicator.h View 1 2 3 4 5 6 7 8 1 chunk +50 lines, -0 lines 0 comments Download
A base/trace_event/heap_profiler_string_deduplicator.cc View 1 2 3 4 5 6 7 8 1 chunk +58 lines, -0 lines 0 comments Download
A base/trace_event/heap_profiler_string_deduplicator_unittest.cc View 1 2 3 4 5 6 7 8 1 chunk +124 lines, -0 lines 0 comments Download
M base/trace_event/heap_profiler_type_name_deduplicator.h View 1 2 3 4 5 6 7 8 1 chunk +22 lines, -10 lines 0 comments Download
M base/trace_event/heap_profiler_type_name_deduplicator.cc View 1 2 3 4 5 6 7 8 3 chunks +21 lines, -27 lines 0 comments Download
M base/trace_event/heap_profiler_type_name_deduplicator_unittest.cc View 1 2 3 4 5 6 7 8 3 chunks +96 lines, -51 lines 0 comments Download
M base/trace_event/malloc_dump_provider.cc View 1 2 3 4 5 6 7 2 chunks +5 lines, -17 lines 0 comments Download
M base/trace_event/memory_dump_manager.cc View 1 2 3 4 5 6 7 8 9 3 chunks +6 lines, -49 lines 0 comments Download
M base/trace_event/memory_dump_session_state.h View 1 2 3 4 5 6 7 8 2 chunks +10 lines, -5 lines 0 comments Download
M base/trace_event/memory_dump_session_state.cc View 1 2 3 4 5 2 chunks +11 lines, -10 lines 0 comments Download
M base/trace_event/process_memory_dump.h View 1 2 3 chunks +3 lines, -4 lines 0 comments Download
M base/trace_event/process_memory_dump.cc View 1 2 3 4 5 6 7 8 9 3 chunks +12 lines, -11 lines 0 comments Download
M base/trace_event/process_memory_dump_unittest.cc View 1 2 3 4 5 6 7 8 9 3 chunks +11 lines, -12 lines 0 comments Download
M third_party/WebKit/Source/platform/PartitionAllocMemoryDumpProvider.cpp View 1 chunk +2 lines, -19 lines 0 comments Download
M third_party/WebKit/Source/platform/heap/BlinkGCMemoryDumpProvider.cpp View 1 chunk +2 lines, -18 lines 0 comments Download
M third_party/WebKit/Source/platform/instrumentation/tracing/web_process_memory_dump.h View 2 chunks +0 lines, -9 lines 0 comments Download
M third_party/WebKit/Source/platform/instrumentation/tracing/web_process_memory_dump.cc View 1 2 3 4 5 2 chunks +0 lines, -11 lines 0 comments Download
M tools/gn/bootstrap/bootstrap.py View 1 2 3 4 5 6 7 8 9 1 chunk +1 line, -0 lines 0 comments Download
Trybot results: Sign in to try more bots
Commit queue not available (can’t edit this change).

Messages

Total messages: 26 (5 generated)
DmitrySkiba
1 month, 4 weeks ago (2017-01-30 19:44:11 UTC) #2
DmitrySkiba
PTAL
1 month, 2 weeks ago (2017-02-14 18:31:15 UTC) #4
Primiano Tucci
On 2017/02/14 18:31:15, DmitrySkiba wrote: > PTAL sorry for the delay, manage to do *almost* ...
1 month, 1 week ago (2017-02-15 18:55:02 UTC) #5
Primiano Tucci
Thanks a lot for this. I'm very excited by the code being ditched here. Everything ...
1 month, 1 week ago (2017-02-17 17:07:05 UTC) #6
DmitrySkiba
Overall things TODO: 1. Tests 2. Optimizing format for size 3. Public format doc (use/extend ...
1 month ago (2017-02-23 07:17:19 UTC) #8
lpy
On 2017/02/23 07:17:19, DmitrySkiba wrote: > 3. Public format doc (use/extend Hector's, or add a ...
1 month ago (2017-02-23 07:23:42 UTC) #9
hjd
https://codereview.chromium.org/2650863003/diff/20001/base/trace_event/memory_dump_manager.cc File base/trace_event/memory_dump_manager.cc (right): https://codereview.chromium.org/2650863003/diff/20001/base/trace_event/memory_dump_manager.cc#newcode725 base/trace_event/memory_dump_manager.cc:725: heap_dump_value->SetInteger("version", 1); On 2017/02/23 07:17:19, DmitrySkiba wrote: > On ...
1 month ago (2017-02-23 12:33:28 UTC) #11
DmitrySkiba
PTAL Changes at a glance: 1. Added StringDeduplicator, and made both StackFrameDeduplicator / TypeNameDeduplicator depend ...
4 weeks, 1 day ago (2017-03-01 08:20:48 UTC) #12
DmitrySkiba
On 2017/03/01 08:20:48, DmitrySkiba wrote: > PTAL > > Changes at a glance: > > ...
4 weeks ago (2017-03-02 19:56:55 UTC) #13
hjd
On 2017/03/02 19:56:55, DmitrySkiba wrote: > On 2017/03/01 08:20:48, DmitrySkiba wrote: > > PTAL > ...
4 weeks ago (2017-03-02 19:58:43 UTC) #14
DmitrySkiba
PTAL. I added unittests and DCHECK for continuous mode.
3 weeks, 1 day ago (2017-03-07 22:45:09 UTC) #15
Primiano Tucci
Thanks! I looked through all the code (a bit less tests) and it looks all ...
3 weeks ago (2017-03-09 11:47:45 UTC) #17
ssid
I have 2 concerns: Does the trace size increase in pseudo mode because of string ...
2 weeks, 3 days ago (2017-03-13 03:38:01 UTC) #18
DmitrySkiba
https://codereview.chromium.org/2650863003/diff/120001/base/BUILD.gn File base/BUILD.gn (right): https://codereview.chromium.org/2650863003/diff/120001/base/BUILD.gn#newcode981 base/BUILD.gn:981: "trace_event/heap_profiler_string_deduplicator.cc", On 2017/03/09 11:47:44, Primiano Tucci (slow - perf) ...
2 weeks, 1 day ago (2017-03-14 22:12:48 UTC) #19
Primiano Tucci
Ok LGTM (plz fix the description though), so you can also get other owners approvalz. ...
2 weeks ago (2017-03-15 20:41:40 UTC) #20
ssid
lgtm, please look at the comments, that should be done in all dumps. https://codereview.chromium.org/2650863003/diff/140001/base/trace_event/heap_profiler_event_writer.cc File ...
2 weeks ago (2017-03-16 02:09:20 UTC) #21
Primiano Tucci
So I talked today with hjd: let's keep having the "heaps" as part of the ...
2 weeks ago (2017-03-16 14:23:56 UTC) #22
DmitrySkiba
https://codereview.chromium.org/2650863003/diff/160001/base/trace_event/process_memory_dump.cc File base/trace_event/process_memory_dump.cc (right): https://codereview.chromium.org/2650863003/diff/160001/base/trace_event/process_memory_dump.cc#newcode254 base/trace_event/process_memory_dump.cc:254: allocation_register.EstimateTraceMemoryOverhead(&overhead); On 2017/03/16 02:09:20, ssid wrote: > This part ...
1 week, 6 days ago (2017-03-17 04:01:38 UTC) #23
DmitrySkiba
On 2017/03/16 14:23:56, Primiano Tucci (slow - perf) wrote: > So I talked today with ...
1 week, 6 days ago (2017-03-17 04:03:03 UTC) #24
hjd
On 2017/03/17 04:03:03, DmitrySkiba wrote: > On 2017/03/16 14:23:56, Primiano Tucci (slow - perf) wrote: ...
1 week, 6 days ago (2017-03-17 12:24:26 UTC) #25
Primiano Tucci
1 week, 6 days ago (2017-03-17 19:00:30 UTC) #26
Great thanks both.
dskiba@ can you add the other OWNERS here? So as as soon as I review hjd patch
(monday) we'll land all the things \o/
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld cc6ac46