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

Unified Diff: base/trace_event/memory_peak_detector_unittest.cc

Issue 2876543002: memory-infra: Don't invoke all dump providers in SUMMARY_ONLY mode (Closed)
Patch Set: address comments Created 3 years, 7 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 | « base/trace_event/memory_infra_background_whitelist.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/trace_event/memory_peak_detector_unittest.cc
diff --git a/base/trace_event/memory_peak_detector_unittest.cc b/base/trace_event/memory_peak_detector_unittest.cc
index c158466d6ea565a10e85b267ff31bf76172cb1a9..1fdf60dadb7025ad4b0f03a5b07718b8756b7962 100644
--- a/base/trace_event/memory_peak_detector_unittest.cc
+++ b/base/trace_event/memory_peak_detector_unittest.cc
@@ -177,7 +177,9 @@ class MemoryPeakDetectorTest : public testing::Test {
MemoryDumpProvider::Options opt;
opt.is_fast_polling_supported = true;
scoped_refptr<MemoryDumpProviderInfo> mdp_info(
- new MemoryDumpProviderInfo(mdp.get(), "Mock MDP", nullptr, opt, false));
+ new MemoryDumpProviderInfo(mdp.get(), "Mock MDP", nullptr, opt,
+ false /* whitelisted_for_background_mode */,
+ false /* whitelisted_for_summary_mode */));
// The |mdp| instance will be destroyed together with the |mdp_info|.
mdp_info->owned_dump_provider = std::move(mdp);
« no previous file with comments | « base/trace_event/memory_infra_background_whitelist.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698