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

Unified Diff: chrome/browser/chromeos/drive/download_handler.cc

Issue 15768004: drive: Rename drive::DriveFileSystemTest to drive::FileSystemTest (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 7 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/drive/download_handler.cc
diff --git a/chrome/browser/chromeos/drive/download_handler.cc b/chrome/browser/chromeos/drive/download_handler.cc
index 748860c39a7365fce7df30723b42ac570c8abfff..d97b69c3239c5147c4acc6b0f519be12488b62b3 100644
--- a/chrome/browser/chromeos/drive/download_handler.cc
+++ b/chrome/browser/chromeos/drive/download_handler.cc
@@ -147,8 +147,8 @@ void DownloadHandler::SubstituteDriveDownloadPath(
// OnCreateDirectory calls SubstituteDriveDownloadPathInternal
const base::FilePath drive_dir_path =
util::ExtractDrivePath(drive_path.DirName());
- // Ensure the directory exists. This also forces FileSystem to
- // initialize DriveRootDirectory.
+ // Check if the directory exists, and create it if the directory does not
+ // exist.
file_system_->GetResourceEntryByPath(
drive_dir_path,
base::Bind(&DownloadHandler::OnEntryFound,
@@ -188,7 +188,7 @@ base::FilePath DownloadHandler::GetTargetPath(
bool DownloadHandler::IsDriveDownload(const DownloadItem* download) {
// We use the existence of the DriveUserData object in download as a
- // signal that this is a DriveDownload.
+ // signal that this is a download to Drive.
return GetDriveUserData(download) != NULL;
}
« no previous file with comments | « chrome/browser/chromeos/drive/debug_info_collector.cc ('k') | chrome/browser/chromeos/drive/drive_url_request_job.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698