diff options
author | tsepez <tsepez@chromium.org> | 2016-11-04 16:00:03 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-11-04 16:00:03 -0700 |
commit | 931d087b991a986cc8bfa72131fe8eda0d987fc4 (patch) | |
tree | 53099f287c7c1dc83349aae6d3930a9462767518 /core/fpdfapi/parser/cpdf_array_unittest.cpp | |
parent | 1d08c50d1c20798001a1944e8a1fd76d09643ef1 (diff) | |
download | pdfium-931d087b991a986cc8bfa72131fe8eda0d987fc4.tar.xz |
Pass object to ReplaceIndirectObjectIfHigherGeneration() by unique_ptr.
Review-Url: https://codereview.chromium.org/2477003002
Diffstat (limited to 'core/fpdfapi/parser/cpdf_array_unittest.cpp')
-rw-r--r-- | core/fpdfapi/parser/cpdf_array_unittest.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/core/fpdfapi/parser/cpdf_array_unittest.cpp b/core/fpdfapi/parser/cpdf_array_unittest.cpp index 800afb0f9a..b1a4605666 100644 --- a/core/fpdfapi/parser/cpdf_array_unittest.cpp +++ b/core/fpdfapi/parser/cpdf_array_unittest.cpp @@ -3,12 +3,13 @@ // found in the LICENSE file. #include "core/fpdfapi/parser/cpdf_array.h" -#include "core/fpdfapi/parser/cpdf_number.h" -#include "core/fpdfapi/parser/cpdf_reference.h" #include <memory> +#include "core/fpdfapi/parser/cpdf_number.h" +#include "core/fpdfapi/parser/cpdf_reference.h" #include "testing/gtest/include/gtest/gtest.h" +#include "third_party/base/ptr_util.h" TEST(cpdf_array, RemoveAt) { { @@ -107,10 +108,11 @@ TEST(cpdf_array, Clone) { for (size_t i = 0; i < kNumOfRows; ++i) { CPDF_Array* arr_elem = new CPDF_Array; for (size_t j = 0; j < kNumOfRowElems; ++j) { - CPDF_Number* obj = new CPDF_Number(elems[i][j]); + std::unique_ptr<CPDF_Number> obj(new CPDF_Number(elems[i][j])); // Starts object number from 1. int obj_num = i * kNumOfRowElems + j + 1; - obj_holder->ReplaceIndirectObjectIfHigherGeneration(obj_num, obj); + obj_holder->ReplaceIndirectObjectIfHigherGeneration(obj_num, + std::move(obj)); arr_elem->InsertAt(j, new CPDF_Reference(obj_holder.get(), obj_num)); } arr->InsertAt(i, arr_elem); |