Remove CPDF_Object::Release() in favor of direct delete

core/fpdfapi/fpdf_parser/cpdf_object_unittest.cpp

Index: core/fpdfapi/fpdf_parser/cpdf_object_unittest.cpp
diff --git a/core/fpdfapi/fpdf_parser/cpdf_object_unittest.cpp b/core/fpdfapi/fpdf_parser/cpdf_object_unittest.cpp
index 2225b2ed057b7f54e906bfca30881698c39fba98..4b0ea0d13ddd9189a03ef28ae238e373cc247ae1 100644
--- a/core/fpdfapi/fpdf_parser/cpdf_object_unittest.cpp
+++ b/core/fpdfapi/fpdf_parser/cpdf_object_unittest.cpp
@@ -22,9 +22,8 @@
 
 namespace {
 
-using ScopedArray = std::unique_ptr<CPDF_Array, ReleaseDeleter<CPDF_Array>>;
-using ScopedDict =
-    std::unique_ptr<CPDF_Dictionary, ReleaseDeleter<CPDF_Dictionary>>;
+using ScopedArray = std::unique_ptr<CPDF_Array>;

[Lei Zhang, 2016/10/03 21:18:30]

We can also just get rid of these.

[Tom Sepez, 2016/10/03 21:41:31]

Done.

+using ScopedDict = std::unique_ptr<CPDF_Dictionary>;
 
 void TestArrayAccessors(const CPDF_Array* arr,
                         size_t index,
@@ -172,7 +171,7 @@ class PDFObjectsTest : public testing::Test {
   }
 
  protected:
-  using ScopedObj = std::unique_ptr<CPDF_Object, ReleaseDeleter<CPDF_Object>>;
+  using ScopedObj = std::unique_ptr<CPDF_Object>;
 
   // m_ObjHolder needs to be declared first and destructed last since it also
   // refers to some objects in m_DirectObjs.