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

Unified Diff: content/browser/leveldb_wrapper_impl.h

Issue 1745603002: Switch LevelDBWrapper::GetAll to use the new Mojo sync IPC mechanism. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comment Created 4 years, 10 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
« no previous file with comments | « content/browser/level_db_wrapper_impl.cc ('k') | content/browser/leveldb_wrapper_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/leveldb_wrapper_impl.h
diff --git a/content/browser/level_db_wrapper_impl.h b/content/browser/leveldb_wrapper_impl.h
similarity index 81%
rename from content/browser/level_db_wrapper_impl.h
rename to content/browser/leveldb_wrapper_impl.h
index 06ed3485f612589a34c8651497967ab37daf4452..445fcaae189c2a1e6d238810eb5198770bfe74f2 100644
--- a/content/browser/level_db_wrapper_impl.h
+++ b/content/browser/leveldb_wrapper_impl.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CONTENT_BROWSER_LEVEL_DB_WRAPPER_IMPL_H_
-#define CONTENT_BROWSER_LEVEL_DB_WRAPPER_IMPL_H_
+#ifndef CONTENT_BROWSER_LEVELDB_WRAPPER_IMPL_H_
+#define CONTENT_BROWSER_LEVELDB_WRAPPER_IMPL_H_
#include "base/callback.h"
#include "base/macros.h"
@@ -28,8 +28,7 @@ class LevelDBWrapperImpl : public LevelDBWrapper {
const base::Closure& no_bindings_callback);
~LevelDBWrapperImpl() override;
- void Bind(mojo::InterfaceRequest<LevelDBWrapper> request,
- LevelDBObserverPtr observer);
+ void Bind(mojo::InterfaceRequest<LevelDBWrapper> request);
private:
// LevelDBWrapperImpl:
@@ -40,10 +39,12 @@ class LevelDBWrapperImpl : public LevelDBWrapper {
void Delete(mojo::Array<uint8_t> key,
const mojo::String& source,
const DeleteCallback& callback) override;
- void DeleteAll(const mojo::String& source,
+ void DeleteAll(LevelDBObserverPtr observer,
+ const mojo::String& source,
const DeleteAllCallback& callback) override;
void Get(mojo::Array<uint8_t> key, const GetCallback& callback) override;
- void GetAll(const GetAllCallback& callback) override;
+ void GetAll(LevelDBObserverPtr observer,
+ const GetAllCallback& callback) override;
void OnConnectionError();
@@ -56,4 +57,4 @@ class LevelDBWrapperImpl : public LevelDBWrapper {
} // namespace content
-#endif // CONTENT_BROWSER_LEVEL_DB_WRAPPER_IMPL_H_
+#endif // CONTENT_BROWSER_LEVELDB_WRAPPER_IMPL_H_
« no previous file with comments | « content/browser/level_db_wrapper_impl.cc ('k') | content/browser/leveldb_wrapper_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698