Implement Worker-MainThread bridge for FileWriter in Chrome

Implement Worker-MainThread bridge for FileWriter in Chrome

This basically adds the same worker/mainThread bridge functionality
for FileWriter in Chrome side.

BUG=257349
TEST=no behavioral changes yet
R=michaeln@chromium.org

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

[kinuko, 2013-07-29 13:07:47]

One more bridge change... can you take a look?  Thanks,

[michaeln, 2013-07-29 21:29:18]

lgtm

https://codereview.chromium.org/21041003/diff/13001/content/child/fileapi/web...
File content/child/fileapi/webfilesystem_impl.cc (right):

https://codereview.chromium.org/21041003/diff/13001/content/child/fileapi/web...
content/child/fileapi/webfilesystem_impl.cc:213: }
nit: the original conditional blocks were slightly more readable (maybe)

 102   if (file_info.is_directory || file_info.size < 0) {	
 103     callbacks->didFail(WebKit::WebFileErrorInvalidState);	
 104     return;	
 105   }	
 106   callbacks->didCreateFileWriter(new WebFileWriterImpl(path, client),	
 107                                  file_info.size);

[kinuko, 2013-07-30 08:31:58]

Thx!

https://codereview.chromium.org/21041003/diff/13001/content/child/fileapi/web...
File content/child/fileapi/webfilesystem_impl.cc (right):

https://codereview.chromium.org/21041003/diff/13001/content/child/fileapi/web...
content/child/fileapi/webfilesystem_impl.cc:213: }
On 2013/07/29 21:29:18, michaeln wrote:
> nit: the original conditional blocks were slightly more readable (maybe)
> 
>  102   if (file_info.is_directory || file_info.size < 0) {	
>  103     callbacks->didFail(WebKit::WebFileErrorInvalidState);	
>  104     return;	
>  105   }	
>  106   callbacks->didCreateFileWriter(new WebFileWriterImpl(path, client),	
>  107                                  file_info.size);

Done.

[commit-bot: I haz the power, 2013-07-30 18:11:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/21041003/33001

[commit-bot: I haz the power, 2013-07-30 21:12:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/21041003/33001

[kinuko, 2013-08-01 05:12:59]

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