diff options
author | ssid <ssid@chromium.org> | 2016-01-13 06:21:57 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-13 14:23:13 +0000 |
commit | 3be5b1ecdf66f6eaaa0ba98f9b36cc2d93bf54d9 (patch) | |
tree | 20326a0c236d72a2e28b007f1782116cf301b07c /sql/sql_memory_dump_provider.h | |
parent | 3a6f49f649c3c413ca10fc60ec85f39532ac9e45 (diff) | |
download | chromium_src-3be5b1ecdf66f6eaaa0ba98f9b36cc2d93bf54d9.zip chromium_src-3be5b1ecdf66f6eaaa0ba98f9b36cc2d93bf54d9.tar.gz chromium_src-3be5b1ecdf66f6eaaa0ba98f9b36cc2d93bf54d9.tar.bz2 |
[tracing] Add separate dump provider for sql connection
The sql connection memory dump is not thread safe since the connections
can get deleted while a dump is happening. To make this thread safe,
this CL introduces a dump provider class owned by the connection. This
class holds a lock when dumping and deleting the database. Also, to
workaround thread safe dump provider registration, it uses the
UnregisterAndDeleteDumpProviderAsync api added in crrev.com/1430073002.
BUG=466141
Review URL: https://codereview.chromium.org/1434993002
Cr-Commit-Position: refs/heads/master@{#369161}
Diffstat (limited to 'sql/sql_memory_dump_provider.h')
-rw-r--r-- | sql/sql_memory_dump_provider.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sql/sql_memory_dump_provider.h b/sql/sql_memory_dump_provider.h index 01d7f04..3d894e2 100644 --- a/sql/sql_memory_dump_provider.h +++ b/sql/sql_memory_dump_provider.h @@ -2,8 +2,8 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. -#ifndef SQL_PROCESS_MEMORY_DUMP_PROVIDER_H -#define SQL_PROCESS_MEMORY_DUMP_PROVIDER_H +#ifndef SQL_SQL_MEMORY_DUMP_PROVIDER_H +#define SQL_SQL_MEMORY_DUMP_PROVIDER_H #include "base/macros.h" #include "base/memory/singleton.h" @@ -34,4 +34,4 @@ class SQL_EXPORT SqlMemoryDumpProvider } // namespace sql -#endif // SQL_PROCESS_MEMORY_DUMP_PROVIDER_H +#endif // SQL_SQL_MEMORY_DUMP_PROVIDER_H |