diff options
| author | kalman <kalman@chromium.org> | 2015-04-09 14:47:43 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-04-09 21:48:18 +0000 |
| commit | 1683a7b8537afbd087128c1fbbe3257c20e4afcf (patch) | |
| tree | ba8e7ee523231c6f2e9cb331d69d32f5f55df917 | |
| parent | 4d20f2877d6133b1c1b8687b9a2644d3d3e50c06 (diff) | |
| download | chromium_src-1683a7b8537afbd087128c1fbbe3257c20e4afcf.zip chromium_src-1683a7b8537afbd087128c1fbbe3257c20e4afcf.tar.gz chromium_src-1683a7b8537afbd087128c1fbbe3257c20e4afcf.tar.bz2 | |
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}
| -rw-r--r-- | extensions/browser/value_store/leveldb_value_store.cc | 4 |
1 files changed, 0 insertions, 4 deletions
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<Error> open_error = EnsureDbIsOpen(); - if (open_error) - LOG(WARNING) << open_error->message; } LeveldbValueStore::~LeveldbValueStore() { |