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

Side by Side Diff: components/leveldb/leveldb_service_impl.h

Issue 1839823002: mojo leveldb: Remove the created file thread. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: General patch cleanup. Created 4 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_LEVELDB_LEVELDB_SERVICE_IMPL_H_ 5 #ifndef COMPONENTS_LEVELDB_LEVELDB_SERVICE_IMPL_H_
6 #define COMPONENTS_LEVELDB_LEVELDB_SERVICE_IMPL_H_ 6 #define COMPONENTS_LEVELDB_LEVELDB_SERVICE_IMPL_H_
7 7
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "components/leveldb/leveldb_file_thread.h" 9 #include "components/leveldb/leveldb_mojo_proxy.h"
10 #include "components/leveldb/public/interfaces/leveldb.mojom.h" 10 #include "components/leveldb/public/interfaces/leveldb.mojom.h"
11 #include "mojo/public/cpp/bindings/binding_set.h" 11 #include "mojo/public/cpp/bindings/binding_set.h"
12 12
13 namespace leveldb { 13 namespace leveldb {
14 14
15 // Creates LevelDBDatabases based scoped to a |directory|/|dbname|. 15 // Creates LevelDBDatabases based scoped to a |directory|/|dbname|.
16 class LevelDBServiceImpl : public LevelDBService { 16 class LevelDBServiceImpl : public LevelDBService {
17 public: 17 public:
18 LevelDBServiceImpl(); 18 LevelDBServiceImpl(scoped_refptr<base::SingleThreadTaskRunner> task_runner);
19 ~LevelDBServiceImpl() override; 19 ~LevelDBServiceImpl() override;
20 20
21 // Overridden from LevelDBService: 21 // Overridden from LevelDBService:
22 void Open(filesystem::DirectoryPtr directory, 22 void Open(filesystem::DirectoryPtr directory,
23 const mojo::String& dbname, 23 const mojo::String& dbname,
24 leveldb::LevelDBDatabaseRequest database, 24 leveldb::LevelDBDatabaseRequest database,
25 const OpenCallback& callback) override; 25 const OpenCallback& callback) override;
26 void OpenInMemory(leveldb::LevelDBDatabaseRequest database, 26 void OpenInMemory(leveldb::LevelDBDatabaseRequest database,
27 const OpenInMemoryCallback& callback) override; 27 const OpenInMemoryCallback& callback) override;
28 28
29 private: 29 private:
30 // Thread to own the mojo message pipe. Because leveldb spawns multiple 30 // Thread to own the mojo message pipe. Because leveldb spawns multiple
31 // threads that want to call file stuff, we create a dedicated thread to send 31 // threads that want to call file stuff, we create a dedicated thread to send
32 // and receive mojo message calls. 32 // and receive mojo message calls.
33 scoped_refptr<LevelDBFileThread> thread_; 33 scoped_refptr<LevelDBMojoProxy> thread_;
34 34
35 DISALLOW_COPY_AND_ASSIGN(LevelDBServiceImpl); 35 DISALLOW_COPY_AND_ASSIGN(LevelDBServiceImpl);
36 }; 36 };
37 37
38 } // namespace leveldb 38 } // namespace leveldb
39 39
40 #endif // COMPONENTS_LEVELDB_LEVELDB_SERVICE_IMPL_H_ 40 #endif // COMPONENTS_LEVELDB_LEVELDB_SERVICE_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698