Files.app: Let the file browser private filesystem APIs use the auto-generated helper classes.

Files.app: Let the file browser private filesystem APIs use the auto-generated helper classes.

Originally, the file browser private filesystem APIs parse the arguments and generate the results manually. This CL lets the APIs use the auto generated helper
classes to parse the arguments.

BUG=241693
TEST=file_manager_browsertests

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

[hirono, 2013-09-12 07:04:12]

Could you take a look the CL?
Thank you very much!

[kinaba, 2013-09-12 07:34:18]

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

https://codereview.chromium.org/23766029/diff/1/chrome/browser/chromeos/exten...
chrome/browser/chromeos/extensions/file_manager/private_api_file_system.cc:464:
render_view_host(), profile(), GURL(params->mount_path));
This parameter should be URL.
Could you fix the .json file side?

https://codereview.chromium.org/23766029/diff/1/chrome/browser/chromeos/exten...
chrome/browser/chromeos/extensions/file_manager/private_api_file_system.cc:603:
render_view_host(), profile(), GURL(params->mount_path));
ditto.

[hirono, 2013-09-17 01:21:19]

Thanks!

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

https://codereview.chromium.org/23766029/diff/1/chrome/browser/chromeos/exten...
chrome/browser/chromeos/extensions/file_manager/private_api_file_system.cc:464:
render_view_host(), profile(), GURL(params->mount_path));
On 2013/09/12 07:34:19, kinaba wrote:
> This parameter should be URL.
> Could you fix the .json file side?

It seems that the name of mountPath is used 6 times in the .json file.
I'd like to replace them in another CL at all once.

[kinaba, 2013-09-17 01:44:56]

On 2013/09/17 01:21:19, hirono wrote:
> Thanks!
> 
>
https://codereview.chromium.org/23766029/diff/1/chrome/browser/chromeos/exten...
> File
chrome/browser/chromeos/extensions/file_manager/private_api_file_system.cc
> (right):
> 
>
https://codereview.chromium.org/23766029/diff/1/chrome/browser/chromeos/exten...
>
chrome/browser/chromeos/extensions/file_manager/private_api_file_system.cc:464:
> render_view_host(), profile(), GURL(params->mount_path));
> On 2013/09/12 07:34:19, kinaba wrote:
> > This parameter should be URL.
> > Could you fix the .json file side?
> 
> It seems that the name of mountPath is used 6 times in the .json file.
> I'd like to replace them in another CL at all once.
Makes sense. lgtm

[commit-bot: I haz the power, 2013-09-17 01:49:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hirono@chromium.org/23766029/10001

[commit-bot: I haz the power, 2013-09-17 01:56:38]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[commit-bot: I haz the power, 2013-09-17 02:01:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hirono@chromium.org/23766029/10001

[commit-bot: I haz the power, 2013-09-17 04:13:00]

Change committed as 223534