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

Unified Diff: third_party/crashpad/crashpad/util/mac/checked_mach_address_range_test.cc

Issue 2804713002: Update Crashpad to b4095401639ebe2ad33169e5c1d994065cbff1b8 (Closed)
Patch Set: Created 3 years, 8 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/crashpad/crashpad/util/mac/checked_mach_address_range_test.cc
diff --git a/third_party/crashpad/crashpad/util/mac/checked_mach_address_range_test.cc b/third_party/crashpad/crashpad/util/mac/checked_mach_address_range_test.cc
index c0126240c9dad3e65562737e5bf1ac8eb3e8c5f5..860bdff8c655161cb97f68de18a63ae00081d660 100644
--- a/third_party/crashpad/crashpad/util/mac/checked_mach_address_range_test.cc
+++ b/third_party/crashpad/crashpad/util/mac/checked_mach_address_range_test.cc
@@ -124,10 +124,10 @@ TEST(CheckedMachAddressRange, IsValid) {
testcase.size));
CheckedMachAddressRange range_32(false, testcase.base, testcase.size);
- EXPECT_EQ(ExpectationForValidity32(testcase.validity), range_32.IsValid());
+ EXPECT_EQ(range_32.IsValid(), ExpectationForValidity32(testcase.validity));
CheckedMachAddressRange range_64(true, testcase.base, testcase.size);
- EXPECT_EQ(ExpectationForValidity64(testcase.validity), range_64.IsValid());
+ EXPECT_EQ(range_64.IsValid(), ExpectationForValidity64(testcase.validity));
}
}
@@ -171,8 +171,8 @@ TEST(CheckedMachAddressRange, ContainsValue) {
SCOPED_TRACE(
base::StringPrintf("index %zu, value 0x%llx", index, testcase.value));
- EXPECT_EQ(testcase.expectation,
- parent_range_32.ContainsValue(testcase.value));
+ EXPECT_EQ(parent_range_32.ContainsValue(testcase.value),
+ testcase.expectation);
}
CheckedMachAddressRange parent_range_64(true, 0x100000000, 0x1000);
@@ -232,8 +232,8 @@ TEST(CheckedMachAddressRange, ContainsRange) {
CheckedMachAddressRange child_range_32(false, testcase.base, testcase.size);
ASSERT_TRUE(child_range_32.IsValid());
- EXPECT_EQ(testcase.expectation,
- parent_range_32.ContainsRange(child_range_32));
+ EXPECT_EQ(parent_range_32.ContainsRange(child_range_32),
+ testcase.expectation);
}
CheckedMachAddressRange parent_range_64(true, 0x100000000, 0x1000);

Powered by Google App Engine
This is Rietveld 408576698