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

Side by Side Diff: courgette/encode_decode_unittest.cc

Issue 1855133002: convert //courgette to std::unique_ptr (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update comment in memory_allocator.h Created 4 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 unified diff | Download patch
« no previous file with comments | « courgette/disassembler_win32_x86_unittest.cc ('k') | courgette/encoded_program.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stddef.h> 5 #include <stddef.h>
6 6
7 #include "base/memory/scoped_ptr.h" 7 #include <memory>
8
8 #include "courgette/assembly_program.h" 9 #include "courgette/assembly_program.h"
9 #include "courgette/base_test_unittest.h" 10 #include "courgette/base_test_unittest.h"
10 #include "courgette/courgette.h" 11 #include "courgette/courgette.h"
11 #include "courgette/encoded_program.h" 12 #include "courgette/encoded_program.h"
12 #include "courgette/program_detector.h" 13 #include "courgette/program_detector.h"
13 #include "courgette/streams.h" 14 #include "courgette/streams.h"
14 15
15 class EncodeDecodeTest : public BaseTest { 16 class EncodeDecodeTest : public BaseTest {
16 public: 17 public:
17 void TestAssembleToStreamDisassemble(std::string file, 18 void TestAssembleToStreamDisassemble(std::string file,
18 size_t expected_encoded_lenth) const; 19 size_t expected_encoded_lenth) const;
19 }; 20 };
20 21
21 void EncodeDecodeTest::TestAssembleToStreamDisassemble( 22 void EncodeDecodeTest::TestAssembleToStreamDisassemble(
22 std::string file, 23 std::string file,
23 size_t expected_encoded_lenth) const { 24 size_t expected_encoded_lenth) const {
24 const void* original_buffer = file.c_str(); 25 const void* original_buffer = file.c_str();
25 size_t original_length = file.length(); 26 size_t original_length = file.length();
26 27
27 scoped_ptr<courgette::AssemblyProgram> program; 28 std::unique_ptr<courgette::AssemblyProgram> program;
28 const courgette::Status parse_status = 29 const courgette::Status parse_status =
29 courgette::ParseDetectedExecutable(original_buffer, 30 courgette::ParseDetectedExecutable(original_buffer,
30 original_length, 31 original_length,
31 &program); 32 &program);
32 EXPECT_EQ(courgette::C_OK, parse_status); 33 EXPECT_EQ(courgette::C_OK, parse_status);
33 34
34 scoped_ptr<courgette::EncodedProgram> encoded; 35 std::unique_ptr<courgette::EncodedProgram> encoded;
35 const courgette::Status encode_status = Encode(*program, &encoded); 36 const courgette::Status encode_status = Encode(*program, &encoded);
36 EXPECT_EQ(courgette::C_OK, encode_status); 37 EXPECT_EQ(courgette::C_OK, encode_status);
37 38
38 program.reset(); 39 program.reset();
39 40
40 courgette::SinkStreamSet sinks; 41 courgette::SinkStreamSet sinks;
41 const courgette::Status write_status = 42 const courgette::Status write_status =
42 WriteEncodedProgram(encoded.get(), &sinks); 43 WriteEncodedProgram(encoded.get(), &sinks);
43 EXPECT_EQ(courgette::C_OK, write_status); 44 EXPECT_EQ(courgette::C_OK, write_status);
44 45
45 encoded.reset(); 46 encoded.reset();
46 47
47 courgette::SinkStream sink; 48 courgette::SinkStream sink;
48 bool can_collect = sinks.CopyTo(&sink); 49 bool can_collect = sinks.CopyTo(&sink);
49 EXPECT_TRUE(can_collect); 50 EXPECT_TRUE(can_collect);
50 51
51 const void* buffer = sink.Buffer(); 52 const void* buffer = sink.Buffer();
52 size_t length = sink.Length(); 53 size_t length = sink.Length();
53 54
54 EXPECT_EQ(expected_encoded_lenth, length); 55 EXPECT_EQ(expected_encoded_lenth, length);
55 56
56 courgette::SourceStreamSet sources; 57 courgette::SourceStreamSet sources;
57 bool can_get_source_streams = sources.Init(buffer, length); 58 bool can_get_source_streams = sources.Init(buffer, length);
58 EXPECT_TRUE(can_get_source_streams); 59 EXPECT_TRUE(can_get_source_streams);
59 60
60 scoped_ptr<courgette::EncodedProgram> encoded2; 61 std::unique_ptr<courgette::EncodedProgram> encoded2;
61 const courgette::Status read_status = ReadEncodedProgram(&sources, &encoded2); 62 const courgette::Status read_status = ReadEncodedProgram(&sources, &encoded2);
62 EXPECT_EQ(courgette::C_OK, read_status); 63 EXPECT_EQ(courgette::C_OK, read_status);
63 64
64 courgette::SinkStream assembled; 65 courgette::SinkStream assembled;
65 const courgette::Status assemble_status = 66 const courgette::Status assemble_status =
66 Assemble(encoded2.get(), &assembled); 67 Assemble(encoded2.get(), &assembled);
67 EXPECT_EQ(courgette::C_OK, assemble_status); 68 EXPECT_EQ(courgette::C_OK, assemble_status);
68 69
69 encoded2.reset(); 70 encoded2.reset();
70 71
(...skipping 21 matching lines...) Expand all
92 93
93 TEST_F(EncodeDecodeTest, Elf_HighBSS) { 94 TEST_F(EncodeDecodeTest, Elf_HighBSS) {
94 std::string file = FileContents("elf-32-high-bss"); 95 std::string file = FileContents("elf-32-high-bss");
95 TestAssembleToStreamDisassemble(file, 7312); 96 TestAssembleToStreamDisassemble(file, 7312);
96 } 97 }
97 98
98 TEST_F(EncodeDecodeTest, Elf_Arm) { 99 TEST_F(EncodeDecodeTest, Elf_Arm) {
99 std::string file = FileContents("elf-armv7"); 100 std::string file = FileContents("elf-armv7");
100 TestAssembleToStreamDisassemble(file, 8531); 101 TestAssembleToStreamDisassemble(file, 8531);
101 } 102 }
OLDNEW
« no previous file with comments | « courgette/disassembler_win32_x86_unittest.cc ('k') | courgette/encoded_program.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698