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

Unified Diff: services/resource_coordinator/memory/coordinator/coordinator_impl.cc

Issue 2819413002: [memory-infra] Remove MemoryDumpManagerDelegate (Closed)
Patch Set: rebase. Created 3 years, 8 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 | « services/resource_coordinator/BUILD.gn ('k') | services/resource_coordinator/public/cpp/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/resource_coordinator/memory/coordinator/coordinator_impl.cc
diff --git a/services/resource_coordinator/memory/coordinator/coordinator_impl.cc b/services/resource_coordinator/memory/coordinator/coordinator_impl.cc
index 3f65716f9ae142d46cd6fc14206c038d124bfd24..4b2f920bb81bc9ee4bc2f79ed45eb75a35ddf2cc 100644
--- a/services/resource_coordinator/memory/coordinator/coordinator_impl.cc
+++ b/services/resource_coordinator/memory/coordinator/coordinator_impl.cc
@@ -13,7 +13,7 @@
#include "base/threading/platform_thread.h"
#include "base/trace_event/memory_dump_manager.h"
#include "base/trace_event/memory_dump_request_args.h"
-#include "services/resource_coordinator/public/cpp/memory/memory_dump_manager_delegate_impl.h"
+#include "services/resource_coordinator/public/cpp/memory/process_local_dump_manager_impl.h"
#include "services/resource_coordinator/public/interfaces/memory/constants.mojom.h"
#include "services/resource_coordinator/public/interfaces/memory/memory_instrumentation.mojom.h"
@@ -34,12 +34,10 @@ CoordinatorImpl::CoordinatorImpl(bool initialize_memory_dump_manager)
: failed_memory_dump_count_(0),
initialize_memory_dump_manager_(initialize_memory_dump_manager) {
if (initialize_memory_dump_manager) {
- MemoryDumpManagerDelegateImpl::Config config(this);
- auto delegate = base::MakeUnique<MemoryDumpManagerDelegateImpl>(config);
+ ProcessLocalDumpManagerImpl::CreateInstance(
+ ProcessLocalDumpManagerImpl::Config(this));
base::trace_event::MemoryDumpManager::GetInstance()->set_tracing_process_id(
mojom::kServiceTracingProcessId);
- base::trace_event::MemoryDumpManager::GetInstance()->Initialize(
- std::move(delegate));
}
g_coordinator_impl = this;
}
« no previous file with comments | « services/resource_coordinator/BUILD.gn ('k') | services/resource_coordinator/public/cpp/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698