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

Unified Diff: third_party/tcmalloc/chromium/src/tests/atomicops_unittest.cc

Issue 1076002: Revert 41938 - Merged third_party/tcmalloc/vendor/src(googleperftools r87) in... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: third_party/tcmalloc/chromium/src/tests/atomicops_unittest.cc
===================================================================
--- third_party/tcmalloc/chromium/src/tests/atomicops_unittest.cc (revision 41940)
+++ third_party/tcmalloc/chromium/src/tests/atomicops_unittest.cc (working copy)
@@ -60,50 +60,50 @@
s.count = 0;
s.next_word = next_word_value;
- ASSERT_EQ(1, base::subtle::NoBarrier_AtomicIncrement(&s.count, 1));
- ASSERT_EQ(1, s.count);
- ASSERT_EQ(prev_word_value, s.prev_word);
- ASSERT_EQ(next_word_value, s.next_word);
+ CHECK_EQ(base::subtle::NoBarrier_AtomicIncrement(&s.count, 1), 1);
+ CHECK_EQ(s.count, 1);
+ CHECK_EQ(s.prev_word, prev_word_value);
+ CHECK_EQ(s.next_word, next_word_value);
- ASSERT_EQ(3, base::subtle::NoBarrier_AtomicIncrement(&s.count, 2));
- ASSERT_EQ(3, s.count);
- ASSERT_EQ(prev_word_value, s.prev_word);
- ASSERT_EQ(next_word_value, s.next_word);
+ CHECK_EQ(base::subtle::NoBarrier_AtomicIncrement(&s.count, 2), 3);
+ CHECK_EQ(s.count, 3);
+ CHECK_EQ(s.prev_word, prev_word_value);
+ CHECK_EQ(s.next_word, next_word_value);
- ASSERT_EQ(6, base::subtle::NoBarrier_AtomicIncrement(&s.count, 3));
- ASSERT_EQ(6, s.count);
- ASSERT_EQ(prev_word_value, s.prev_word);
- ASSERT_EQ(next_word_value, s.next_word);
+ CHECK_EQ(base::subtle::NoBarrier_AtomicIncrement(&s.count, 3), 6);
+ CHECK_EQ(s.count, 6);
+ CHECK_EQ(s.prev_word, prev_word_value);
+ CHECK_EQ(s.next_word, next_word_value);
- ASSERT_EQ(3, base::subtle::NoBarrier_AtomicIncrement(&s.count, -3));
- ASSERT_EQ(3, s.count);
- ASSERT_EQ(prev_word_value, s.prev_word);
- ASSERT_EQ(next_word_value, s.next_word);
+ CHECK_EQ(base::subtle::NoBarrier_AtomicIncrement(&s.count, -3), 3);
+ CHECK_EQ(s.count, 3);
+ CHECK_EQ(s.prev_word, prev_word_value);
+ CHECK_EQ(s.next_word, next_word_value);
- ASSERT_EQ(1, base::subtle::NoBarrier_AtomicIncrement(&s.count, -2));
- ASSERT_EQ(1, s.count);
- ASSERT_EQ(prev_word_value, s.prev_word);
- ASSERT_EQ(next_word_value, s.next_word);
+ CHECK_EQ(base::subtle::NoBarrier_AtomicIncrement(&s.count, -2), 1);
+ CHECK_EQ(s.count, 1);
+ CHECK_EQ(s.prev_word, prev_word_value);
+ CHECK_EQ(s.next_word, next_word_value);
- ASSERT_EQ(0, base::subtle::NoBarrier_AtomicIncrement(&s.count, -1));
- ASSERT_EQ(0, s.count);
- ASSERT_EQ(prev_word_value, s.prev_word);
- ASSERT_EQ(next_word_value, s.next_word);
+ CHECK_EQ(base::subtle::NoBarrier_AtomicIncrement(&s.count, -1), 0);
+ CHECK_EQ(s.count, 0);
+ CHECK_EQ(s.prev_word, prev_word_value);
+ CHECK_EQ(s.next_word, next_word_value);
- ASSERT_EQ(-1, base::subtle::NoBarrier_AtomicIncrement(&s.count, -1));
- ASSERT_EQ(-1, s.count);
- ASSERT_EQ(prev_word_value, s.prev_word);
- ASSERT_EQ(next_word_value, s.next_word);
+ CHECK_EQ(base::subtle::NoBarrier_AtomicIncrement(&s.count, -1), -1);
+ CHECK_EQ(s.count, -1);
+ CHECK_EQ(s.prev_word, prev_word_value);
+ CHECK_EQ(s.next_word, next_word_value);
- ASSERT_EQ(-5, base::subtle::NoBarrier_AtomicIncrement(&s.count, -4));
- ASSERT_EQ(-5, s.count);
- ASSERT_EQ(prev_word_value, s.prev_word);
- ASSERT_EQ(next_word_value, s.next_word);
+ CHECK_EQ(base::subtle::NoBarrier_AtomicIncrement(&s.count, -4), -5);
+ CHECK_EQ(s.count, -5);
+ CHECK_EQ(s.prev_word, prev_word_value);
+ CHECK_EQ(s.next_word, next_word_value);
- ASSERT_EQ(0, base::subtle::NoBarrier_AtomicIncrement(&s.count, 5));
- ASSERT_EQ(0, s.count);
- ASSERT_EQ(prev_word_value, s.prev_word);
- ASSERT_EQ(next_word_value, s.next_word);
+ CHECK_EQ(base::subtle::NoBarrier_AtomicIncrement(&s.count, 5), 0);
+ CHECK_EQ(s.count, 0);
+ CHECK_EQ(s.prev_word, prev_word_value);
+ CHECK_EQ(s.next_word, next_word_value);
}
@@ -114,8 +114,8 @@
static void TestCompareAndSwap() {
AtomicType value = 0;
AtomicType prev = base::subtle::NoBarrier_CompareAndSwap(&value, 0, 1);
- ASSERT_EQ(1, value);
- ASSERT_EQ(0, prev);
+ CHECK_EQ(1, value);
+ CHECK_EQ(0, prev);
// Use test value that has non-zero bits in both halves, more for testing
// 64-bit implementation on 32-bit platforms.
@@ -123,13 +123,13 @@
(NUM_BITS(AtomicType) - 2)) + 11;
value = k_test_val;
prev = base::subtle::NoBarrier_CompareAndSwap(&value, 0, 5);
- ASSERT_EQ(k_test_val, value);
- ASSERT_EQ(k_test_val, prev);
+ CHECK_EQ(k_test_val, value);
+ CHECK_EQ(k_test_val, prev);
value = k_test_val;
prev = base::subtle::NoBarrier_CompareAndSwap(&value, k_test_val, 5);
- ASSERT_EQ(5, value);
- ASSERT_EQ(k_test_val, prev);
+ CHECK_EQ(5, value);
+ CHECK_EQ(k_test_val, prev);
}
@@ -137,8 +137,8 @@
static void TestAtomicExchange() {
AtomicType value = 0;
AtomicType new_value = base::subtle::NoBarrier_AtomicExchange(&value, 1);
- ASSERT_EQ(1, value);
- ASSERT_EQ(0, new_value);
+ CHECK_EQ(1, value);
+ CHECK_EQ(0, new_value);
// Use test value that has non-zero bits in both halves, more for testing
// 64-bit implementation on 32-bit platforms.
@@ -146,13 +146,13 @@
(NUM_BITS(AtomicType) - 2)) + 11;
value = k_test_val;
new_value = base::subtle::NoBarrier_AtomicExchange(&value, k_test_val);
- ASSERT_EQ(k_test_val, value);
- ASSERT_EQ(k_test_val, new_value);
+ CHECK_EQ(k_test_val, value);
+ CHECK_EQ(k_test_val, new_value);
value = k_test_val;
new_value = base::subtle::NoBarrier_AtomicExchange(&value, 5);
- ASSERT_EQ(5, value);
- ASSERT_EQ(k_test_val, new_value);
+ CHECK_EQ(5, value);
+ CHECK_EQ(k_test_val, new_value);
}
@@ -163,11 +163,11 @@
AtomicType test_val = GG_ULONGLONG(1) << (NUM_BITS(AtomicType) / 2);
AtomicType value = test_val - 1;
AtomicType new_value = base::subtle::NoBarrier_AtomicIncrement(&value, 1);
- ASSERT_EQ(test_val, value);
- ASSERT_EQ(value, new_value);
+ CHECK_EQ(test_val, value);
+ CHECK_EQ(value, new_value);
base::subtle::NoBarrier_AtomicIncrement(&value, -1);
- ASSERT_EQ(test_val - 1, value);
+ CHECK_EQ(test_val - 1, value);
}
// This is a simple sanity check that values are correct. Not testing
@@ -180,19 +180,19 @@
AtomicType value;
base::subtle::NoBarrier_Store(&value, kVal1);
- ASSERT_EQ(kVal1, value);
+ CHECK_EQ(kVal1, value);
base::subtle::NoBarrier_Store(&value, kVal2);
- ASSERT_EQ(kVal2, value);
+ CHECK_EQ(kVal2, value);
base::subtle::Acquire_Store(&value, kVal1);
- ASSERT_EQ(kVal1, value);
+ CHECK_EQ(kVal1, value);
base::subtle::Acquire_Store(&value, kVal2);
- ASSERT_EQ(kVal2, value);
+ CHECK_EQ(kVal2, value);
base::subtle::Release_Store(&value, kVal1);
- ASSERT_EQ(kVal1, value);
+ CHECK_EQ(kVal1, value);
base::subtle::Release_Store(&value, kVal2);
- ASSERT_EQ(kVal2, value);
+ CHECK_EQ(kVal2, value);
}
// This is a simple sanity check that values are correct. Not testing
@@ -205,19 +205,19 @@
AtomicType value;
value = kVal1;
- ASSERT_EQ(kVal1, base::subtle::NoBarrier_Load(&value));
+ CHECK_EQ(kVal1, base::subtle::NoBarrier_Load(&value));
value = kVal2;
- ASSERT_EQ(kVal2, base::subtle::NoBarrier_Load(&value));
+ CHECK_EQ(kVal2, base::subtle::NoBarrier_Load(&value));
value = kVal1;
- ASSERT_EQ(kVal1, base::subtle::Acquire_Load(&value));
+ CHECK_EQ(kVal1, base::subtle::Acquire_Load(&value));
value = kVal2;
- ASSERT_EQ(kVal2, base::subtle::Acquire_Load(&value));
+ CHECK_EQ(kVal2, base::subtle::Acquire_Load(&value));
value = kVal1;
- ASSERT_EQ(kVal1, base::subtle::Release_Load(&value));
+ CHECK_EQ(kVal1, base::subtle::Release_Load(&value));
value = kVal2;
- ASSERT_EQ(kVal2, base::subtle::Release_Load(&value));
+ CHECK_EQ(kVal2, base::subtle::Release_Load(&value));
}
template <class AtomicType>
« no previous file with comments | « third_party/tcmalloc/chromium/src/tcmalloc.cc ('k') | third_party/tcmalloc/chromium/src/tests/debugallocation_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698