drive: Switch Drive mount path to per-profile one.

drive: Switch Drive mount path to per-profile one.

This is for fully supporting multi-profile in Files.app.
Drive folder from different profiles need to have different paths.

The change includes:
- Change mount path from /special/drive to /special/drive-<user-id-hash>.
 Stored settings (like Download path) are migrated by MigratePathFromOldFormat.

- Profile extraction function from the above path format that
 will be used in next patch. (I included the function in the current
 patch to emphasize that the new path format is intended for
 this type of use.)

- Change mount point name from drive to drive-<user-id-hash>

- Mount point is added to the system global instance, not
the per-profile ones (otherwise cross-profile copy is impossible.)

BUG=336123
R=hashimoto@chromium.org, hirono@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248848

[kinaba, 2014-02-04 00:13:13]

+hirono, +hashimoto: ptal.
CC +yoshiki: I hope we are ready for the URL/path change in Files.app side, but
let me know if you have any concern still.

[hashimoto, 2014-02-04 02:32:04]

lgtm

[hirono, 2014-02-04 03:19:24]

lgtm with nit!

https://codereview.chromium.org/150163003/diff/110015/chrome/browser/chromeos...
File chrome/browser/chromeos/drive/file_system_util.cc (right):

https://codereview.chromium.org/150163003/diff/110015/chrome/browser/chromeos...
chrome/browser/chromeos/drive/file_system_util.cc:232: base::FilePath base =
GetDriveMountPointPath(original_profile);
nit: const

[kinaba, 2014-02-04 04:18:44]

https://codereview.chromium.org/150163003/diff/110015/chrome/browser/chromeos...
File chrome/browser/chromeos/drive/file_system_util.cc (right):

https://codereview.chromium.org/150163003/diff/110015/chrome/browser/chromeos...
chrome/browser/chromeos/drive/file_system_util.cc:232: base::FilePath base =
GetDriveMountPointPath(original_profile);
On 2014/02/04 03:19:24, hirono wrote:
> nit: const

Done.

[kinaba, 2014-02-04 04:18:56]

The CQ bit was checked by kinaba@chromium.org

[commit-bot: I haz the power, 2014-02-04 04:19:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinaba@chromium.org/150163003/390001

[commit-bot: I haz the power, 2014-02-04 05:35:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-04 05:35:24]

Retried try job too often on linux_rel for step(s) base_unittests,
browser_tests, interactive_ui_tests, net_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[kinaba, 2014-02-04 06:44:23]

The CQ bit was checked by kinaba@chromium.org

[commit-bot: I haz the power, 2014-02-04 06:44:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinaba@chromium.org/150163003/390001

[commit-bot: I haz the power, 2014-02-04 07:50:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-04 07:50:15]

Retried try job too often on ios_dbg_simulator for step(s) url_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[kinaba, 2014-02-04 09:01:25]

The CQ bit was checked by kinaba@chromium.org

[commit-bot: I haz the power, 2014-02-04 09:02:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinaba@chromium.org/150163003/390001

[commit-bot: I haz the power, 2014-02-04 10:05:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-04 10:05:17]

Retried try job too often on ios_dbg_simulator for step(s) url_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[kinaba, 2014-02-05 00:04:11]

The CQ bit was checked by kinaba@chromium.org

[commit-bot: I haz the power, 2014-02-05 00:10:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinaba@chromium.org/150163003/390001

[commit-bot: I haz the power, 2014-02-05 01:43:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinaba@chromium.org/150163003/390001

[kinaba, 2014-02-05 04:27:44]

Committed patchset #2 manually as r248848 (presubmit successful).