Allow custom deleters to opt out of self reset checks for scoped_ptr.

base/memory/scoped_ptr_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/memory/scoped_ptr.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 638 matching lines @@
   scoped_ptr<Sub> sub2(new Sub);
 
   // Upcast with Pass() works.
   scoped_ptr<Super> super1 = sub1.Pass();
   super1 = sub2.Pass();
 
   // Upcast with an rvalue works.
   scoped_ptr<Super> super2 = SubClassReturn();
   super2 = SubClassReturn();
 }
+
+#if defined(GTEST_HAS_DEATH_TEST)
+TEST(ScopedPtrTest, SelfResetAbortsWithDefaultDeleter) {
+  scoped_ptr<int> x(new int);
+  EXPECT_DEATH(x.reset(x.get()), "");
+}
+
+TEST(ScopedPtrTest, SelfResetAbortsWithDefaultArrayDeleter) {
+  scoped_ptr<int[]> y(new int[4]);
+  EXPECT_DEATH(y.reset(y.get()), "");
+}
+
+TEST(ScopedPtrTest, SelfResetAbortsWithDefaultFreeDeleter) {

[pneubeck (no reviews), 2014/10/07 09:30:08]

It's a bit irritating that you test with a 'default' deleter here: I had to read
the scoped_ptr implementation to find out whether FreeDeleter is somehow special
handled.

shouldn't this test that a custom deleter without using the opt-out still
aborts?
i.e. the same as SelfResetWithCustomDeleter but without 'typedef void
AllowSelfReset;' ?

[dcheng, 2014/10/07 22:40:37]

I don't completely follow what you'd like me to change for this comment.
Done.

+  scoped_ptr<int, base::FreeDeleter> z(static_cast<int*>(malloc(sizeof(int))));
+  EXPECT_DEATH(z.reset(z.get()), "");
+}
+#endif
+
+TEST(ScopedPtrTest, SelfResetWithCustomDeleter) {
+  // A custom deleter should be able to opt out of self-reset abort behavior.
+  struct NoOpDeleter {
+    typedef void AllowSelfReset;
+    inline void operator()(int*) {}
+  };
+  scoped_ptr<int> owner(new int);
+  scoped_ptr<int, NoOpDeleter> x(owner.get());
+  x.reset(x.get());
+}