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

Unified Diff: ppapi/proxy/ppb_url_loader_proxy.h

Issue 6334016: Refactor PPAPI proxy resource handling to maintain which host they came from,... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 11 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
« no previous file with comments | « ppapi/proxy/ppb_testing_proxy.cc ('k') | ppapi/proxy/ppb_url_loader_proxy.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/ppb_url_loader_proxy.h
===================================================================
--- ppapi/proxy/ppb_url_loader_proxy.h (revision 72840)
+++ ppapi/proxy/ppb_url_loader_proxy.h (working copy)
@@ -12,6 +12,7 @@
#include "ppapi/c/pp_size.h"
#include "ppapi/c/pp_var.h"
#include "ppapi/cpp/completion_callback.h"
+#include "ppapi/proxy/host_resource.h"
#include "ppapi/proxy/interface_proxy.h"
#include "ppapi/proxy/proxy_non_thread_safe_ref_count.h"
@@ -21,6 +22,8 @@
namespace pp {
namespace proxy {
+struct PPBURLLoader_UpdateProgress_Params;
+
class PPB_URLLoader_Proxy : public InterfaceProxy {
public:
PPB_URLLoader_Proxy(Dispatcher* dispatcher, const void* target_interface);
@@ -30,8 +33,8 @@
// they are also provided to PPP_Instance.OnMsgHandleDocumentLoad. This
// function allows the proxy for DocumentLoad to create the correct plugin
// proxied info for the given browser-supplied URLLoader resource ID.
- static void TrackPluginResource(PP_Instance instance,
- PP_Resource url_loader_resource);
+ static PP_Resource TrackPluginResource(
+ const HostResource& url_loader_resource);
const PPB_URLLoader* ppb_url_loader_target() const {
return reinterpret_cast<const PPB_URLLoader*>(target_interface());
@@ -48,27 +51,24 @@
// Plugin->renderer message handlers.
void OnMsgCreate(PP_Instance instance,
- PP_Resource* result);
- void OnMsgOpen(PP_Resource loader,
- PP_Resource request_info,
+ HostResource* result);
+ void OnMsgOpen(const HostResource& loader,
+ const HostResource& request_info,
uint32_t serialized_callback);
- void OnMsgFollowRedirect(PP_Resource loader,
+ void OnMsgFollowRedirect(const HostResource& loader,
uint32_t serialized_callback);
- void OnMsgGetResponseInfo(PP_Resource loader,
- PP_Resource* result);
- void OnMsgReadResponseBody(PP_Resource loader,
+ void OnMsgGetResponseInfo(const HostResource& loader,
+ HostResource* result);
+ void OnMsgReadResponseBody(const HostResource& loader,
int32_t bytes_to_read);
- void OnMsgFinishStreamingToFile(PP_Resource loader,
+ void OnMsgFinishStreamingToFile(const HostResource& loader,
uint32_t serialized_callback);
- void OnMsgClose(PP_Resource loader);
+ void OnMsgClose(const HostResource& loader);
// Renderer->plugin message handlers.
- void OnMsgUpdateProgress(PP_Resource resource,
- int64_t bytes_sent,
- int64_t total_bytes_to_be_sent,
- int64_t bytes_received,
- int64_t total_bytes_to_be_received);
- void OnMsgReadResponseBodyAck(PP_Resource pp_resource,
+ void OnMsgUpdateProgress(
+ const PPBURLLoader_UpdateProgress_Params& params);
+ void OnMsgReadResponseBodyAck(const HostResource& pp_resource,
int32_t result,
const std::string& data);
@@ -97,7 +97,7 @@
private:
// Plugin->renderer message handlers.
- void OnMsgGrantUniversalAccess(PP_Resource loader);
+ void OnMsgGrantUniversalAccess(const HostResource& loader);
};
} // namespace proxy
« no previous file with comments | « ppapi/proxy/ppb_testing_proxy.cc ('k') | ppapi/proxy/ppb_url_loader_proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698