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

Unified Diff: content/browser/service_worker/service_worker_database.cc

Issue 2953473002: Use leveldb_env::OpenDB() to open leveldb databases. (Closed)
Patch Set: Use OpenDB in unittests; add allocation edges Created 3 years, 5 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
Index: content/browser/service_worker/service_worker_database.cc
diff --git a/content/browser/service_worker/service_worker_database.cc b/content/browser/service_worker/service_worker_database.cc
index 9f1bc46069a7bd360b55c7390bd5a971127f5610..b4310b7c0579e4d3f3262aad6d14a7192f25fa8b 100644
--- a/content/browser/service_worker/service_worker_database.cc
+++ b/content/browser/service_worker/service_worker_database.cc
@@ -1231,16 +1231,13 @@ ServiceWorkerDatabase::Status ServiceWorkerDatabase::LazyOpen(
options.env = g_service_worker_env.Pointer();
}
- leveldb::DB* db = NULL;
Status status = LevelDBStatusToStatus(
- leveldb::DB::Open(options, path_.AsUTF8Unsafe(), &db));
+ leveldb_env::OpenDB(options, path_.AsUTF8Unsafe(), &db_));
HandleOpenResult(FROM_HERE, status);
if (status != STATUS_OK) {
- DCHECK(!db);
// TODO(nhiroki): Should we retry to open the database?
return status;
}
- db_.reset(db);
int64_t db_version;
status = ReadDatabaseVersion(&db_version);

Powered by Google App Engine
This is Rietveld 408576698