Pepper/Flapper: Add an interface to do sync file ops on FileRefs.

Pepper/Flapper: Add an interface to do sync file ops on FileRefs.

Such FileRefs are typically obtained from the Pepper file chooser. The interface
corresponds exactly to the one for module-local files. (The implementation is
only enabled if Flapper hacks are enabled.)

BUG=none
TEST=Flapper file uploads work for me

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

[piman, 2011-03-01 21:54:29]

LGTM.
Don't forget the proxy.

http://codereview.chromium.org/6592071/diff/3001/webkit/plugins/ppapi/ppb_fla...
File webkit/plugins/ppapi/ppb_flash_file_impl.cc (right):

http://codereview.chromium.org/6592071/diff/3001/webkit/plugins/ppapi/ppb_fla...
webkit/plugins/ppapi/ppb_flash_file_impl.cc:255: PP_Resource to_file_ref_id) {
I'm not sure we actually need Rename/Delete/CreateDir or even GetDirContents.
It's probably better to just stub them out.

[viettrungluu, 2011-03-01 22:18:20]

Thanks. I'll do the proxy in a separate CL.

On 2011/03/01 21:54:29, piman wrote:
> LGTM.
> Don't forget the proxy.
> 
>
http://codereview.chromium.org/6592071/diff/3001/webkit/plugins/ppapi/ppb_fla...
> File webkit/plugins/ppapi/ppb_flash_file_impl.cc (right):
> 
>
http://codereview.chromium.org/6592071/diff/3001/webkit/plugins/ppapi/ppb_fla...
> webkit/plugins/ppapi/ppb_flash_file_impl.cc:255: PP_Resource to_file_ref_id) {
> I'm not sure we actually need Rename/Delete/CreateDir or even GetDirContents.
> It's probably better to just stub them out.

Done.

[piman, 2011-03-01 23:03:19]

LGTM