Manual fixups in PPAPI for scoped_refptr operator T* removal.

content/browser/renderer_host/pepper/pepper_file_io_host.cc

 // Copyright 2013 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 "content/browser/renderer_host/pepper/pepper_file_io_host.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/callback_helpers.h"
 #include "base/files/file_util_proxy.h"
@@ skipping 91 matching lines @@
 }
 
 }  // namespace
 
 PepperFileIOHost::PepperFileIOHost(BrowserPpapiHostImpl* host,
                                    PP_Instance instance,
                                    PP_Resource resource)
     : ResourceHost(host->GetPpapiHost(), instance, resource),
       browser_ppapi_host_(host),
       render_process_host_(NULL),
-      file_(BrowserThread::GetMessageLoopProxyForThread(BrowserThread::FILE)),
+      // TODO(dcheng): This seems sketchy. Does this not need to be ref'ed?
+      file_(BrowserThread::GetMessageLoopProxyForThread(BrowserThread::FILE)

[dcheng, 2014/08/28 16:40:16]

Do we need to ref this? Most other places seem to ref their task runner.

[raymes, 2014/08/29 01:12:45]

It looks like the base::FileProxy holds a ref, so I think it's ok :)
https://code.google.com/p/chromium/codesearch#chromium/src/base/files/file_pr...

[dcheng, 2014/08/29 06:38:01]

OK I've removed this comment then.

+                .get()),
       open_flags_(0),
       file_system_type_(PP_FILESYSTEMTYPE_INVALID),
       max_written_offset_(0),
       check_quota_(false) {
   int unused;
   if (!host->GetRenderFrameIDsForInstance(
           instance, &render_process_id_, &unused)) {
     render_process_id_ = -1;
   }
 }
@@ skipping 365 matching lines @@
 
   ppapi::proxy::SerializedHandle file_handle;
   // A non-zero resource id signals NaClIPCAdapter to create a NaClQuotaDesc.
   PP_Resource quota_file_io = check_quota_ ? pp_resource() : 0;
   file_handle.set_file_handle(transit_file, open_flags, quota_file_io);
   reply_context->params.AppendHandle(file_handle);
   return true;
 }
 
 }  // namespace content