Implement proxying for FileRef and FileChooser....

Implement proxying for FileRef and FileChooser.

This also changes the FileRef interface to remove the Query function, since
there is no close function, there's no way to cancel the current request, which
makes memory management of the info structure very difficult.


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

[piman, 2011-02-17 21:25:07]

LGTM with 2 nits

http://codereview.chromium.org/6519057/diff/1/ppapi/proxy/ppb_file_chooser_pr...
File ppapi/proxy/ppb_file_chooser_proxy.cc (right):

http://codereview.chromium.org/6519057/diff/1/ppapi/proxy/ppb_file_chooser_pr...
ppapi/proxy/ppb_file_chooser_proxy.cc:75: static_cast<int>(options->mode),
options->accept_mime_types, &result));
Is the static_cast necessary ? I'd assume the cast would be implicit.

http://codereview.chromium.org/6519057/diff/1/ppapi/proxy/serialized_var.cc
File ppapi/proxy/serialized_var.cc (right):

http://codereview.chromium.org/6519057/diff/1/ppapi/proxy/serialized_var.cc#n...
ppapi/proxy/serialized_var.cc:282: static_cast<SerializedVar&>(ret) =
serialized;
How about adding an explicit constructor for ReceiveSerializedVarReturnValue
that takes a const SerializedVar & and delegates to the SerializeVar copy
constructor ? That would be cleaner (no static cast required)