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

Unified Diff: third_party/WebKit/Source/platform/exported/Platform.cpp

Issue 2009483002: Remove WebMemoryDumpProvider (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
Index: third_party/WebKit/Source/platform/exported/Platform.cpp
diff --git a/third_party/WebKit/Source/platform/exported/Platform.cpp b/third_party/WebKit/Source/platform/exported/Platform.cpp
index 0a1b6101b4ab7dba7d77d48d45f5db41ed117991..171c4a82b06dfb2e01edf7da300feb2e1910271a 100644
--- a/third_party/WebKit/Source/platform/exported/Platform.cpp
+++ b/third_party/WebKit/Source/platform/exported/Platform.cpp
@@ -37,7 +37,6 @@
#include "platform/graphics/CompositorFactory.h"
#include "platform/heap/BlinkGCMemoryDumpProvider.h"
#include "platform/heap/GCTaskRunner.h"
-#include "platform/web_memory_dump_provider_adapter.h"
#include "public/platform/Platform.h"
#include "public/platform/ServiceRegistry.h"
#include "public/platform/WebPrerenderingSupport.h"
@@ -47,20 +46,9 @@
namespace blink {
static Platform* s_platform = nullptr;
-using ProviderToAdapterMap = HashMap<WebMemoryDumpProvider*, OwnPtr<WebMemoryDumpProviderAdapter>>;
static GCTaskRunner* s_gcTaskRunner = nullptr;
-namespace {
-
-ProviderToAdapterMap& memoryDumpProviders()
-{
- DEFINE_STATIC_LOCAL(ProviderToAdapterMap, providerToAdapterMap, ());
- return providerToAdapterMap;
-}
-
-} // namespace
-
Platform::Platform()
: m_mainThread(0)
{
@@ -157,35 +145,6 @@ WebThread* Platform::mainThread() const
return m_mainThread;
}
-void Platform::registerMemoryDumpProvider(WebMemoryDumpProvider* provider, const char* name)
-{
- // MemoryDumpProvider needs a message loop.
- if (!Platform::current()->currentThread())
- return;
-
- WebMemoryDumpProviderAdapter* adapter = new WebMemoryDumpProviderAdapter(provider);
- ProviderToAdapterMap::AddResult result = memoryDumpProviders().add(provider, adoptPtr(adapter));
- if (!result.isNewEntry)
- return;
- adapter->set_is_registered(true);
- base::trace_event::MemoryDumpManager::GetInstance()->RegisterDumpProvider(adapter, name, base::ThreadTaskRunnerHandle::Get());
-}
-
-void Platform::unregisterMemoryDumpProvider(WebMemoryDumpProvider* provider)
-{
- // MemoryDumpProvider needs a message loop.
- if (!Platform::current()->currentThread())
- return;
-
- ProviderToAdapterMap::iterator it = memoryDumpProviders().find(provider);
- if (it == memoryDumpProviders().end())
- return;
- WebMemoryDumpProviderAdapter* adapter = it->value.get();
- base::trace_event::MemoryDumpManager::GetInstance()->UnregisterDumpProvider(adapter);
- adapter->set_is_registered(false);
- memoryDumpProviders().remove(it);
-}
-
ServiceRegistry* Platform::serviceRegistry()
{
return ServiceRegistry::getEmptyServiceRegistry();

Powered by Google App Engine
This is Rietveld 408576698