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

Unified Diff: base/trace_event/memory_dump_provider_info.h

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_dump_manager_unittest.cc ('k') | base/trace_event/memory_dump_provider_info.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/trace_event/memory_dump_provider_info.h
diff --git a/base/trace_event/memory_dump_provider_info.h b/base/trace_event/memory_dump_provider_info.h
index ca63a987b2c4398a3c734001b19c7de1c886f8b0..aeacea1255c7245c8901effa85163323c2377be3 100644
--- a/base/trace_event/memory_dump_provider_info.h
+++ b/base/trace_event/memory_dump_provider_info.h
@@ -58,7 +58,8 @@ struct BASE_EXPORT MemoryDumpProviderInfo
const char* name,
scoped_refptr<SequencedTaskRunner> task_runner,
const MemoryDumpProvider::Options& options,
- bool whitelisted_for_background_mode);
+ bool whitelisted_for_background_mode,
+ bool whitelisted_for_summary_mode);
// It is safe to access the const fields below from any thread as they are
// never mutated.
@@ -80,6 +81,9 @@ struct BASE_EXPORT MemoryDumpProviderInfo
// True if the dump provider is whitelisted for background mode.
const bool whitelisted_for_background_mode;
+ // True if the dump provider is whitelisted for summary mode.
+ const bool whitelisted_for_summary_mode;
+
// These fields below, instead, are not thread safe and can be mutated only:
// - On the |task_runner|, when not null (i.e. for thread-bound MDPS).
// - By the MDM's background thread (or in any other way that guarantees
« no previous file with comments | « base/trace_event/memory_dump_manager_unittest.cc ('k') | base/trace_event/memory_dump_provider_info.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698