Remove implicit conversions from scoped_refptr to T* in content/*/service_worker/

content/browser/service_worker/service_worker_cache.cc

Index: content/browser/service_worker/service_worker_cache.cc
diff --git a/content/browser/service_worker/service_worker_cache.cc b/content/browser/service_worker/service_worker_cache.cc
index 6a63b6b3e37d2dac223c3d911def4bc2636f3135..c2cfe64ee2c7a49a23dcace09ce7c6e3c2679dde 100644
--- a/content/browser/service_worker/service_worker_cache.cc
+++ b/content/browser/service_worker/service_worker_cache.cc
@@ -131,7 +131,7 @@ class BlobReader : public net::URLRequest::Delegate {
 
     int rv = entry_->WriteData(INDEX_RESPONSE_BODY,
                                cache_entry_offset_,
-                               buffer_,
+                               buffer_.get(),
                                bytes_read,
                                cache_write_callback,
                                true /* truncate */);
@@ -280,7 +280,7 @@ void PutDidCreateEntry(ServiceWorkerFetchRequest* request,
 
   rv = tmp_entry_ptr->WriteData(INDEX_HEADERS,
                                 0 /* offset */,
-                                buffer,
+                                buffer.get(),
                                 buffer->size(),
                                 write_headers_callback,
                                 true /* truncate */);
@@ -491,7 +491,7 @@ void MatchDidReadResponseBodyData(
   int total_bytes_read = response_context->total_bytes_read;
 
   // Grab some pointers before passing them in bind.
-  net::IOBufferWithSize* buffer = response_context->buffer;
+  net::IOBufferWithSize* buffer = response_context->buffer.get();

[dcheng, 2014/08/26 17:10:57]

Note: the tool believes this to be safe because it's not grabbing a pointer to
something on the stack. Please let me know if you believe that the variable type
should change instead.

   disk_cache::Entry* tmp_entry_ptr = entry.get();
 
   net::CompletionCallback read_callback =