Description[tracing] Fix MemoryDumpManager periodic scheduler and improve tests
This a cleanup CL in preparation of next changes to memory-infra.
- Add much more coverage to the TraceConfig-related tests as this is
very critical to avoid regressions in either the chrome://tracing
UI and telemetry w.r.t. periodic dumps.
- Refactor the tests to use sane and readable in-line expectations
instead of the awkward back and forths between test fixtures
and MockDumpProvvider.
- Minor fix in the loogic which schedules periodic dumps in the
MemoryDumpManager. If ligh_period=1ms and heavy_period=4ms,
the sequence sholud be HLLLHLLL, not HLLLLHLLLL as it was
before this change.
- Normalize the "virtual_size" column in the malloc dump provider.
BUG=518823, 499731
Committed: https://crrev.com/49b700716fb5d9a306237df1ed00e162d89b3a4c
Cr-Commit-Position: refs/heads/master@{#347964}
Patch Set 1 #
Total comments: 38
Patch Set 2 : Re petrcermak review #
Messages
Total messages: 10 (2 generated)
|