Convert url request info to new proxy API.

webkit/plugins/ppapi/ppb_url_loader_impl.cc

Index: webkit/plugins/ppapi/ppb_url_loader_impl.cc
diff --git a/webkit/plugins/ppapi/ppb_url_loader_impl.cc b/webkit/plugins/ppapi/ppb_url_loader_impl.cc
index 625ce12bfde4a024563113ebf30381bbe5ed156c..4f0e5894fc0762b80296cbcbea26592b8f32d73a 100644
--- a/webkit/plugins/ppapi/ppb_url_loader_impl.cc
+++ b/webkit/plugins/ppapi/ppb_url_loader_impl.cc
@@ -11,6 +11,7 @@
 #include "ppapi/c/ppb_url_loader.h"
 #include "ppapi/c/trusted/ppb_url_loader_trusted.h"
 #include "ppapi/thunk/enter.h"
+#include "ppapi/thunk/ppb_url_request_info_api.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebDocument.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebElement.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
@@ -27,9 +28,9 @@
 #include "webkit/plugins/ppapi/common.h"
 #include "webkit/plugins/ppapi/plugin_module.h"
 #include "webkit/plugins/ppapi/ppapi_plugin_instance.h"
-#include "webkit/plugins/ppapi/ppb_url_request_info_impl.h"
-#include "webkit/plugins/ppapi/ppb_url_response_info_impl.h"
 #include "webkit/plugins/ppapi/resource_helper.h"
+#include "webkit/plugins/ppapi/url_request_info_util.h"
+#include "webkit/plugins/ppapi/ppb_url_response_info_impl.h"
 
 using appcache::WebApplicationCacheHostImpl;
 using ppapi::Resource;
@@ -97,11 +98,6 @@ void PPB_URLLoader_Impl::InstanceWasDeleted() {
 
 int32_t PPB_URLLoader_Impl::Open(PP_Resource request_id,
                                  scoped_refptr<TrackedCallback> callback) {
-  // Main document loads are already open, so don't allow people to open them
-  // again.
-  if (main_document_loader_)
-    return PP_ERROR_INPROGRESS;
-

[bbudge, 2012/09/14 17:48:52]

Why don't we need this anymore?

[brettw, 2012/09/14 22:50:47]

I moved it to the internal version below (so it only exists in one place). I
couldn't think if why it needs to be above or below the request ID validation.

   EnterResourceNoLock<PPB_URLRequestInfo_API> enter_request(request_id, true);
   if (enter_request.failed()) {
     Log(PP_LOGLEVEL_ERROR,
@@ -110,14 +106,26 @@ int32_t PPB_URLLoader_Impl::Open(PP_Resource request_id,
         " else the request will be null.)");
     return PP_ERROR_BADARGUMENT;
   }
-  PPB_URLRequestInfo_Impl* request = static_cast<PPB_URLRequestInfo_Impl*>(
-      enter_request.object());
+  return Open(enter_request.object()->GetData(), callback);
+}
+
+int32_t PPB_URLLoader_Impl::Open(const ::ppapi::URLRequestInfoData& in_data,

[bbudge, 2012/09/14 17:48:52]

It would be clearer to me if 'in_data' was called 'request_data' and you named
the copy below 'filled_in_request_data' or something equivalent.

+                                 scoped_refptr<TrackedCallback> callback) {
+  // Main document loads are already open, so don't allow people to open them
+  // again.
+  if (main_document_loader_)
+    return PP_ERROR_INPROGRESS;
 
   int32_t rv = ValidateCallback(callback);
   if (rv != PP_OK)
     return rv;
 
-  if (request->RequiresUniversalAccess() && !has_universal_access_) {
+  // Create a copy of the request data since CreateWebURLRequest will populate
+  // the file refs.
+  ::ppapi::URLRequestInfoData request_data = in_data;
+
+  if (URLRequestRequiresUniversalAccess(request_data) &&
+      !has_universal_access_) {
     Log(PP_LOGLEVEL_ERROR, "PPB_URLLoader.Open: The URL you're requesting is "
         " on a different security origin than your plugin. To request "
         " cross-origin resources, see "
@@ -132,13 +140,13 @@ int32_t PPB_URLLoader_Impl::Open(PP_Resource request_id,
   if (!frame)
     return PP_ERROR_FAILED;
   WebURLRequest web_request;
-  if (!request->ToWebURLRequest(frame, &web_request))
+  if (!CreateWebURLRequest(&request_data, frame, &web_request))
     return PP_ERROR_FAILED;
 
   // Save a copy of the request info so the plugin can continue to use and
   // change it while we're doing the request without affecting us. We must do
   // this after ToWebURLRequest since that fills out the file refs.

[bbudge, 2012/09/14 17:48:52]

s/ToWebURLRequest/CreateWebURLRequest

-  request_data_ = request->GetData();
+  request_data_ = request_data;
 
   WebURLLoaderOptions options;
   if (has_universal_access_) {
@@ -178,8 +186,6 @@ int32_t PPB_URLLoader_Impl::FollowRedirect(
   if (rv != PP_OK)
     return rv;
 
-  WebURL redirect_url = GURL(response_info_->redirect_url());
-
   SetDefersLoading(false);  // Allow the redirect to continue.
   RegisterCallback(callback);
   return PP_OK_COMPLETIONPENDING;