Don't set the request target type to object when navigating from a Pepper plug-in.

content/renderer/pepper/pepper_url_request_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/compiler_specific.h"
 #include "content/public/test/render_view_test.h"
 #include "content/renderer/pepper/url_request_info_util.h"
 #include "ppapi/proxy/connection.h"
 #include "ppapi/proxy/url_request_info_resource.h"
 #include "ppapi/shared_impl/test_globals.h"
@@ skipping 58 matching lines @@
                                        URLRequestInfoData());
   }
 
   virtual void TearDown() OVERRIDE {
     test_globals_.GetResourceTracker()->DidDeleteInstance(pp_instance_);
     RenderViewTest::TearDown();
   }
 
   bool GetDownloadToFile() {
     WebURLRequest web_request;
-    URLRequestInfoData data = info_->GetData();
-    if (!CreateWebURLRequest(&data, GetMainFrame(), &web_request))
+    if (!CreateWebURLRequestInternal(&web_request))
       return false;
+
     return web_request.downloadToFile();
   }
 
   WebCString GetURL() {
     WebURLRequest web_request;
-    URLRequestInfoData data = info_->GetData();
-    if (!CreateWebURLRequest(&data, GetMainFrame(), &web_request))
+    if (!CreateWebURLRequestInternal(&web_request))
       return WebCString();
+
     return web_request.url().spec();
   }
 
   WebString GetMethod() {
     WebURLRequest web_request;
-    URLRequestInfoData data = info_->GetData();
-    if (!CreateWebURLRequest(&data, GetMainFrame(), &web_request))
+    if (!CreateWebURLRequestInternal(&web_request))
       return WebString();
+
     return web_request.httpMethod();
   }
 
   WebString GetHeaderValue(const char* field) {
     WebURLRequest web_request;
-    URLRequestInfoData data = info_->GetData();
-    if (!CreateWebURLRequest(&data, GetMainFrame(), &web_request))
+    if (!CreateWebURLRequestInternal(&web_request))
       return WebString();
+
     return web_request.httpHeaderField(WebString::fromUTF8(field));
   }
 
   bool SetBooleanProperty(PP_URLRequestProperty prop, bool b) {
     return info_->SetBooleanProperty(prop, b);
   }
+
   bool SetStringProperty(PP_URLRequestProperty prop, const std::string& s) {
     return info_->SetStringProperty(prop, s);
   }
 
+ protected:
+  bool CreateWebURLRequestInternal(WebURLRequest* web_request) {
+    URLRequestInfoData data = info_->GetData();
+    return CreateWebURLRequest(
+        &data, WebURLRequest::TargetIsUnspecified, GetMainFrame(), web_request);
+  }
+
   PP_Instance pp_instance_;
 
   // Needs to be alive for resource tracking to work.
   ppapi::TestGlobals test_globals_;
 
   scoped_refptr<URLRequestInfoResource> info_;
 };
 
 TEST_F(URLRequestInfoTest, GetInterface) {
   const PPB_URLRequestInfo* request_info =
@@ skipping 120 matching lines @@
       PP_URLREQUESTPROPERTY_HEADERS, "foo: bar\nbar: baz"));
   EXPECT_TRUE(IsExpected(
       GetHeaderValue("foo"), "bar"));
   EXPECT_TRUE(IsExpected(
       GetHeaderValue("bar"), "baz"));
 }
 
 // TODO(bbudge) Unit tests for AppendDataToBody, AppendFileToBody.
 
 }  // namespace content