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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/browser/dom_storage/local_storage_context_mojo.cc

Issue 2722293002: Fix lifetime of leveldb::MojoEnv instances. (Closed)
Patch Set: annotate leaks 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
« components/leveldb/leveldb_app.h ('K') | « content/browser/browser_context.cc ('k') | content/browser/dom_storage/local_storage_context_mojo_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/dom_storage/local_storage_context_mojo.cc
diff --git a/content/browser/dom_storage/local_storage_context_mojo.cc b/content/browser/dom_storage/local_storage_context_mojo.cc
index a875729d2f19e0ef04c2a568e5a54f7e0abe0f7f..ddc0ef8323938f5d4c310ebce7e034d3eee53070 100644
--- a/content/browser/dom_storage/local_storage_context_mojo.cc
+++ b/content/browser/dom_storage/local_storage_context_mojo.cc
@@ -358,7 +358,6 @@ void LocalStorageContextMojo::OnDirectoryOpened(
// Now that we have a directory, connect to the LevelDB service and get our
// database.
connector_->BindInterface(file::mojom::kServiceName, &leveldb_service_);
- leveldb_service_->SetEnvironmentName("LevelDBEnv.LocalStorage");
// We might still need to use the directory, so create a clone.
filesystem::mojom::DirectoryPtr directory_clone;
« components/leveldb/leveldb_app.h ('K') | « content/browser/browser_context.cc ('k') | content/browser/dom_storage/local_storage_context_mojo_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698