Set drive as the default download folder

chrome/browser/chromeos/drive/file_system_util.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_UTIL_H_
 #define CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_UTIL_H_
 
 #include <string>
 
 #include "base/callback_forward.h"
@@ skipping 44 matching lines @@
 
 // Returns the path of the top root of the pseudo tree.
 const base::FilePath& GetDriveGrandRootPath();
 
 // Returns the path of the directory representing "My Drive".
 const base::FilePath& GetDriveMyDriveRootPath();
 
 // Returns the Drive mount point path, which looks like "/special/drive-<hash>".
 base::FilePath GetDriveMountPointPath(Profile* profile);
 
+// Returns the Drive mount point path, which looks like
+// "/special/drive-<username_hash>" when provided with the <username_hash>

[bartfab (slow), 2014/03/13 13:20:00]

Nit 1: Add comma before "when".
Nit 2: Add full stop at the end.
Nit 3: Surround symbols in comments with ||, not with <>.
Nit 4: Specifying the format of the mount path in the comment makes no sense to
me. The whole point of this function is to hide this detail from anyone wishing
to access Drive.

+base::FilePath GetDriveMountPointPathUsingUserHash(std::string username_hash);

[bartfab (slow), 2014/03/13 13:20:00]

Nit 1: After an unfortunate proliferation of many different names for the same
thing, we eventually settled on |user_id|. Thus, s/username_hash/user_id_hash/.
Nit 2: Make the argument a const reference.
Nit 3: How about a method name like GetDriveMountPointForUser() instead?

+
 // Returns the FileSystem for the |profile|. If not available (not mounted
 // or disabled), returns NULL.
 FileSystemInterface* GetFileSystemByProfile(Profile* profile);
 
 // Returns a FileSystemInterface instance for the |profile_id|, or NULL
 // if the Profile for |profile_id| is destructed or Drive File System is
 // disabled for the profile.
 // Note: |profile_id| should be the pointer of the Profile instance if it is
 // alive. Considering timing issues due to task posting across threads,
 // this function can accept a dangling pointer as |profile_id| (and will return
@@ skipping 137 matching lines @@
   DRIVE_CONNECTED,
 };
 
 // Returns the Drive connection status for the |profile|.
 ConnectionStatusType GetDriveConnectionStatus(Profile* profile);
 
 }  // namespace util
 }  // namespace drive
 
 #endif  // CHROME_BROWSER_CHROMEOS_DRIVE_FILE_SYSTEM_UTIL_H_