Remove implicit conversions from scoped_refptr to T* in c/b/chromeos/file*

chrome/browser/chromeos/file_manager/file_browser_handlers.cc

Index: chrome/browser/chromeos/file_manager/file_browser_handlers.cc
diff --git a/chrome/browser/chromeos/file_manager/file_browser_handlers.cc b/chrome/browser/chromeos/file_manager/file_browser_handlers.cc
index e7d2e2ad668d427cd8f82424b0cf3e08f23c043c..a8599695d918885a83760364fd20fb0f776e0737 100644
--- a/chrome/browser/chromeos/file_manager/file_browser_handlers.cc
+++ b/chrome/browser/chromeos/file_manager/file_browser_handlers.cc
@@ -407,8 +407,8 @@ void FileBrowserHandlerExecutor::SetupHandlerHostFileAccessPermissions(
     FileDefinitionList* file_definition_list,
     const Extension* extension,

[mtomasz, 2014/09/11 01:59:04]

The |extension| argument seems unused, and it's confusing. Could you remove
please since you're here?

[dcheng, 2014/09/11 02:04:07]

I'm happy to do this cleanup, but would you mind if I removed it in a followup
CL? This was split off a large master diff locally, and deferring this to a
separate patch will make merges a bit easier for me.

     int handler_pid) {
-  const FileBrowserHandler* action = FindFileBrowserHandlerForActionId(
-      extension_, action_id_);
+  const FileBrowserHandler* action =
+      FindFileBrowserHandlerForActionId(extension_.get(), action_id_);
   for (FileDefinitionList::const_iterator iter = file_definition_list->begin();
        iter != file_definition_list->end();
        ++iter) {