Enable Google Drive in all save-file dialogs of Chrome OS.

Enable Google Drive in all save-file dialogs of Chrome OS.

All set.
For save dialog called from Drive-unaware code, we provide
a local snapshot created by GetFileByPathForSaving so that
modification on that file will automatically sync to the
remote server.

BUG=140425
TEST=Try export from bookmark manager, pixlr.com editor,  certificate manager, or about:tracing.

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

[kinaba, 2013-08-08 07:00:32]

This requires https://codereview.chromium.org/22335004/ to land first.

+hashimoto: thanks for your review for this series of patches this week. Here's
the final piece.
+yoshiki: ptal as a JS-side owner.

[yoshiki, 2013-08-08 07:04:19]

Great patch(es)! LGTM!

[hashimoto, 2013-08-08 07:07:28]

lgtm, let's see if the 5-second delay trick would work...

https://codereview.chromium.org/22523003/diff/1/chrome/browser/chromeos/exten...
File chrome/browser/chromeos/extensions/file_manager/private_api_util.cc
(right):

https://codereview.chromium.org/22523003/diff/1/chrome/browser/chromeos/exten...
chrome/browser/chromeos/extensions/file_manager/private_api_util.cc:79: } else {
nit: Can't we have DCHECK_EQ(NEED_LOCAL_PATH_FOR_SAVING, ...), or make this
chain of 'if's to a 'switch'?

[kinaba, 2013-08-08 08:07:13]

https://codereview.chromium.org/22523003/diff/1/chrome/browser/chromeos/exten...
File chrome/browser/chromeos/extensions/file_manager/private_api_util.cc
(right):

https://codereview.chromium.org/22523003/diff/1/chrome/browser/chromeos/exten...
chrome/browser/chromeos/extensions/file_manager/private_api_util.cc:79: } else {
On 2013/08/08 07:07:28, hashimoto wrote:
> nit: Can't we have DCHECK_EQ(NEED_LOCAL_PATH_FOR_SAVING, ...), or make this
> chain of 'if's to a 'switch'?

Done.

[hashimoto, 2013-08-08 08:09:42]

lgtm

[commit-bot: I haz the power, 2013-08-08 12:06:40]

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

[commit-bot: I haz the power, 2013-08-09 01:09:20]

Change committed as 216524