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

Side by Side Diff: chrome/browser/media_galleries/win/mtp_device_object_enumerator_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 (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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 // MTPDeviceObjectEnumerator unit tests. 5 // MTPDeviceObjectEnumerator unit tests.
6 6
7 #include "chrome/browser/media_galleries/win/mtp_device_object_enumerator.h" 7 #include "chrome/browser/media_galleries/win/mtp_device_object_enumerator.h"
8 8
9 #include <ctime> 9 #include <ctime>
10 10
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 MTPDeviceObjectEntries entries; 56 MTPDeviceObjectEntries entries;
57 MTPDeviceObjectEnumerator enumerator(entries); 57 MTPDeviceObjectEnumerator enumerator(entries);
58 TestEnumeratorIsEmpty(&enumerator); 58 TestEnumeratorIsEmpty(&enumerator);
59 TestNextEntryIsEmpty(&enumerator); 59 TestNextEntryIsEmpty(&enumerator);
60 TestNextEntryIsEmpty(&enumerator); 60 TestNextEntryIsEmpty(&enumerator);
61 TestEnumeratorIsEmpty(&enumerator); 61 TestEnumeratorIsEmpty(&enumerator);
62 } 62 }
63 63
64 TEST_F(MTPDeviceObjectEnumeratorWinTest, Traversal) { 64 TEST_F(MTPDeviceObjectEnumeratorWinTest, Traversal) {
65 MTPDeviceObjectEntries entries; 65 MTPDeviceObjectEntries entries;
66 for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kTestCases); ++i) { 66 for (size_t i = 0; i < arraysize(kTestCases); ++i) {
67 entries.push_back(MTPDeviceObjectEntry( 67 entries.push_back(MTPDeviceObjectEntry(
68 kTestCases[i].object_id, 68 kTestCases[i].object_id,
69 kTestCases[i].name, 69 kTestCases[i].name,
70 kTestCases[i].is_directory, 70 kTestCases[i].is_directory,
71 kTestCases[i].size, 71 kTestCases[i].size,
72 base::Time::FromTimeT(kTestCases[i].last_modified_time))); 72 base::Time::FromTimeT(kTestCases[i].last_modified_time)));
73 } 73 }
74 MTPDeviceObjectEnumerator enumerator(entries); 74 MTPDeviceObjectEnumerator enumerator(entries);
75 TestEnumeratorIsEmpty(&enumerator); 75 TestEnumeratorIsEmpty(&enumerator);
76 TestEnumeratorIsEmpty(&enumerator); 76 TestEnumeratorIsEmpty(&enumerator);
77 for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kTestCases); ++i) { 77 for (size_t i = 0; i < arraysize(kTestCases); ++i) {
78 EXPECT_EQ(kTestCases[i].name, enumerator.Next().value()); 78 EXPECT_EQ(kTestCases[i].name, enumerator.Next().value());
79 EXPECT_EQ(kTestCases[i].object_id, enumerator.GetObjectId()); 79 EXPECT_EQ(kTestCases[i].object_id, enumerator.GetObjectId());
80 EXPECT_EQ(kTestCases[i].size, enumerator.Size()); 80 EXPECT_EQ(kTestCases[i].size, enumerator.Size());
81 EXPECT_EQ(kTestCases[i].is_directory, enumerator.IsDirectory()); 81 EXPECT_EQ(kTestCases[i].is_directory, enumerator.IsDirectory());
82 EXPECT_EQ(kTestCases[i].last_modified_time, 82 EXPECT_EQ(kTestCases[i].last_modified_time,
83 enumerator.LastModifiedTime().ToTimeT()); 83 enumerator.LastModifiedTime().ToTimeT());
84 } 84 }
85 TestNextEntryIsEmpty(&enumerator); 85 TestNextEntryIsEmpty(&enumerator);
86 TestNextEntryIsEmpty(&enumerator); 86 TestNextEntryIsEmpty(&enumerator);
87 TestEnumeratorIsEmpty(&enumerator); 87 TestEnumeratorIsEmpty(&enumerator);
88 TestNextEntryIsEmpty(&enumerator); 88 TestNextEntryIsEmpty(&enumerator);
89 TestEnumeratorIsEmpty(&enumerator); 89 TestEnumeratorIsEmpty(&enumerator);
90 } 90 }
91 91
92 } // namespace 92 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698