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

Side by Side Diff: chrome/browser/predictors/resource_prefetch_predictor_tables_unittest.cc

Issue 2478823002: predictors: ResourcePrefetchPredictorTables cleanup. (Closed)
Patch Set: lint Created 4 years, 1 month 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 <memory> 5 #include <memory>
6 #include <set> 6 #include <set>
7 #include <utility> 7 #include <utility>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/message_loop/message_loop.h" 10 #include "base/message_loop/message_loop.h"
(...skipping 611 matching lines...) Expand 10 before | Expand all | Expand 10 after
622 622
623 TEST_F(ResourcePrefetchPredictorTablesTest, DeleteSingleDataPoint) { 623 TEST_F(ResourcePrefetchPredictorTablesTest, DeleteSingleDataPoint) {
624 TestDeleteSingleDataPoint(); 624 TestDeleteSingleDataPoint();
625 } 625 }
626 626
627 TEST_F(ResourcePrefetchPredictorTablesTest, DeleteAllData) { 627 TEST_F(ResourcePrefetchPredictorTablesTest, DeleteAllData) {
628 TestDeleteAllData(); 628 TestDeleteAllData();
629 } 629 }
630 630
631 TEST_F(ResourcePrefetchPredictorTablesTest, DatabaseVersionIsSet) { 631 TEST_F(ResourcePrefetchPredictorTablesTest, DatabaseVersionIsSet) {
632 sql::Connection* db = tables_->DB();
633 const int version = ResourcePrefetchPredictorTables::kDatabaseVersion; 632 const int version = ResourcePrefetchPredictorTables::kDatabaseVersion;
634 EXPECT_EQ(version, ResourcePrefetchPredictorTables::GetDatabaseVersion(db)); 633 EXPECT_EQ(version, tables_->GetDatabaseVersion());
635 } 634 }
636 635
637 TEST_F(ResourcePrefetchPredictorTablesTest, DatabaseIsResetWhenIncompatible) { 636 TEST_F(ResourcePrefetchPredictorTablesTest, DatabaseIsResetWhenIncompatible) {
638 const int version = ResourcePrefetchPredictorTables::kDatabaseVersion; 637 const int version = ResourcePrefetchPredictorTables::kDatabaseVersion;
639 sql::Connection* db = tables_->DB(); 638 ASSERT_TRUE(tables_->SetDatabaseVersion(version + 1));
640 ASSERT_TRUE( 639 EXPECT_EQ(version + 1, tables_->GetDatabaseVersion());
641 ResourcePrefetchPredictorTables::SetDatabaseVersion(db, version + 1));
642 EXPECT_EQ(version + 1,
643 ResourcePrefetchPredictorTables::GetDatabaseVersion(db));
644 640
645 ReopenDatabase(); 641 ReopenDatabase();
646 642
647 db = tables_->DB(); 643 ASSERT_EQ(version, tables_->GetDatabaseVersion());
648 ASSERT_EQ(version, ResourcePrefetchPredictorTables::GetDatabaseVersion(db));
649 644
650 PrefetchDataMap url_data, host_data; 645 PrefetchDataMap url_data, host_data;
651 RedirectDataMap url_redirect_data, host_redirect_data; 646 RedirectDataMap url_redirect_data, host_redirect_data;
652 tables_->GetAllData(&url_data, &host_data, &url_redirect_data, 647 tables_->GetAllData(&url_data, &host_data, &url_redirect_data,
653 &host_redirect_data); 648 &host_redirect_data);
654 EXPECT_TRUE(url_data.empty()); 649 EXPECT_TRUE(url_data.empty());
655 EXPECT_TRUE(host_data.empty()); 650 EXPECT_TRUE(host_data.empty());
651 EXPECT_TRUE(url_redirect_data.empty());
652 EXPECT_TRUE(host_redirect_data.empty());
656 } 653 }
657 654
658 TEST_F(ResourcePrefetchPredictorTablesReopenTest, GetAllData) { 655 TEST_F(ResourcePrefetchPredictorTablesReopenTest, GetAllData) {
659 TestGetAllData(); 656 TestGetAllData();
660 } 657 }
661 658
662 TEST_F(ResourcePrefetchPredictorTablesReopenTest, UpdateData) { 659 TEST_F(ResourcePrefetchPredictorTablesReopenTest, UpdateData) {
663 TestUpdateData(); 660 TestUpdateData();
664 } 661 }
665 662
666 TEST_F(ResourcePrefetchPredictorTablesReopenTest, DeleteData) { 663 TEST_F(ResourcePrefetchPredictorTablesReopenTest, DeleteData) {
667 TestDeleteData(); 664 TestDeleteData();
668 } 665 }
669 666
670 TEST_F(ResourcePrefetchPredictorTablesReopenTest, DeleteSingleDataPoint) { 667 TEST_F(ResourcePrefetchPredictorTablesReopenTest, DeleteSingleDataPoint) {
671 TestDeleteSingleDataPoint(); 668 TestDeleteSingleDataPoint();
672 } 669 }
673 670
674 TEST_F(ResourcePrefetchPredictorTablesReopenTest, DeleteAllData) { 671 TEST_F(ResourcePrefetchPredictorTablesReopenTest, DeleteAllData) {
675 TestDeleteAllData(); 672 TestDeleteAllData();
676 } 673 }
677 674
678 } // namespace predictors 675 } // namespace predictors
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698