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

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

Issue 2856053004: [Refactor] Use ScopedTestPublicSessionLoginState in code (Closed)
Patch Set: Created 3 years, 7 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
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 <stddef.h> 5 #include <stddef.h>
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/message_loop/message_loop.h" 10 #include "base/message_loop/message_loop.h"
11 #include "chrome/common/extensions/extension_test_util.h" 11 #include "chrome/common/extensions/extension_test_util.h"
12 #include "chromeos/login/login_state.h" 12 #include "chromeos/login/scoped_test_public_session_login_state.h"
13 #include "content/public/browser/resource_request_info.h" 13 #include "content/public/browser/resource_request_info.h"
14 #include "content/public/common/previews_state.h" 14 #include "content/public/common/previews_state.h"
15 #include "content/public/test/test_browser_thread_bundle.h" 15 #include "content/public/test/test_browser_thread_bundle.h"
16 #include "extensions/browser/api/web_request/web_request_permissions.h" 16 #include "extensions/browser/api/web_request/web_request_permissions.h"
17 #include "extensions/browser/info_map.h" 17 #include "extensions/browser/info_map.h"
18 #include "extensions/common/constants.h" 18 #include "extensions/common/constants.h"
19 #include "extensions/common/permissions/permissions_data.h" 19 #include "extensions/common/permissions/permissions_data.h"
20 #include "ipc/ipc_message.h" 20 #include "ipc/ipc_message.h"
21 #include "net/base/request_priority.h" 21 #include "net/base/request_priority.h"
22 #include "net/traffic_annotation/network_traffic_annotation_test_helper.h" 22 #include "net/traffic_annotation/network_traffic_annotation_test_helper.h"
(...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after
211 211
212 // com_extension_ doesn't have host permission for .org URLs. 212 // com_extension_ doesn't have host permission for .org URLs.
213 EXPECT_EQ(PermissionsData::ACCESS_DENIED, 213 EXPECT_EQ(PermissionsData::ACCESS_DENIED,
214 WebRequestPermissions::CanExtensionAccessURL( 214 WebRequestPermissions::CanExtensionAccessURL(
215 extension_info_map_.get(), com_policy_extension_->id(), 215 extension_info_map_.get(), com_policy_extension_->id(),
216 org_request->url(), 216 org_request->url(),
217 -1, // No tab id. 217 -1, // No tab id.
218 false, // crosses_incognito 218 false, // crosses_incognito
219 WebRequestPermissions::REQUIRE_HOST_PERMISSION)); 219 WebRequestPermissions::REQUIRE_HOST_PERMISSION));
220 220
221 // Set Public Session state. 221 chromeos::ScopedTestPublicSessionLoginState login_state;
222 chromeos::LoginState::Initialize();
223 chromeos::LoginState::Get()->SetLoggedInState(
224 chromeos::LoginState::LOGGED_IN_ACTIVE,
225 chromeos::LoginState::LOGGED_IN_USER_PUBLIC_ACCOUNT);
226 222
227 // Host permission checks are disabled in Public Sessions, instead all URLs 223 // Host permission checks are disabled in Public Sessions, instead all URLs
228 // are whitelisted. 224 // are whitelisted.
229 EXPECT_EQ(PermissionsData::ACCESS_ALLOWED, 225 EXPECT_EQ(PermissionsData::ACCESS_ALLOWED,
230 WebRequestPermissions::CanExtensionAccessURL( 226 WebRequestPermissions::CanExtensionAccessURL(
231 extension_info_map_.get(), com_policy_extension_->id(), 227 extension_info_map_.get(), com_policy_extension_->id(),
232 org_request->url(), 228 org_request->url(),
233 -1, // No tab id. 229 -1, // No tab id.
234 false, // crosses_incognito 230 false, // crosses_incognito
235 WebRequestPermissions::REQUIRE_HOST_PERMISSION)); 231 WebRequestPermissions::REQUIRE_HOST_PERMISSION));
(...skipping 13 matching lines...) Expand all
249 245
250 EXPECT_EQ(PermissionsData::ACCESS_DENIED, 246 EXPECT_EQ(PermissionsData::ACCESS_DENIED,
251 WebRequestPermissions::CanExtensionAccessURL( 247 WebRequestPermissions::CanExtensionAccessURL(
252 extension_info_map_.get(), com_policy_extension_->id(), 248 extension_info_map_.get(), com_policy_extension_->id(),
253 chrome_request->url(), 249 chrome_request->url(),
254 -1, // No tab id. 250 -1, // No tab id.
255 false, // crosses_incognito 251 false, // crosses_incognito
256 WebRequestPermissions::REQUIRE_HOST_PERMISSION)); 252 WebRequestPermissions::REQUIRE_HOST_PERMISSION));
257 #endif 253 #endif
258 } 254 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698