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

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: Rebase 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/scoped_test_public_session_login_state.h"
12 #include "content/public/browser/resource_request_info.h" 13 #include "content/public/browser/resource_request_info.h"
13 #include "content/public/common/previews_state.h" 14 #include "content/public/common/previews_state.h"
14 #include "content/public/test/test_browser_thread_bundle.h" 15 #include "content/public/test/test_browser_thread_bundle.h"
15 #include "extensions/browser/api/web_request/web_request_permissions.h" 16 #include "extensions/browser/api/web_request/web_request_permissions.h"
16 #include "extensions/browser/info_map.h" 17 #include "extensions/browser/info_map.h"
17 #include "extensions/common/constants.h" 18 #include "extensions/common/constants.h"
18 #include "extensions/common/permissions/permissions_data.h" 19 #include "extensions/common/permissions/permissions_data.h"
19 #include "ipc/ipc_message.h" 20 #include "ipc/ipc_message.h"
20 #include "net/base/request_priority.h" 21 #include "net/base/request_priority.h"
21 #include "net/traffic_annotation/network_traffic_annotation_test_helper.h" 22 #include "net/traffic_annotation/network_traffic_annotation_test_helper.h"
(...skipping 192 matching lines...) Expand 10 before | Expand all | Expand 10 after
214 215
215 // com_extension_ doesn't have host permission for .org URLs. 216 // com_extension_ doesn't have host permission for .org URLs.
216 EXPECT_EQ(PermissionsData::ACCESS_DENIED, 217 EXPECT_EQ(PermissionsData::ACCESS_DENIED,
217 WebRequestPermissions::CanExtensionAccessURL( 218 WebRequestPermissions::CanExtensionAccessURL(
218 extension_info_map_.get(), com_policy_extension_->id(), 219 extension_info_map_.get(), com_policy_extension_->id(),
219 org_request->url(), 220 org_request->url(),
220 -1, // No tab id. 221 -1, // No tab id.
221 false, // crosses_incognito 222 false, // crosses_incognito
222 WebRequestPermissions::REQUIRE_HOST_PERMISSION)); 223 WebRequestPermissions::REQUIRE_HOST_PERMISSION));
223 224
224 // Set Public Session state. 225 chromeos::ScopedTestPublicSessionLoginState login_state;
225 chromeos::LoginState::Initialize();
226 chromeos::LoginState::Get()->SetLoggedInState(
227 chromeos::LoginState::LOGGED_IN_ACTIVE,
228 chromeos::LoginState::LOGGED_IN_USER_PUBLIC_ACCOUNT);
229 226
230 // Host permission checks are disabled in Public Sessions, instead all URLs 227 // Host permission checks are disabled in Public Sessions, instead all URLs
231 // are whitelisted. 228 // are whitelisted.
232 EXPECT_EQ(PermissionsData::ACCESS_ALLOWED, 229 EXPECT_EQ(PermissionsData::ACCESS_ALLOWED,
233 WebRequestPermissions::CanExtensionAccessURL( 230 WebRequestPermissions::CanExtensionAccessURL(
234 extension_info_map_.get(), com_policy_extension_->id(), 231 extension_info_map_.get(), com_policy_extension_->id(),
235 org_request->url(), 232 org_request->url(),
236 -1, // No tab id. 233 -1, // No tab id.
237 false, // crosses_incognito 234 false, // crosses_incognito
238 WebRequestPermissions::REQUIRE_HOST_PERMISSION)); 235 WebRequestPermissions::REQUIRE_HOST_PERMISSION));
(...skipping 13 matching lines...) Expand all
252 249
253 EXPECT_EQ(PermissionsData::ACCESS_DENIED, 250 EXPECT_EQ(PermissionsData::ACCESS_DENIED,
254 WebRequestPermissions::CanExtensionAccessURL( 251 WebRequestPermissions::CanExtensionAccessURL(
255 extension_info_map_.get(), com_policy_extension_->id(), 252 extension_info_map_.get(), com_policy_extension_->id(),
256 chrome_request->url(), 253 chrome_request->url(),
257 -1, // No tab id. 254 -1, // No tab id.
258 false, // crosses_incognito 255 false, // crosses_incognito
259 WebRequestPermissions::REQUIRE_HOST_PERMISSION)); 256 WebRequestPermissions::REQUIRE_HOST_PERMISSION));
260 #endif 257 #endif
261 } 258 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698