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

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

Issue 349543003: [SyncFS] Use pointers to MetadataDatabaseIndexInterface (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Work for nits Created 6 years, 6 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 | Annotate | Revision Log
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/metadata_database.h" 5 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/files/scoped_temp_dir.h" 8 #include "base/files/scoped_temp_dir.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/message_loop/message_loop_proxy.h" 10 #include "base/message_loop/message_loop_proxy.h"
11 #include "base/strings/string_number_conversions.h" 11 #include "base/strings/string_number_conversions.h"
12 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_constants. h" 12 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_constants. h"
13 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_test_util. h" 13 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_test_util. h"
14 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_util.h" 14 #include "chrome/browser/sync_file_system/drive_backend/drive_backend_util.h"
15 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.pb.h" 15 #include "chrome/browser/sync_file_system/drive_backend/metadata_database.pb.h"
16 #include "chrome/browser/sync_file_system/drive_backend/metadata_database_index. h" 16 #include "chrome/browser/sync_file_system/drive_backend/metadata_database_index. h"
17 #include "chrome/browser/sync_file_system/drive_backend/metadata_database_index_ interface.h"
17 #include "chrome/browser/sync_file_system/sync_file_system_test_util.h" 18 #include "chrome/browser/sync_file_system/sync_file_system_test_util.h"
18 #include "google_apis/drive/drive_api_parser.h" 19 #include "google_apis/drive/drive_api_parser.h"
19 #include "testing/gtest/include/gtest/gtest.h" 20 #include "testing/gtest/include/gtest/gtest.h"
20 #include "third_party/leveldatabase/src/helpers/memenv/memenv.h" 21 #include "third_party/leveldatabase/src/helpers/memenv/memenv.h"
21 #include "third_party/leveldatabase/src/include/leveldb/db.h" 22 #include "third_party/leveldatabase/src/include/leveldb/db.h"
22 #include "third_party/leveldatabase/src/include/leveldb/env.h" 23 #include "third_party/leveldatabase/src/include/leveldb/env.h"
23 #include "third_party/leveldatabase/src/include/leveldb/write_batch.h" 24 #include "third_party/leveldatabase/src/include/leveldb/write_batch.h"
24 25
25 #define FPL(a) FILE_PATH_LITERAL(a) 26 #define FPL(a) FILE_PATH_LITERAL(a)
26 27
(...skipping 434 matching lines...) Expand 10 before | Expand all | Expand 10 after
461 462
462 void VerifyReloadConsistency() { 463 void VerifyReloadConsistency() {
463 scoped_ptr<MetadataDatabase> metadata_database_2; 464 scoped_ptr<MetadataDatabase> metadata_database_2;
464 ASSERT_EQ(SYNC_STATUS_OK, 465 ASSERT_EQ(SYNC_STATUS_OK,
465 MetadataDatabase::CreateForTesting( 466 MetadataDatabase::CreateForTesting(
466 metadata_database_->db_.Pass(), 467 metadata_database_->db_.Pass(),
467 &metadata_database_2)); 468 &metadata_database_2));
468 metadata_database_->db_ = metadata_database_2->db_.Pass(); 469 metadata_database_->db_ = metadata_database_2->db_.Pass();
469 470
470 const MetadataDatabaseIndex* on_memory = 471 const MetadataDatabaseIndex* on_memory =
471 metadata_database_->index_.get(); 472 static_cast<MetadataDatabaseIndex*>(metadata_database_->index_.get());
472 const MetadataDatabaseIndex* reloaded = 473 const MetadataDatabaseIndex* reloaded =
473 metadata_database_2->index_.get(); 474 static_cast<MetadataDatabaseIndex*>(metadata_database_2->index_.get());
474 475
475 { 476 {
476 SCOPED_TRACE("Expect equivalent service_metadata"); 477 SCOPED_TRACE("Expect equivalent service_metadata");
477 ExpectEquivalent(metadata_database_->service_metadata_.get(), 478 ExpectEquivalent(metadata_database_->service_metadata_.get(),
478 metadata_database_2->service_metadata_.get()); 479 metadata_database_2->service_metadata_.get());
479 } 480 }
480 481
481 { 482 {
482 SCOPED_TRACE("Expect equivalent metadata_by_id_ contents."); 483 SCOPED_TRACE("Expect equivalent metadata_by_id_ contents.");
483 ExpectEquivalent(on_memory->metadata_by_id_, 484 ExpectEquivalent(on_memory->metadata_by_id_,
(...skipping 671 matching lines...) Expand 10 before | Expand all | Expand 10 after
1155 EXPECT_TRUE(file->HasKey("details")); 1156 EXPECT_TRUE(file->HasKey("details"));
1156 1157
1157 ASSERT_TRUE(files->GetDictionary(1, &file)); 1158 ASSERT_TRUE(files->GetDictionary(1, &file));
1158 EXPECT_TRUE(file->GetString("title", &str) && str == "file_0"); 1159 EXPECT_TRUE(file->GetString("title", &str) && str == "file_0");
1159 EXPECT_TRUE(file->GetString("type", &str) && str == "file"); 1160 EXPECT_TRUE(file->GetString("type", &str) && str == "file");
1160 EXPECT_TRUE(file->HasKey("details")); 1161 EXPECT_TRUE(file->HasKey("details"));
1161 } 1162 }
1162 1163
1163 } // namespace drive_backend 1164 } // namespace drive_backend
1164 } // namespace sync_file_system 1165 } // namespace sync_file_system
OLDNEW
« no previous file with comments | « chrome/browser/sync_file_system/drive_backend/metadata_database_index_unittest.cc ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698