Add CreateFileWriter method to MountPointProvider.

Add CreateFileWriter method to MountPointProvider.

In this patch, just adding an interface. Concrete implementations that return
appropriate FileWriter subclass instances are on the way.

BUG=123993
TEST=check it compiles.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137363

[kinaba, 2012-05-15 05:00:16]

Kinuko-san, could you take a look?

[kinuko, 2012-05-15 06:51:21]

lgtm

http://codereview.chromium.org/10332164/diff/1/webkit/fileapi/isolated_mount_...
File webkit/fileapi/isolated_mount_point_provider.cc (right):

http://codereview.chromium.org/10332164/diff/1/webkit/fileapi/isolated_mount_...
webkit/fileapi/isolated_mount_point_provider.cc:127: // TODO(kinaba,kinuko):
return SandboxFileWriter when it is implemented.
This one could simply return LocalFileWriter in the same way as CreateFileReader
does.  (It's ok if you want to implement it in a separate CL though)

[kinaba, 2012-05-15 07:14:56]

Thanks, Implemented for IsolatedFS in the current CL.

http://codereview.chromium.org/10332164/diff/1/webkit/fileapi/isolated_mount_...
File webkit/fileapi/isolated_mount_point_provider.cc (right):

http://codereview.chromium.org/10332164/diff/1/webkit/fileapi/isolated_mount_...
webkit/fileapi/isolated_mount_point_provider.cc:127: // TODO(kinaba,kinuko):
return SandboxFileWriter when it is implemented.
On 2012/05/15 06:51:21, kinuko wrote:
> This one could simply return LocalFileWriter in the same way as
CreateFileReader
> does.  (It's ok if you want to implement it in a separate CL though)

Done.

[commit-bot: I haz the power, 2012-05-16 03:21:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/10332164/6001

[commit-bot: I haz the power, 2012-05-16 04:42:04]

Change committed as 137363