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

Unified Diff: base/trace_event/memory_dump_manager_unittest.cc

Issue 2952693002: Revert of Change memory dumps to not use periodic dumps by default. (Closed)
Patch Set: Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | base/trace_event/trace_config.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/trace_event/memory_dump_manager_unittest.cc
diff --git a/base/trace_event/memory_dump_manager_unittest.cc b/base/trace_event/memory_dump_manager_unittest.cc
index fe1b2fe5ac8a1e9fd3102af8b4488a6cdce4d859..9806c5008ec4bd563b33b6e9278f90c5e5729872 100644
--- a/base/trace_event/memory_dump_manager_unittest.cc
+++ b/base/trace_event/memory_dump_manager_unittest.cc
@@ -980,17 +980,17 @@
.WillByDefault(Return());
// Enabling memory-infra with the legacy TraceConfig (category filter) in
- // a coordinator process should not enable periodic dumps.
- EnableTracingWithLegacyCategories(MemoryDumpManager::kTraceCategory);
- EXPECT_FALSE(IsPeriodicDumpingEnabled());
+ // a coordinator process should enable periodic dumps.
+ EnableTracingWithLegacyCategories(MemoryDumpManager::kTraceCategory);
+ EXPECT_TRUE(IsPeriodicDumpingEnabled());
DisableTracing();
// Enabling memory-infra with the new (JSON) TraceConfig in a coordinator
- // process while specifying a "memory_dump_config" section should enable
+ // process without specifying any "memory_dump_config" section should enable
// periodic dumps. This is to preserve the behavior chrome://tracing UI, that
- // is: ticking memory-infra should dump periodically with an explicit config.
+ // is: ticking memory-infra should dump periodically with the default config.
EnableTracingWithTraceConfig(
- TraceConfigMemoryTestUtil::GetTraceConfig_PeriodicTriggers(100, 5));
+ TraceConfigMemoryTestUtil::GetTraceConfig_NoTriggers());
EXPECT_TRUE(IsPeriodicDumpingEnabled());
DisableTracing();
« no previous file with comments | « no previous file | base/trace_event/trace_config.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698