diff options
author | bbudge@chromium.org <bbudge@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-20 01:08:54 +0000 |
---|---|---|
committer | bbudge@chromium.org <bbudge@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-20 01:08:54 +0000 |
commit | 7dea88fd2340d36442020166c1aec0be431607e1 (patch) | |
tree | d483bd89251095ed5504f544ca4700ab9e55f4b4 /webkit/plugins/ppapi/url_request_info_unittest.cc | |
parent | a57c07c99ed88fd98ba37cec080f246975c05574 (diff) | |
download | chromium_src-7dea88fd2340d36442020166c1aec0be431607e1.zip chromium_src-7dea88fd2340d36442020166c1aec0be431607e1.tar.gz chromium_src-7dea88fd2340d36442020166c1aec0be431607e1.tar.bz2 |
Move check for Javascript URL to PPB_URLRequestInfo_Impl::RequiresUniversalAccess to fix issue http://code.google.com/p/chromium/issues/detail?id=89738 and http://code.google.com/p/chromium-os/issues/detail?id=17401. This is a redo of this CL http://codereview.chromium.org/7006004/.
Review URL: http://codereview.chromium.org/7448003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93142 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/plugins/ppapi/url_request_info_unittest.cc')
-rw-r--r-- | webkit/plugins/ppapi/url_request_info_unittest.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/webkit/plugins/ppapi/url_request_info_unittest.cc b/webkit/plugins/ppapi/url_request_info_unittest.cc index 5234ff2..e4f04a9 100644 --- a/webkit/plugins/ppapi/url_request_info_unittest.cc +++ b/webkit/plugins/ppapi/url_request_info_unittest.cc @@ -195,10 +195,11 @@ TEST_F(URLRequestInfoTest, SetURL) { ASSERT_TRUE(IsExpected(GetURL(), url)); } -TEST_F(URLRequestInfoTest, SetInvalidURL) { +TEST_F(URLRequestInfoTest, JavascriptURL) { const char* url = "javascript:foo = bar"; - ASSERT_FALSE(info_->SetStringProperty( - PP_URLREQUESTPROPERTY_URL, url)); + ASSERT_FALSE(info_->RequiresUniversalAccess()); + info_->SetStringProperty(PP_URLREQUESTPROPERTY_URL, url); + ASSERT_TRUE(info_->RequiresUniversalAccess()); } TEST_F(URLRequestInfoTest, SetMethod) { |