Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(131)

Side by Side Diff: chrome/browser/chromeos/extensions/file_manager/private_api_file_system.cc

Issue 564043004: Remove implicit conversions from scoped_refptr to T* in extensions code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/extensions/file_manager/private_api_file_syste m.h" 5 #include "chrome/browser/chromeos/extensions/file_manager/private_api_file_syste m.h"
6 6
7 #include <sys/statvfs.h> 7 #include <sys/statvfs.h>
8 8
9 #include "base/posix/eintr_wrapper.h" 9 #include "base/posix/eintr_wrapper.h"
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
(...skipping 602 matching lines...) Expand 10 before | Expand all | Expand 10 after
613 613
614 bool FileManagerPrivateInternalResolveIsolatedEntriesFunction::RunAsync() { 614 bool FileManagerPrivateInternalResolveIsolatedEntriesFunction::RunAsync() {
615 using extensions::api::file_manager_private_internal::ResolveIsolatedEntries:: 615 using extensions::api::file_manager_private_internal::ResolveIsolatedEntries::
616 Params; 616 Params;
617 const scoped_ptr<Params> params(Params::Create(*args_)); 617 const scoped_ptr<Params> params(Params::Create(*args_));
618 EXTENSION_FUNCTION_VALIDATE(params); 618 EXTENSION_FUNCTION_VALIDATE(params);
619 619
620 scoped_refptr<storage::FileSystemContext> file_system_context = 620 scoped_refptr<storage::FileSystemContext> file_system_context =
621 file_manager::util::GetFileSystemContextForRenderViewHost( 621 file_manager::util::GetFileSystemContextForRenderViewHost(
622 GetProfile(), render_view_host()); 622 GetProfile(), render_view_host());
623 DCHECK(file_system_context); 623 DCHECK(file_system_context.get());
624 624
625 const storage::ExternalFileSystemBackend* external_backend = 625 const storage::ExternalFileSystemBackend* external_backend =
626 file_system_context->external_backend(); 626 file_system_context->external_backend();
627 DCHECK(external_backend); 627 DCHECK(external_backend);
628 628
629 file_manager::util::FileDefinitionList file_definition_list; 629 file_manager::util::FileDefinitionList file_definition_list;
630 for (size_t i = 0; i < params->urls.size(); ++i) { 630 for (size_t i = 0; i < params->urls.size(); ++i) {
631 FileSystemURL fileSystemUrl = 631 FileSystemURL fileSystemUrl =
632 file_system_context->CrackURL(GURL(params->urls[i])); 632 file_system_context->CrackURL(GURL(params->urls[i]));
633 DCHECK(external_backend->CanHandleType(fileSystemUrl.type())); 633 DCHECK(external_backend->CanHandleType(fileSystemUrl.type()));
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
673 "/" + entry_definition_list->at(i).full_path.AsUTF8Unsafe(); 673 "/" + entry_definition_list->at(i).full_path.AsUTF8Unsafe();
674 entry->file_is_directory = entry_definition_list->at(i).is_directory; 674 entry->file_is_directory = entry_definition_list->at(i).is_directory;
675 entries.push_back(entry); 675 entries.push_back(entry);
676 } 676 }
677 677
678 results_ = extensions::api::file_manager_private_internal:: 678 results_ = extensions::api::file_manager_private_internal::
679 ResolveIsolatedEntries::Results::Create(entries); 679 ResolveIsolatedEntries::Results::Create(entries);
680 SendResponse(true); 680 SendResponse(true);
681 } 681 }
682 } // namespace extensions 682 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698