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

Side by Side Diff: courgette/third_party/paged_array_unittest.cc

Issue 850783002: Disable two PagedArrayTests that fail under AddressSanitizer on Windows (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reword the comment Created 5 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 "courgette/third_party/paged_array.h" 5 #include "courgette/third_party/paged_array.h"
6 6
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 8
9 class PagedArrayTest : public testing::Test { 9 class PagedArrayTest : public testing::Test {
10 public: 10 public:
11 // Total allocation of 4GB will fail in 32 bit programs if allocations are 11 // Total allocation of 4GB will fail in 32 bit programs if allocations are
12 // leaked. 12 // leaked.
13 static const int kIterations = 20; 13 static const int kIterations = 20;
14 static const int kSize = 200 * 1024 * 1024 / sizeof(int); // 200MB 14 static const int kSize = 200 * 1024 * 1024 / sizeof(int); // 200MB
15 }; 15 };
16 16
17 // AddressSanitizer on Windows adds additional memory overhead, which
18 // causes these tests to go OOM and fail.
19 #if !defined(ADDRESS_SANITIZER) || !defined(OS_WIN)
17 TEST_F(PagedArrayTest, TestManyAllocationsDestructorFree) { 20 TEST_F(PagedArrayTest, TestManyAllocationsDestructorFree) {
18 for (int i = 0; i < kIterations; ++i) { 21 for (int i = 0; i < kIterations; ++i) {
19 courgette::PagedArray<int> a; 22 courgette::PagedArray<int> a;
20 EXPECT_TRUE(a.Allocate(kSize)); 23 EXPECT_TRUE(a.Allocate(kSize));
21 } 24 }
22 } 25 }
23 26
24 TEST_F(PagedArrayTest, TestManyAllocationsManualFree) { 27 TEST_F(PagedArrayTest, TestManyAllocationsManualFree) {
25 courgette::PagedArray<int> a; 28 courgette::PagedArray<int> a;
26 for (int i = 0; i < kIterations; ++i) { 29 for (int i = 0; i < kIterations; ++i) {
27 EXPECT_TRUE(a.Allocate(kSize)); 30 EXPECT_TRUE(a.Allocate(kSize));
28 a.clear(); 31 a.clear();
29 } 32 }
30 } 33 }
34 #endif
31 35
32 TEST_F(PagedArrayTest, TestAccess) { 36 TEST_F(PagedArrayTest, TestAccess) {
33 const int kAccessSize = 3 * 1024 * 1024; 37 const int kAccessSize = 3 * 1024 * 1024;
34 courgette::PagedArray<int> a; 38 courgette::PagedArray<int> a;
35 a.Allocate(kAccessSize); 39 a.Allocate(kAccessSize);
36 for (int i = 0; i < kAccessSize; ++i) { 40 for (int i = 0; i < kAccessSize; ++i) {
37 a[i] = i; 41 a[i] = i;
38 } 42 }
39 for (int i = 0; i < kAccessSize; ++i) { 43 for (int i = 0; i < kAccessSize; ++i) {
40 EXPECT_EQ(i, a[i]); 44 EXPECT_EQ(i, a[i]);
41 } 45 }
42 } 46 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698