Use leveldb_env::OpenDB() to open leveldb databases.

components/leveldb_proto/leveldb_database.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/leveldb_proto/leveldb_database.h"
 
 #include <inttypes.h>
 
 #include <string>
 #include <vector>
@@ skipping 44 matching lines @@
   base::trace_event::MemoryDumpManager::GetInstance()->UnregisterDumpProvider(
       this);
 }
 
 bool LevelDB::InitWithOptions(const base::FilePath& database_dir,
                               const leveldb::Options& options) {
   DFAKE_SCOPED_LOCK(thread_checker_);
 
   std::string path = database_dir.AsUTF8Unsafe();
 
-  leveldb::DB* db = NULL;
-  leveldb::Status status = leveldb::DB::Open(options, path, &db);
+  leveldb::Status status = leveldb_env::OpenDB(options, path, &db_);
   if (open_histogram_)
     open_histogram_->Add(leveldb_env::GetLevelDBStatusUMAValue(status));
   if (status.IsCorruption()) {
     base::DeleteFile(database_dir, true);
-    status = leveldb::DB::Open(options, path, &db);
+    status = leveldb_env::OpenDB(options, path, &db_);
   }
 
   if (status.ok()) {
-    CHECK(db);

[nyquist, 2017/07/18 18:48:16]

This CHECK seems to be kept in place some places, is there a particular reason
why it is not changed here to become CHECK(db_) instead of removed?

[DmitrySkiba, 2017/07/18 19:01:05]

OpenDB() guarantees that now (i.e. CHECK effectively moved there). Specifically
it guarantees that (1) db_ is not touched on failure, and (2) db_ is not NULL on
success.

-    db_.reset(db);
-
     base::trace_event::MemoryDumpManager::GetInstance()
         ->RegisterDumpProviderWithSequencedTaskRunner(
             this, "LevelDB", base::SequencedTaskRunnerHandle::Get(),
             base::trace_event::MemoryDumpProvider::Options());
     return true;
   }
 
   LOG(WARNING) << "Unable to open " << database_dir.value() << ": "
                << status.ToString();
   return false;
@@ skipping 128 matching lines @@
       base::StringPrintf("leveldb/leveldb_proto/0x%" PRIXPTR,
                          reinterpret_cast<uintptr_t>(db_.get())));
   dump->AddScalar(base::trace_event::MemoryAllocatorDump::kNameSize,
                   base::trace_event::MemoryAllocatorDump::kUnitsBytes, size);
   if (!client_name_.empty() &&
       dump_args.level_of_detail !=
           base::trace_event::MemoryDumpLevelOfDetail::BACKGROUND) {
     dump->AddString("client_name", "", client_name_);
   }
 
-  // Memory is allocated from system allocator (malloc).
-  const char* system_allocator_pool_name =
-      base::trace_event::MemoryDumpManager::GetInstance()
-          ->system_allocator_pool_name();
-  if (system_allocator_pool_name)
-    pmd->AddSuballocation(dump->guid(), system_allocator_pool_name);
+  // All leveldb databases are already dumped by leveldb_env::DBTracker. Add
+  // an edge to avoid double counting.
+  pmd->AddSuballocation(dump->guid(),
+                        leveldb_env::DBTracker::GetMemoryDumpName(db_.get()));
 
   return true;
 }
 
 }  // namespace leveldb_proto