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

Unified Diff: chrome/browser/chromeos/drive/drive_download_handler.h

Issue 12040095: chromeos: Rename DriveDownloadHandler::GetDrivePath() to GetTargetPath(), make it return full path (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: _ Created 7 years, 11 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
« no previous file with comments | « no previous file | chrome/browser/chromeos/drive/drive_download_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/drive/drive_download_handler.h
diff --git a/chrome/browser/chromeos/drive/drive_download_handler.h b/chrome/browser/chromeos/drive/drive_download_handler.h
index ec82b766db81b951581cbf4978e04b341e8f28cf..effe3436f5b0708c6e7b4a8609df5e6cc1de9c5a 100644
--- a/chrome/browser/chromeos/drive/drive_download_handler.h
+++ b/chrome/browser/chromeos/drive/drive_download_handler.h
@@ -54,11 +54,8 @@ class DriveDownloadHandler : public AllDownloadItemNotifier::Observer {
void SetDownloadParams(const FilePath& drive_path,
content::DownloadItem* download);
- // Gets the drive_path from external data in |download|.
- // GetDrivePath may return an empty path in case SetDrivePath was not
- // previously called or there was some other internal error
- // (there is a DCHECK for this).
- FilePath GetDrivePath(const content::DownloadItem* download);
+ // Gets the target drive path from external data in |download|.
+ FilePath GetTargetPath(const content::DownloadItem* download);
// Checks if there is a Drive upload associated with |download|
bool IsDriveDownload(const content::DownloadItem* download);
« no previous file with comments | « no previous file | chrome/browser/chromeos/drive/drive_download_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698