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

Side by Side Diff: chrome/browser/sync_file_system/local/local_file_sync_service_unittest.cc

Issue 655413002: Convert ARRAYSIZE_UNSAFE -> arraysize in chrome/browser/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 <vector> 5 #include <vector>
6 6
7 #include "base/basictypes.h" 7 #include "base/basictypes.h"
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/files/file_util.h" 9 #include "base/files/file_util.h"
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 612 matching lines...) Expand 10 before | Expand all | Expand 10 after
623 SetOriginChangeCount(kOrigin2, 2); 623 SetOriginChangeCount(kOrigin2, 2);
624 624
625 ASSERT_EQ(1 + 2, GetTotalChangeCount()); 625 ASSERT_EQ(1 + 2, GetTotalChangeCount());
626 626
627 SetOriginChangeCount(kOrigin3, 4); 627 SetOriginChangeCount(kOrigin3, 4);
628 628
629 ASSERT_EQ(1 + 2 + 4, GetTotalChangeCount()); 629 ASSERT_EQ(1 + 2 + 4, GetTotalChangeCount());
630 630
631 const GURL kOrigins[] = { kOrigin1, kOrigin2, kOrigin3 }; 631 const GURL kOrigins[] = { kOrigin1, kOrigin2, kOrigin3 };
632 std::set<GURL> all_origins; 632 std::set<GURL> all_origins;
633 all_origins.insert(kOrigins, kOrigins + ARRAYSIZE_UNSAFE(kOrigins)); 633 all_origins.insert(kOrigins, kOrigins + arraysize(kOrigins));
634 634
635 GURL origin; 635 GURL origin;
636 while (!all_origins.empty()) { 636 while (!all_origins.empty()) {
637 ASSERT_TRUE(NextOriginToProcess(&origin)); 637 ASSERT_TRUE(NextOriginToProcess(&origin));
638 ASSERT_TRUE(ContainsKey(all_origins, origin)); 638 ASSERT_TRUE(ContainsKey(all_origins, origin));
639 all_origins.erase(origin); 639 all_origins.erase(origin);
640 } 640 }
641 641
642 // Set kOrigin2's change count 0. 642 // Set kOrigin2's change count 0.
643 SetOriginChangeCount(kOrigin2, 0); 643 SetOriginChangeCount(kOrigin2, 0);
(...skipping 16 matching lines...) Expand all
660 while (!all_origins.empty()) { 660 while (!all_origins.empty()) {
661 ASSERT_TRUE(NextOriginToProcess(&origin)); 661 ASSERT_TRUE(NextOriginToProcess(&origin));
662 ASSERT_TRUE(ContainsKey(all_origins, origin)); 662 ASSERT_TRUE(ContainsKey(all_origins, origin));
663 all_origins.erase(origin); 663 all_origins.erase(origin);
664 } 664 }
665 665
666 // Set kOrigin2's change count 8. 666 // Set kOrigin2's change count 8.
667 SetOriginChangeCount(kOrigin2, 8); 667 SetOriginChangeCount(kOrigin2, 8);
668 ASSERT_EQ(1 + 4 + 8, GetTotalChangeCount()); 668 ASSERT_EQ(1 + 4 + 8, GetTotalChangeCount());
669 669
670 all_origins.insert(kOrigins, kOrigins + ARRAYSIZE_UNSAFE(kOrigins)); 670 all_origins.insert(kOrigins, kOrigins + arraysize(kOrigins));
671 while (!all_origins.empty()) { 671 while (!all_origins.empty()) {
672 ASSERT_TRUE(NextOriginToProcess(&origin)); 672 ASSERT_TRUE(NextOriginToProcess(&origin));
673 ASSERT_TRUE(ContainsKey(all_origins, origin)); 673 ASSERT_TRUE(ContainsKey(all_origins, origin));
674 all_origins.erase(origin); 674 all_origins.erase(origin);
675 } 675 }
676 } 676 }
677 677
678 TEST_F(OriginChangeMapTest, WithDisabled) { 678 TEST_F(OriginChangeMapTest, WithDisabled) {
679 const GURL kOrigin1("chrome-extension://foo"); 679 const GURL kOrigin1("chrome-extension://foo");
680 const GURL kOrigin2("chrome-extension://bar"); 680 const GURL kOrigin2("chrome-extension://bar");
681 const GURL kOrigin3("chrome-extension://baz"); 681 const GURL kOrigin3("chrome-extension://baz");
682 const GURL kOrigins[] = { kOrigin1, kOrigin2, kOrigin3 }; 682 const GURL kOrigins[] = { kOrigin1, kOrigin2, kOrigin3 };
683 683
684 ASSERT_EQ(0, GetTotalChangeCount()); 684 ASSERT_EQ(0, GetTotalChangeCount());
685 685
686 SetOriginChangeCount(kOrigin1, 1); 686 SetOriginChangeCount(kOrigin1, 1);
687 SetOriginChangeCount(kOrigin2, 2); 687 SetOriginChangeCount(kOrigin2, 2);
688 SetOriginChangeCount(kOrigin3, 4); 688 SetOriginChangeCount(kOrigin3, 4);
689 689
690 ASSERT_EQ(1 + 2 + 4, GetTotalChangeCount()); 690 ASSERT_EQ(1 + 2 + 4, GetTotalChangeCount());
691 691
692 std::set<GURL> all_origins; 692 std::set<GURL> all_origins;
693 all_origins.insert(kOrigins, kOrigins + ARRAYSIZE_UNSAFE(kOrigins)); 693 all_origins.insert(kOrigins, kOrigins + arraysize(kOrigins));
694 694
695 GURL origin; 695 GURL origin;
696 while (!all_origins.empty()) { 696 while (!all_origins.empty()) {
697 ASSERT_TRUE(NextOriginToProcess(&origin)); 697 ASSERT_TRUE(NextOriginToProcess(&origin));
698 ASSERT_TRUE(ContainsKey(all_origins, origin)); 698 ASSERT_TRUE(ContainsKey(all_origins, origin));
699 all_origins.erase(origin); 699 all_origins.erase(origin);
700 } 700 }
701 701
702 SetOriginEnabled(kOrigin2, false); 702 SetOriginEnabled(kOrigin2, false);
703 ASSERT_EQ(1 + 4, GetTotalChangeCount()); 703 ASSERT_EQ(1 + 4, GetTotalChangeCount());
(...skipping 22 matching lines...) Expand all
726 all_origins.insert(kOrigin2); 726 all_origins.insert(kOrigin2);
727 all_origins.insert(kOrigin3); 727 all_origins.insert(kOrigin3);
728 while (!all_origins.empty()) { 728 while (!all_origins.empty()) {
729 ASSERT_TRUE(NextOriginToProcess(&origin)); 729 ASSERT_TRUE(NextOriginToProcess(&origin));
730 ASSERT_TRUE(ContainsKey(all_origins, origin)); 730 ASSERT_TRUE(ContainsKey(all_origins, origin));
731 all_origins.erase(origin); 731 all_origins.erase(origin);
732 } 732 }
733 } 733 }
734 734
735 } // namespace sync_file_system 735 } // namespace sync_file_system
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698