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

Unified Diff: third_party/crashpad/crashpad/minidump/minidump_misc_info_writer_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/minidump/minidump_misc_info_writer_test.cc
diff --git a/third_party/crashpad/crashpad/minidump/minidump_misc_info_writer_test.cc b/third_party/crashpad/crashpad/minidump/minidump_misc_info_writer_test.cc
index 71fdbfbd7782287fceca7efbdd96f7140218b4a1..74a417013bd13d51d039532e72a45213a9e9b391 100644
--- a/third_party/crashpad/crashpad/minidump/minidump_misc_info_writer_test.cc
+++ b/third_party/crashpad/crashpad/minidump/minidump_misc_info_writer_test.cc
@@ -49,7 +49,7 @@ void GetMiscInfoStream(const std::string& file_contents, const T** misc_info) {
const size_t kMiscInfoStreamSize = sizeof(T);
const size_t kFileSize = kMiscInfoStreamOffset + kMiscInfoStreamSize;
- ASSERT_EQ(kFileSize, file_contents.size());
+ ASSERT_EQ(file_contents.size(), kFileSize);
const MINIDUMP_DIRECTORY* directory;
const MINIDUMP_HEADER* header =
@@ -57,8 +57,8 @@ void GetMiscInfoStream(const std::string& file_contents, const T** misc_info) {
ASSERT_NO_FATAL_FAILURE(VerifyMinidumpHeader(header, 1, 0));
ASSERT_TRUE(directory);
- ASSERT_EQ(kMinidumpStreamTypeMiscInfo, directory[0].StreamType);
- EXPECT_EQ(kMiscInfoStreamOffset, directory[0].Location.Rva);
+ ASSERT_EQ(directory[0].StreamType, kMinidumpStreamTypeMiscInfo);
+ EXPECT_EQ(directory[0].Location.Rva, kMiscInfoStreamOffset);
*misc_info = MinidumpWritableAtLocationDescriptor<T>(file_contents,
directory[0].Location);
@@ -70,19 +70,19 @@ void ExpectNULPaddedString16Equal(const base::char16* expected,
size_t size) {
base::string16 expected_string(expected, size);
base::string16 observed_string(observed, size);
- EXPECT_EQ(expected_string, observed_string);
+ EXPECT_EQ(observed_string, expected_string);
}
void ExpectSystemTimeEqual(const SYSTEMTIME* expected,
const SYSTEMTIME* observed) {
- EXPECT_EQ(expected->wYear, observed->wYear);
- EXPECT_EQ(expected->wMonth, observed->wMonth);
- EXPECT_EQ(expected->wDayOfWeek, observed->wDayOfWeek);
- EXPECT_EQ(expected->wDay, observed->wDay);
- EXPECT_EQ(expected->wHour, observed->wHour);
- EXPECT_EQ(expected->wMinute, observed->wMinute);
- EXPECT_EQ(expected->wSecond, observed->wSecond);
- EXPECT_EQ(expected->wMilliseconds, observed->wMilliseconds);
+ EXPECT_EQ(observed->wYear, expected->wYear);
+ EXPECT_EQ(observed->wMonth, expected->wMonth);
+ EXPECT_EQ(observed->wDayOfWeek, expected->wDayOfWeek);
+ EXPECT_EQ(observed->wDay, expected->wDay);
+ EXPECT_EQ(observed->wHour, expected->wHour);
+ EXPECT_EQ(observed->wMinute, expected->wMinute);
+ EXPECT_EQ(observed->wSecond, expected->wSecond);
+ EXPECT_EQ(observed->wMilliseconds, expected->wMilliseconds);
}
template <typename T>
@@ -92,11 +92,11 @@ template <>
void ExpectMiscInfoEqual<MINIDUMP_MISC_INFO>(
const MINIDUMP_MISC_INFO* expected,
const MINIDUMP_MISC_INFO* observed) {
- EXPECT_EQ(expected->Flags1, observed->Flags1);
- EXPECT_EQ(expected->ProcessId, observed->ProcessId);
- EXPECT_EQ(expected->ProcessCreateTime, observed->ProcessCreateTime);
- EXPECT_EQ(expected->ProcessUserTime, observed->ProcessUserTime);
- EXPECT_EQ(expected->ProcessKernelTime, observed->ProcessKernelTime);
+ EXPECT_EQ(observed->Flags1, expected->Flags1);
+ EXPECT_EQ(observed->ProcessId, expected->ProcessId);
+ EXPECT_EQ(observed->ProcessCreateTime, expected->ProcessCreateTime);
+ EXPECT_EQ(observed->ProcessUserTime, expected->ProcessUserTime);
+ EXPECT_EQ(observed->ProcessKernelTime, expected->ProcessKernelTime);
}
template <>
@@ -106,12 +106,12 @@ void ExpectMiscInfoEqual<MINIDUMP_MISC_INFO_2>(
ExpectMiscInfoEqual<MINIDUMP_MISC_INFO>(
reinterpret_cast<const MINIDUMP_MISC_INFO*>(expected),
reinterpret_cast<const MINIDUMP_MISC_INFO*>(observed));
- EXPECT_EQ(expected->ProcessorMaxMhz, observed->ProcessorMaxMhz);
- EXPECT_EQ(expected->ProcessorCurrentMhz, observed->ProcessorCurrentMhz);
- EXPECT_EQ(expected->ProcessorMhzLimit, observed->ProcessorMhzLimit);
- EXPECT_EQ(expected->ProcessorMaxIdleState, observed->ProcessorMaxIdleState);
- EXPECT_EQ(expected->ProcessorCurrentIdleState,
- observed->ProcessorCurrentIdleState);
+ EXPECT_EQ(observed->ProcessorMaxMhz, expected->ProcessorMaxMhz);
+ EXPECT_EQ(observed->ProcessorCurrentMhz, expected->ProcessorCurrentMhz);
+ EXPECT_EQ(observed->ProcessorMhzLimit, expected->ProcessorMhzLimit);
+ EXPECT_EQ(observed->ProcessorMaxIdleState, expected->ProcessorMaxIdleState);
+ EXPECT_EQ(observed->ProcessorCurrentIdleState,
+ expected->ProcessorCurrentIdleState);
}
template <>
@@ -121,11 +121,11 @@ void ExpectMiscInfoEqual<MINIDUMP_MISC_INFO_3>(
ExpectMiscInfoEqual<MINIDUMP_MISC_INFO_2>(
reinterpret_cast<const MINIDUMP_MISC_INFO_2*>(expected),
reinterpret_cast<const MINIDUMP_MISC_INFO_2*>(observed));
- EXPECT_EQ(expected->ProcessIntegrityLevel, observed->ProcessIntegrityLevel);
- EXPECT_EQ(expected->ProcessExecuteFlags, observed->ProcessExecuteFlags);
- EXPECT_EQ(expected->ProtectedProcess, observed->ProtectedProcess);
- EXPECT_EQ(expected->TimeZoneId, observed->TimeZoneId);
- EXPECT_EQ(expected->TimeZone.Bias, observed->TimeZone.Bias);
+ EXPECT_EQ(observed->ProcessIntegrityLevel, expected->ProcessIntegrityLevel);
+ EXPECT_EQ(observed->ProcessExecuteFlags, expected->ProcessExecuteFlags);
+ EXPECT_EQ(observed->ProtectedProcess, expected->ProtectedProcess);
+ EXPECT_EQ(observed->TimeZoneId, expected->TimeZoneId);
+ EXPECT_EQ(observed->TimeZone.Bias, expected->TimeZone.Bias);
{
SCOPED_TRACE("Standard");
ExpectNULPaddedString16Equal(expected->TimeZone.StandardName,
@@ -133,7 +133,7 @@ void ExpectMiscInfoEqual<MINIDUMP_MISC_INFO_3>(
arraysize(expected->TimeZone.StandardName));
ExpectSystemTimeEqual(&expected->TimeZone.StandardDate,
&observed->TimeZone.StandardDate);
- EXPECT_EQ(expected->TimeZone.StandardBias, observed->TimeZone.StandardBias);
+ EXPECT_EQ(observed->TimeZone.StandardBias, expected->TimeZone.StandardBias);
}
{
SCOPED_TRACE("Daylight");
@@ -142,7 +142,7 @@ void ExpectMiscInfoEqual<MINIDUMP_MISC_INFO_3>(
arraysize(expected->TimeZone.DaylightName));
ExpectSystemTimeEqual(&expected->TimeZone.DaylightDate,
&observed->TimeZone.DaylightDate);
- EXPECT_EQ(expected->TimeZone.DaylightBias, observed->TimeZone.DaylightBias);
+ EXPECT_EQ(observed->TimeZone.DaylightBias, expected->TimeZone.DaylightBias);
}
}
@@ -174,20 +174,20 @@ void ExpectMiscInfoEqual<MINIDUMP_MISC_INFO_5>(
ExpectMiscInfoEqual<MINIDUMP_MISC_INFO_4>(
reinterpret_cast<const MINIDUMP_MISC_INFO_4*>(expected),
reinterpret_cast<const MINIDUMP_MISC_INFO_4*>(observed));
- EXPECT_EQ(expected->XStateData.SizeOfInfo, observed->XStateData.SizeOfInfo);
- EXPECT_EQ(expected->XStateData.ContextSize, observed->XStateData.ContextSize);
- EXPECT_EQ(expected->XStateData.EnabledFeatures,
- observed->XStateData.EnabledFeatures);
+ EXPECT_EQ(observed->XStateData.SizeOfInfo, expected->XStateData.SizeOfInfo);
+ EXPECT_EQ(observed->XStateData.ContextSize, expected->XStateData.ContextSize);
+ EXPECT_EQ(observed->XStateData.EnabledFeatures,
+ expected->XStateData.EnabledFeatures);
for (size_t feature_index = 0;
feature_index < arraysize(observed->XStateData.Features);
++feature_index) {
SCOPED_TRACE(base::StringPrintf("feature_index %" PRIuS, feature_index));
- EXPECT_EQ(expected->XStateData.Features[feature_index].Offset,
- observed->XStateData.Features[feature_index].Offset);
- EXPECT_EQ(expected->XStateData.Features[feature_index].Size,
- observed->XStateData.Features[feature_index].Size);
+ EXPECT_EQ(observed->XStateData.Features[feature_index].Offset,
+ expected->XStateData.Features[feature_index].Offset);
+ EXPECT_EQ(observed->XStateData.Features[feature_index].Size,
+ expected->XStateData.Features[feature_index].Size);
}
- EXPECT_EQ(expected->ProcessCookie, observed->ProcessCookie);
+ EXPECT_EQ(observed->ProcessCookie, expected->ProcessCookie);
}
TEST(MinidumpMiscInfoWriter, Empty) {

Powered by Google App Engine
This is Rietveld 408576698