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

Unified Diff: courgette/disassembler_elf_32_x86_unittest.cc

Issue 2771753004: [Courgette] Refactor: Unify Disassembler::Disassemble() and instantiate AssemblyProgram there. (Closed)
Patch Set: Created 3 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: courgette/disassembler_elf_32_x86_unittest.cc
diff --git a/courgette/disassembler_elf_32_x86_unittest.cc b/courgette/disassembler_elf_32_x86_unittest.cc
index 902aeeec213daec27dc4bf7d481b394c5ce26adf..e2d7368369e97619a4a28a5c62edde6a4d507f8d 100644
--- a/courgette/disassembler_elf_32_x86_unittest.cc
+++ b/courgette/disassembler_elf_32_x86_unittest.cc
@@ -76,6 +76,8 @@ void DisassemblerElf32X86Test::TestExe(const char* file_name,
bool can_parse_header = disassembler->ParseHeader();
EXPECT_TRUE(can_parse_header);
EXPECT_TRUE(disassembler->ok());
+ EXPECT_EQ(EXE_ELF_32_X86, disassembler->kind());
+ EXPECT_EQ(0, disassembler->image_base());
// The length of the disassembled value will be slightly smaller than the
// real file, since trailing debug info is not included
@@ -89,9 +91,8 @@ void DisassemblerElf32X86Test::TestExe(const char* file_name,
EXPECT_EQ('L', offset_p[2]);
EXPECT_EQ('F', offset_p[3]);
- auto program = base::MakeUnique<AssemblyProgram>(EXE_ELF_32_X86, 0);
-
- EXPECT_TRUE(disassembler->Disassemble(program.get()));
+ std::unique_ptr<AssemblyProgram> program = disassembler->Disassemble();
+ EXPECT_TRUE(nullptr != program.get());
const std::vector<RVA>& abs32_list = disassembler->Abs32Locations();

Powered by Google App Engine
This is Rietveld 408576698