Use std::default_delete as the default deleter for scoped_ptr.

crypto/scoped_nss_types.h

Index: crypto/scoped_nss_types.h
diff --git a/crypto/scoped_nss_types.h b/crypto/scoped_nss_types.h
index fdfb83c154f8c47996633031c9dcc45abb285e36..8e96e8d4e20ffedb34a529e62ec7e92451bd8c5d 100644
--- a/crypto/scoped_nss_types.h
+++ b/crypto/scoped_nss_types.h
@@ -16,7 +16,6 @@ namespace crypto {
 
 template <typename Type, void (*Destroyer)(Type*)>
 struct NSSDestroyer {
-  typedef void AllowSelfReset;

[danakj, 2015/11/17 19:09:27]

Can you maybe replace each of these AllowSelfReset with a comment saying that
this destroyer was written in a way that will allow self reset?

[dcheng, 2015/11/17 20:21:03]

Hmm... that's not really immediately obvious that comment would be true though
(as it depends on the actual Destroyer implementation), right? TBH, looking at
the CL that introduced this, I'm not even sure why it was needed in the end:
https://codereview.chromium.org/610533003

+davidben, is this something you'd find valuable?

[davidben, 2015/11/17 21:01:45]

Part of this change is to remove the self-reset check in scoped_ptr altogether,
right? If so, then there's probably no need to keep a mention around as scar
tissue. So this lgtm.

(Self-reset was needed with the old scoped_ptr behavior because NSS and OpenSSL
types are all intrusively reference-counted. And so two ScopedFOO might contain
the same FOO* pointer by value, but each owns its own reference to it.)

   void operator()(Type* ptr) const {
     Destroyer(ptr);
   }
@@ -24,7 +23,6 @@ struct NSSDestroyer {
 
 template <typename Type, void (*Destroyer)(Type*, PRBool), PRBool freeit>
 struct NSSDestroyer1 {
-  typedef void AllowSelfReset;
   void operator()(Type* ptr) const {
     Destroyer(ptr, freeit);
   }