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

Side by Side Diff: chrome/browser/sync_file_system/drive_backend/register_app_task_unittest.cc

Issue 2953473002: Use leveldb_env::OpenDB() to open leveldb databases. (Closed)
Patch Set: Rebase; add comments to CHECK() 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #include "chrome/browser/sync_file_system/drive_backend/register_app_task.h" 5 #include "chrome/browser/sync_file_system/drive_backend/register_app_task.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <utility> 10 #include <utility>
(...skipping 13 matching lines...) Expand all
24 #include "chrome/browser/sync_file_system/drive_backend/leveldb_wrapper.h" 24 #include "chrome/browser/sync_file_system/drive_backend/leveldb_wrapper.h"
25 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.h" 25 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.h"
26 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.pb.h" 26 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.pb.h"
27 #include "chrome/browser/sync_file_system/drive_backend/sync_engine_context.h" 27 #include "chrome/browser/sync_file_system/drive_backend/sync_engine_context.h"
28 #include "chrome/browser/sync_file_system/sync_file_system_test_util.h" 28 #include "chrome/browser/sync_file_system/sync_file_system_test_util.h"
29 #include "components/drive/drive_uploader.h" 29 #include "components/drive/drive_uploader.h"
30 #include "components/drive/service/fake_drive_service.h" 30 #include "components/drive/service/fake_drive_service.h"
31 #include "content/public/test/test_browser_thread_bundle.h" 31 #include "content/public/test/test_browser_thread_bundle.h"
32 #include "google_apis/drive/drive_api_parser.h" 32 #include "google_apis/drive/drive_api_parser.h"
33 #include "testing/gtest/include/gtest/gtest.h" 33 #include "testing/gtest/include/gtest/gtest.h"
34 #include "third_party/leveldatabase/env_chromium.h"
34 #include "third_party/leveldatabase/src/helpers/memenv/memenv.h" 35 #include "third_party/leveldatabase/src/helpers/memenv/memenv.h"
35 #include "third_party/leveldatabase/src/include/leveldb/db.h" 36 #include "third_party/leveldatabase/src/include/leveldb/db.h"
36 #include "third_party/leveldatabase/src/include/leveldb/env.h" 37 #include "third_party/leveldatabase/src/include/leveldb/env.h"
37 38
38 namespace sync_file_system { 39 namespace sync_file_system {
39 namespace drive_backend { 40 namespace drive_backend {
40 41
41 namespace { 42 namespace {
42 const int64_t kSyncRootTrackerID = 100; 43 const int64_t kSyncRootTrackerID = 100;
43 } // namespace 44 } // namespace
(...skipping 29 matching lines...) Expand all
73 kSyncRootFolderTitle, &sync_root_folder_id_)); 74 kSyncRootFolderTitle, &sync_root_folder_id_));
74 } 75 }
75 76
76 void TearDown() override { 77 void TearDown() override {
77 context_.reset(); 78 context_.reset();
78 base::RunLoop().RunUntilIdle(); 79 base::RunLoop().RunUntilIdle();
79 } 80 }
80 81
81 protected: 82 protected:
82 std::unique_ptr<LevelDBWrapper> OpenLevelDB() { 83 std::unique_ptr<LevelDBWrapper> OpenLevelDB() {
83 leveldb::DB* db = nullptr; 84 std::unique_ptr<leveldb::DB> db;
84 leveldb::Options options; 85 leveldb::Options options;
85 options.create_if_missing = true; 86 options.create_if_missing = true;
86 options.env = in_memory_env_.get(); 87 options.env = in_memory_env_.get();
87 leveldb::Status status = 88 leveldb::Status status = leveldb_env::OpenDB(
88 leveldb::DB::Open(options, database_dir_.GetPath().AsUTF8Unsafe(), &db); 89 options, database_dir_.GetPath().AsUTF8Unsafe(), &db);
89 EXPECT_TRUE(status.ok()); 90 EXPECT_TRUE(status.ok());
90 return base::MakeUnique<LevelDBWrapper>(base::WrapUnique(db)); 91 return base::MakeUnique<LevelDBWrapper>(std::move(db));
91 } 92 }
92 93
93 void SetUpInitialData(LevelDBWrapper* db) { 94 void SetUpInitialData(LevelDBWrapper* db) {
94 ServiceMetadata service_metadata; 95 ServiceMetadata service_metadata;
95 service_metadata.set_largest_change_id(100); 96 service_metadata.set_largest_change_id(100);
96 service_metadata.set_sync_root_tracker_id(kSyncRootTrackerID); 97 service_metadata.set_sync_root_tracker_id(kSyncRootTrackerID);
97 service_metadata.set_next_tracker_id(next_tracker_id_); 98 service_metadata.set_next_tracker_id(next_tracker_id_);
98 99
99 FileDetails sync_root_details; 100 FileDetails sync_root_details;
100 sync_root_details.set_title(kSyncRootFolderTitle); 101 sync_root_details.set_title(kSyncRootFolderTitle);
(...skipping 234 matching lines...) Expand 10 before | Expand all | Expand 10 after
335 336
336 CreateMetadataDatabase(std::move(db)); 337 CreateMetadataDatabase(std::move(db));
337 RunRegisterAppTask(kAppID); 338 RunRegisterAppTask(kAppID);
338 339
339 EXPECT_EQ(1u, CountRegisteredAppRoot()); 340 EXPECT_EQ(1u, CountRegisteredAppRoot());
340 EXPECT_TRUE(IsAppRegistered(kAppID)); 341 EXPECT_TRUE(IsAppRegistered(kAppID));
341 } 342 }
342 343
343 } // namespace drive_backend 344 } // namespace drive_backend
344 } // namespace sync_file_system 345 } // namespace sync_file_system
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698