From 1683a7b8537afbd087128c1fbbe3257c20e4afcf Mon Sep 17 00:00:00 2001 From: kalman Date: Thu, 9 Apr 2015 14:47:43 -0700 Subject: Remove the eager opening of the leveldatabase from LeveldbValueStore's constructor. This code is probably legacy, disagrees with the comment in the header file, and appears to cause a sizeable performance hit on startup. BUG=475622 R=rdevlin.cronin@chromium.org Review URL: https://codereview.chromium.org/1073923002 Cr-Commit-Position: refs/heads/master@{#324511} --- extensions/browser/value_store/leveldb_value_store.cc | 4 ---- 1 file changed, 4 deletions(-) (limited to 'extensions') diff --git a/extensions/browser/value_store/leveldb_value_store.cc b/extensions/browser/value_store/leveldb_value_store.cc index 56fd265..ad8076d 100644 --- a/extensions/browser/value_store/leveldb_value_store.cc +++ b/extensions/browser/value_store/leveldb_value_store.cc @@ -51,10 +51,6 @@ class ScopedSnapshot { LeveldbValueStore::LeveldbValueStore(const base::FilePath& db_path) : db_path_(db_path) { DCHECK_CURRENTLY_ON(BrowserThread::FILE); - - scoped_ptr open_error = EnsureDbIsOpen(); - if (open_error) - LOG(WARNING) << open_error->message; } LeveldbValueStore::~LeveldbValueStore() { -- cgit v1.1