Reland: memory-infra: Stop using RequestGlobalDump in MemoryDumpManagerTest
This reverts commit ac66f89e4be521caa45a972b20e7890992901469.
Reason for reland: Fixed race in the test
Original change's description:
> Revert "memory-infra: Stop using RequestGlobalDump in MemoryDumpManagerTest"
>
> This reverts commit 16e1788871253de5241941eaeb0a5fdb34b3d21f.
>
> Reason for revert: MemoryDumpManagerTest.TestBackgroundTracingSetup fails under LSAN: https://luci-milo.appspot.com/buildbot/chromium.memory/Linux%20ASan%20LSan%20Tests%20%281%29/36339
>
> Original change's description:
> > memory-infra: Stop using RequestGlobalDump in MemoryDumpManagerTest
> >
> > This CL changes the memory-infra unittests to not depend on
> > the RequestGlobalDump() API. From a logical viewpoint,
> > the RequestGlobalDump() is not needed for an in-process-only
> > test like MemoryDumpManagerTest.
> > From a more practical viewpoint, this refactoring is required
> > in preparation of the next CLs that will remove
> > MemoryDumpManager::RequestGlobalDump() method and move all the
> > clients to directly use the service.
> >
> > This CL doesn't cause any behavior change in memory-infra.
> >
> > BUG=720352
> > TBR=thakis
> >
> > Change-Id: I82ee1bf94dab55798be727f0ae70a1fede6634ca
> > Reviewed-on: https://chromium-review.googlesource.com/525534
> > Reviewed-by: Primiano Tucci <primiano@chromium.org>
> > Reviewed-by: siddhartha sivakumar <ssid@chromium.org>
> > Reviewed-by: Helen Li <xunjieli@chromium.org>
> > Reviewed-by: Hector Dearman <hjd@chromium.org>
> > Commit-Queue: siddhartha sivakumar <ssid@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#477851}
>
> TBR=thakis@chromium.org,primiano@chromium.org,hjd@chromium.org,xunjieli@chromium.org,ssid@chromium.org
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> BUG=720352
>
> Change-Id: I7088bd3549c3a8a733d4b87226593209bd40c42e
> Reviewed-on: https://chromium-review.googlesource.com/528032
> Reviewed-by: Sam McNally <sammc@chromium.org>
> Commit-Queue: Sam McNally <sammc@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#477875}
TBR=thakis@chromium.org,jam@chromium.org,primiano@chromium.org,fmeawad@chromium.org,hjd@chromium.org,sammc@chromium.org,xunjieli@chromium.org,ssid@chromium.org,chromium-reviews@chromium.org
Change-Id: Ic6439b97024d65d19012133171249dddd2de8b88
Reviewed-on: https://chromium-review.googlesource.com/528098
Commit-Queue: Primiano Tucci <primiano@chromium.org>
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#477988}
7 files changed