Remove scoped_refptr::release which is confusing and causes leaks.

base/memory/ref_counted.h

Index: base/memory/ref_counted.h
diff --git a/base/memory/ref_counted.h b/base/memory/ref_counted.h
index 98dce1d719e84361b8ca44fd3684ca92cd9e080e..0c6a71fdb3e7abd62669d13bd8617bdc0a5412ad 100644
--- a/base/memory/ref_counted.h
+++ b/base/memory/ref_counted.h
@@ -257,17 +257,6 @@ class scoped_refptr {
     return ptr_;
   }
 
-  // Release a pointer.
-  // The return value is the current pointer held by this object.
-  // If this object holds a NULL pointer, the return value is NULL.
-  // After this operation, this object will hold a NULL pointer,
-  // and will not own the object any more.
-  T* release() WARN_UNUSED_RESULT {
-    T* retVal = ptr_;
-    ptr_ = NULL;
-    return retVal;
-  }
-
   scoped_refptr<T>& operator=(T* p) {
     // AddRef first so that self assignment should work
     if (p)