[memory-infra] Trigger first periodic dump after initial period

base/trace_event/memory_dump_scheduler.cc

Index: base/trace_event/memory_dump_scheduler.cc
diff --git a/base/trace_event/memory_dump_scheduler.cc b/base/trace_event/memory_dump_scheduler.cc
index 17921959d38fe7275a37c73341569a8ba1622548..d436afd208ca8a0dc291bca4eae39492553690d2 100644
--- a/base/trace_event/memory_dump_scheduler.cc
+++ b/base/trace_event/memory_dump_scheduler.cc
@@ -72,9 +72,12 @@ void MemoryDumpScheduler::StartInternal(MemoryDumpScheduler::Config config) {
   tick_count_ = 0;
   light_dump_rate_ = light_dump_period_ms / min_period_ms;
   heavy_dump_rate_ = heavy_dump_period_ms / min_period_ms;
-  SequencedTaskRunnerHandle::Get()->PostTask(
+  // Keep the legacy behavior of periodic dumps where the first dump is

[Primiano Tucci (use gerrit), 2017/04/13 09:48:28]

I'd reword this comment to say:
(also add a +blank line before)
// For startup tracing we expect the first dump to happen after |period_ms| and
not immediatley

saying "legacy behavior" is really confusing because the legacy code has already
gone :)
This code is now the new legacy :P

+  // triggered after |period_ms_|.
+  SequencedTaskRunnerHandle::Get()->PostDelayedTask(
       FROM_HERE,
-      BindOnce(&MemoryDumpScheduler::Tick, Unretained(this), ++generation_));
+      BindOnce(&MemoryDumpScheduler::Tick, Unretained(this), ++generation_),
+      TimeDelta::FromMilliseconds(period_ms_));
 }
 
 void MemoryDumpScheduler::StopInternal() {