Change chrome.filesystem to handle the Chrome OS Drive file entries directly.

Change chrome.filesystem to handle the Chrome OS Drive file entries directly.

This CL shows the Drive folder in the open file dialog,
and allows the app to select the real entry of the remote file,
rather than its local cache.

BUG=156277
TEST=Run the sample text editor chrome app, and open a Drive file via
the open-file dialog on Chrome OS. Should see "/special/drive/..." path.

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

[kinaba, 2013-03-22 08:32:41]

Please take a look, thanks.

This should be landed on top of https://codereview.chromium.org/12717014/.
I'll send out a following patch to enable file saving soon,
and it'll complete the Drive support.
(Only thing that has to be changed for that is that CheckWritableFile is
writing to a local path.)

[benwells, 2013-03-24 05:16:54]

lgtm

Can you add a TODO about write support if you don't get to it in this patch?

[benwells, 2013-03-24 05:17:23]

On 2013/03/24 05:16:54, benwells wrote:
> lgtm
> 
> Can you add a TODO about write support if you don't get to it in this patch?

Oh, scratch that, just saw the next patch in my queue...

[commit-bot: I haz the power, 2013-03-24 22:39:18]

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

[kinaba, 2013-03-25 00:14:49]

On 2013/03/24 22:39:18, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
> https://chromium-status.appspot.com/cq/kinaba%40chromium.org/13023002/1

Some relevant tests look really broken. Taking a look...

[kinaba, 2013-03-25 04:19:43]

Fixed the tests.
Ben, would you mind taking another look at Patch Set 3?
I'm sorry that I didn't checked well beforehands.

[benwells, 2013-03-25 10:08:13]

lgtm with comment added.

https://codereview.chromium.org/13023002/diff/23001/chrome/browser/extensions...
File chrome/browser/extensions/api/file_system/file_system_api.h (right):

https://codereview.chromium.org/13023002/diff/23001/chrome/browser/extensions...
chrome/browser/extensions/api/file_system/file_system_api.h:76: static void
RegisterTempExternalFileSystemForTest(const std::string& name,
Could you add a comment here explaining when this needs to be done?

[kinaba, 2013-03-25 12:07:51]

https://codereview.chromium.org/13023002/diff/23001/chrome/browser/extensions...
File chrome/browser/extensions/api/file_system/file_system_api.h (right):

https://codereview.chromium.org/13023002/diff/23001/chrome/browser/extensions...
chrome/browser/extensions/api/file_system/file_system_api.h:76: static void
RegisterTempExternalFileSystemForTest(const std::string& name,
On 2013/03/25 10:08:13, benwells wrote:
> Could you add a comment here explaining when this needs to be done?

Done.

[commit-bot: I haz the power, 2013-03-25 12:08:37]

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

[commit-bot: I haz the power, 2013-03-25 14:15:55]

Change committed as 190394