Chromium Code Reviews| Index: minidump/minidump_handle_writer_test.cc |
| diff --git a/minidump/minidump_handle_writer_test.cc b/minidump/minidump_handle_writer_test.cc |
| index f2692068b47e245af9f1b53a889c1a1ca1fc7f6e..602ce1c37117d5689b3c06e59c24d438734287f9 100644 |
| --- a/minidump/minidump_handle_writer_test.cc |
| +++ b/minidump/minidump_handle_writer_test.cc |
| @@ -16,6 +16,7 @@ |
| #include <string> |
| +#include "base/strings/utf_string_conversions.h" |
| #include "gtest/gtest.h" |
| #include "minidump/minidump_file_writer.h" |
| #include "minidump/test/minidump_file_writer_test_util.h" |
| @@ -78,7 +79,7 @@ TEST(MinidumpHandleDataWriter, OneHandle) { |
| HandleSnapshot handle_snapshot; |
| handle_snapshot.handle = 0x1234; |
| - handle_snapshot.type_name = L"Something"; |
|
Mark Mentovai
2015/10/21 18:31:28
It looks like this was the only offending use. An
scottmg
2015/10/21 18:35:44
Yeah, that's what I tried at first, but the SetUTF
|
| + handle_snapshot.type_name = "Something"; |
| handle_snapshot.attributes = 0x12345678; |
| handle_snapshot.granted_access = 0x9abcdef0; |
| handle_snapshot.pointer_count = 4567; |
| @@ -95,8 +96,7 @@ TEST(MinidumpHandleDataWriter, OneHandle) { |
| ASSERT_TRUE(minidump_file_writer.WriteEverything(&string_file)); |
| const size_t kTypeNameStringDataLength = |
| - (handle_snapshot.type_name.size() + 1) * |
| - sizeof(handle_snapshot.type_name[0]); |
| + (handle_snapshot.type_name.size() + 1) * sizeof(base::char16); |
| ASSERT_EQ(sizeof(MINIDUMP_HEADER) + sizeof(MINIDUMP_DIRECTORY) + |
| sizeof(MINIDUMP_HANDLE_DATA_STREAM) + |
| sizeof(MINIDUMP_HANDLE_DESCRIPTOR) + sizeof(MINIDUMP_STRING) + |
| @@ -113,8 +113,8 @@ TEST(MinidumpHandleDataWriter, OneHandle) { |
| &handle_data_stream[1]); |
| EXPECT_EQ(handle_snapshot.handle, handle_descriptor->Handle); |
| EXPECT_EQ(handle_snapshot.type_name, |
| - MinidumpStringAtRVAAsString(string_file.string(), |
| - handle_descriptor->TypeNameRva)); |
| + base::UTF16ToUTF8(MinidumpStringAtRVAAsString( |
| + string_file.string(), handle_descriptor->TypeNameRva))); |
| EXPECT_EQ(0u, handle_descriptor->ObjectNameRva); |
| EXPECT_EQ(handle_snapshot.attributes, handle_descriptor->Attributes); |
| EXPECT_EQ(handle_snapshot.granted_access, handle_descriptor->GrantedAccess); |