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

Unified Diff: chrome/browser/chromeos/extensions/file_manager/private_api_drive.h

Issue 557223003: Files.app: Remove getDriveFiles function from the private API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed. 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/extensions/file_manager/private_api_drive.h
diff --git a/chrome/browser/chromeos/extensions/file_manager/private_api_drive.h b/chrome/browser/chromeos/extensions/file_manager/private_api_drive.h
index 9c7be6abd0061c68ee627b81c632b4ca50cd5281..3b00722cbf533a7118949b5ac4d9e5bf579e7b56 100644
--- a/chrome/browser/chromeos/extensions/file_manager/private_api_drive.h
+++ b/chrome/browser/chromeos/extensions/file_manager/private_api_drive.h
@@ -78,43 +78,6 @@ class FileManagerPrivatePinDriveFileFunction
void OnPinStateSet(drive::FileError error);
};
-// Get drive files for the given list of file URLs. Initiate downloading of
-// drive files if these are not cached. Return a list of local file names.
-// This function puts empty strings instead of local paths for files could
-// not be obtained. For instance, this can happen if the user specifies a new
-// file name to save a file on drive. There may be other reasons to fail. The
-// file manager should check if the local paths returned from getDriveFiles()
-// contain empty paths.
-// TODO(satorux): Should we propagate error types to the JavaScript layer?
-class FileManagerPrivateGetDriveFilesFunction
- : public LoggedAsyncExtensionFunction {
- public:
- DECLARE_EXTENSION_FUNCTION("fileManagerPrivate.getDriveFiles",
- FILEMANAGERPRIVATE_GETDRIVEFILES)
-
- FileManagerPrivateGetDriveFilesFunction();
-
- protected:
- virtual ~FileManagerPrivateGetDriveFilesFunction();
-
- // AsyncExtensionFunction overrides.
- virtual bool RunAsync() OVERRIDE;
-
- private:
- // Gets the file on the top of the |remaining_drive_paths_| or sends the
- // response if the queue is empty.
- void GetFileOrSendResponse();
-
- // Called by FileSystem::GetFile(). Pops the file from
- // |remaining_drive_paths_|, and calls GetFileOrSendResponse().
- void OnFileReady(drive::FileError error,
- const base::FilePath& local_path,
- scoped_ptr<drive::ResourceEntry> entry);
-
- std::queue<base::FilePath> remaining_drive_paths_;
- std::vector<std::string> local_paths_;
-};
-
// Implements the chrome.fileManagerPrivate.cancelFileTransfers method.
class FileManagerPrivateCancelFileTransfersFunction
: public LoggedAsyncExtensionFunction {

Powered by Google App Engine
This is Rietveld 408576698