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

Side by Side Diff: chrome/browser/extensions/api/web_request/web_request_permissions_unittest.cc

Issue 16295003: Update chrome/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/api/web_request/web_request_permissions.h" 5 #include "chrome/browser/extensions/api/web_request/web_request_permissions.h"
6 6
7 #include "base/message_loop.h" 7 #include "base/message_loop.h"
8 #include "chrome/browser/extensions/extension_info_map.h" 8 #include "chrome/browser/extensions/extension_info_map.h"
9 #include "chrome/common/extensions/extension_constants.h" 9 #include "chrome/common/extensions/extension_constants.h"
10 #include "chrome/common/extensions/extension_manifest_constants.h" 10 #include "chrome/common/extensions/extension_manifest_constants.h"
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
45 void ExtensionWebRequestHelpersTestWithThreadsTest::SetUp() { 45 void ExtensionWebRequestHelpersTestWithThreadsTest::SetUp() {
46 testing::Test::SetUp(); 46 testing::Test::SetUp();
47 47
48 std::string error; 48 std::string error;
49 permissionless_extension_ = LoadManifestUnchecked("permissions", 49 permissionless_extension_ = LoadManifestUnchecked("permissions",
50 "web_request_no_host.json", 50 "web_request_no_host.json",
51 Manifest::INVALID_LOCATION, 51 Manifest::INVALID_LOCATION,
52 Extension::NO_FLAGS, 52 Extension::NO_FLAGS,
53 "ext_id_1", 53 "ext_id_1",
54 &error); 54 &error);
55 ASSERT_TRUE(permissionless_extension_) << error; 55 ASSERT_TRUE(permissionless_extension_.get()) << error;
56 com_extension_ = 56 com_extension_ =
57 LoadManifestUnchecked("permissions", 57 LoadManifestUnchecked("permissions",
58 "web_request_com_host_permissions.json", 58 "web_request_com_host_permissions.json",
59 Manifest::INVALID_LOCATION, 59 Manifest::INVALID_LOCATION,
60 Extension::NO_FLAGS, 60 Extension::NO_FLAGS,
61 "ext_id_2", 61 "ext_id_2",
62 &error); 62 &error);
63 ASSERT_TRUE(com_extension_) << error; 63 ASSERT_TRUE(com_extension_.get()) << error;
64 extension_info_map_ = new ExtensionInfoMap; 64 extension_info_map_ = new ExtensionInfoMap;
65 extension_info_map_->AddExtension(permissionless_extension_.get(), 65 extension_info_map_->AddExtension(permissionless_extension_.get(),
66 base::Time::Now(), 66 base::Time::Now(),
67 false /*incognito_enabled*/); 67 false /*incognito_enabled*/);
68 extension_info_map_->AddExtension( 68 extension_info_map_->AddExtension(
69 com_extension_.get(), base::Time::Now(), false /*incognito_enabled*/); 69 com_extension_.get(), base::Time::Now(), false /*incognito_enabled*/);
70 } 70 }
71 71
72 TEST(ExtensionWebRequestHelpersTest, TestHideRequestForURL) { 72 TEST(ExtensionWebRequestHelpersTest, TestHideRequestForURL) {
73 base::MessageLoopForIO message_loop; 73 base::MessageLoopForIO message_loop;
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
142 &sensitive_request)); 142 &sensitive_request));
143 } 143 }
144 } 144 }
145 145
146 TEST_F(ExtensionWebRequestHelpersTestWithThreadsTest, 146 TEST_F(ExtensionWebRequestHelpersTestWithThreadsTest,
147 TestCanExtensionAccessURL_HostPermissions) { 147 TestCanExtensionAccessURL_HostPermissions) {
148 net::TestURLRequest request( 148 net::TestURLRequest request(
149 GURL("http://example.com"), NULL, &context, NULL); 149 GURL("http://example.com"), NULL, &context, NULL);
150 150
151 EXPECT_TRUE(WebRequestPermissions::CanExtensionAccessURL( 151 EXPECT_TRUE(WebRequestPermissions::CanExtensionAccessURL(
152 extension_info_map_, 152 extension_info_map_.get(),
153 permissionless_extension_->id(), 153 permissionless_extension_->id(),
154 request.url(), 154 request.url(),
155 false /*crosses_incognito*/, 155 false /*crosses_incognito*/,
156 WebRequestPermissions::DO_NOT_CHECK_HOST)); 156 WebRequestPermissions::DO_NOT_CHECK_HOST));
157 EXPECT_FALSE(WebRequestPermissions::CanExtensionAccessURL( 157 EXPECT_FALSE(WebRequestPermissions::CanExtensionAccessURL(
158 extension_info_map_, 158 extension_info_map_.get(),
159 permissionless_extension_->id(), 159 permissionless_extension_->id(),
160 request.url(), 160 request.url(),
161 false /*crosses_incognito*/, 161 false /*crosses_incognito*/,
162 WebRequestPermissions::REQUIRE_HOST_PERMISSION)); 162 WebRequestPermissions::REQUIRE_HOST_PERMISSION));
163 EXPECT_TRUE(WebRequestPermissions::CanExtensionAccessURL( 163 EXPECT_TRUE(WebRequestPermissions::CanExtensionAccessURL(
164 extension_info_map_, 164 extension_info_map_.get(),
165 com_extension_->id(), 165 com_extension_->id(),
166 request.url(), 166 request.url(),
167 false /*crosses_incognito*/, 167 false /*crosses_incognito*/,
168 WebRequestPermissions::REQUIRE_HOST_PERMISSION)); 168 WebRequestPermissions::REQUIRE_HOST_PERMISSION));
169 EXPECT_FALSE(WebRequestPermissions::CanExtensionAccessURL( 169 EXPECT_FALSE(WebRequestPermissions::CanExtensionAccessURL(
170 extension_info_map_, 170 extension_info_map_.get(),
171 com_extension_->id(), 171 com_extension_->id(),
172 request.url(), 172 request.url(),
173 false /*crosses_incognito*/, 173 false /*crosses_incognito*/,
174 WebRequestPermissions::REQUIRE_ALL_URLS)); 174 WebRequestPermissions::REQUIRE_ALL_URLS));
175 } 175 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698