Add RetainedRef uses where needed.

base/bind_helpers.h

 // Copyright (c) 2011 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.
 
 // This defines a set of argument wrappers and related factory methods that
 // can be used specify the refcounting and reference semantics of arguments
 // that are bound by the Bind() function in base/bind.h.
 //
 // It also defines a set of simple functions and utilities that people want
 // when using Callback<> and Bind().
@@ skipping 413 matching lines @@
 T* Unwrap(UnretainedWrapper<T> unretained) {
   return unretained.get();
 }
 
 template <typename T>
 const T& Unwrap(ConstRefWrapper<T> const_ref) {
   return const_ref.get();
 }
 
 template <typename T>
-T* Unwrap(const scoped_refptr<T>& o) {
-  return o.get();
-}
-
-template <typename T>

[Nico, 2016/03/18 21:01:01]

I'd land this bit in a different CL, so that it can be reverted independently of
the rest in case this breaks some obscure bot

[vmpstr, 2016/03/18 22:20:23]

Good idea. I've separated this. 

 T* Unwrap(const RetainedRefWrapper<T>& o) {
   return o.get();
 }
 
 template <typename T>
 const WeakPtr<T>& Unwrap(const WeakPtr<T>& o) {
   return o;
 }
 
 template <typename T>
@@ skipping 178 matching lines @@
 BASE_EXPORT void DoNothing();
 
 template<typename T>
 void DeletePointer(T* obj) {
   delete obj;
 }
 
 }  // namespace base
 
 #endif  // BASE_BIND_HELPERS_H_