Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2449)

Unified Diff: ppapi/proxy/ppb_file_chooser_proxy.h

Issue 9728001: Make the file chooser use PP_ArrayOutput (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ppapi/proxy/ppb_file_chooser_proxy.h
diff --git a/ppapi/proxy/ppb_file_chooser_proxy.h b/ppapi/proxy/ppb_file_chooser_proxy.h
index da45a2c57623831afe303b8153e737cc0e4e1b34..0d66d944fb377e2e755b0aefd252a372c44c46be 100644
--- a/ppapi/proxy/ppb_file_chooser_proxy.h
+++ b/ppapi/proxy/ppb_file_chooser_proxy.h
@@ -11,8 +11,11 @@
#include "base/basictypes.h"
#include "ppapi/c/pp_instance.h"
#include "ppapi/proxy/interface_proxy.h"
+#include "ppapi/proxy/proxy_array_output.h"
#include "ppapi/proxy/proxy_non_thread_safe_ref_count.h"
+#include "ppapi/proxy/serialized_var.h"
#include "ppapi/thunk/ppb_file_chooser_api.h"
+#include "ppapi/cpp/output_traits.h"
#include "ppapi/utility/completion_callback_factory.h"
namespace ppapi {
@@ -46,8 +49,8 @@ class PPB_FileChooser_Proxy : public InterfaceProxy {
std::string accept_mime_types,
ppapi::HostResource* result);
void OnMsgShow(const ppapi::HostResource& chooser,
- bool save_as,
- std::string suggested_file_name,
+ PP_Bool save_as,
+ SerializedVarReceiveInput suggested_file_name,
bool require_user_gesture);
// Host -> plugin message handlers.
@@ -58,7 +61,11 @@ class PPB_FileChooser_Proxy : public InterfaceProxy {
// Called when the show is complete in the host. This will notify the plugin
// via IPC and OnMsgChooseComplete will be called there.
- void OnShowCallback(int32_t result, const ppapi::HostResource& chooser);
+ void OnShowCallback(
+ int32_t result,
+ scoped_refptr<RefCountedArrayOutputAdapter<PP_Resource> >
viettrungluu 2012/03/26 16:55:19 no need to split this line
+ output,
+ HostResource chooser);
pp::CompletionCallbackFactory<PPB_FileChooser_Proxy,
ProxyNonThreadSafeRefCount> callback_factory_;

Powered by Google App Engine
This is Rietveld 408576698