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

Side by Side Diff: chrome/browser/chrome_content_browser_client_unittest.cc

Issue 1020683003: Make content::PermissionType an enum class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix android compile Created 5 years, 9 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/chrome_content_browser_client.h" 5 #include "chrome/browser/chrome_content_browser_client.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/metrics/field_trial.h" 8 #include "base/metrics/field_trial.h"
9 #include "chrome/browser/search_engines/template_url_service_factory.h" 9 #include "chrome/browser/search_engines/template_url_service_factory.h"
10 #include "chrome/browser/ui/browser.h" 10 #include "chrome/browser/ui/browser.h"
11 #include "chrome/browser/ui/tabs/tab_strip_model.h" 11 #include "chrome/browser/ui/tabs/tab_strip_model.h"
12 #include "chrome/test/base/browser_with_test_window_test.h" 12 #include "chrome/test/base/browser_with_test_window_test.h"
13 #include "chrome/test/base/ui_test_utils.h" 13 #include "chrome/test/base/ui_test_utils.h"
14 #include "components/content_settings/core/browser/host_content_settings_map.h" 14 #include "components/content_settings/core/browser/host_content_settings_map.h"
15 #include "components/search_engines/template_url_service.h" 15 #include "components/search_engines/template_url_service.h"
16 #include "components/variations/entropy_provider.h" 16 #include "components/variations/entropy_provider.h"
17 #include "content/public/browser/navigation_controller.h" 17 #include "content/public/browser/navigation_controller.h"
18 #include "content/public/browser/navigation_entry.h" 18 #include "content/public/browser/navigation_entry.h"
19 #include "content/public/browser/permission_type.h"
19 #include "content/public/browser/web_contents.h" 20 #include "content/public/browser/web_contents.h"
20 #include "content/public/common/content_switches.h" 21 #include "content/public/common/content_switches.h"
21 #include "testing/gtest/include/gtest/gtest.h" 22 #include "testing/gtest/include/gtest/gtest.h"
22 #include "url/gurl.h" 23 #include "url/gurl.h"
23 24
25 using content::PermissionType;
26
24 namespace chrome { 27 namespace chrome {
25 28
26 using ChromeContentBrowserClientTest = testing::Test; 29 using ChromeContentBrowserClientTest = testing::Test;
27 30
28 TEST_F(ChromeContentBrowserClientTest, ShouldAssignSiteForURL) { 31 TEST_F(ChromeContentBrowserClientTest, ShouldAssignSiteForURL) {
29 ChromeContentBrowserClient client; 32 ChromeContentBrowserClient client;
30 EXPECT_FALSE(client.ShouldAssignSiteForURL(GURL("chrome-native://test"))); 33 EXPECT_FALSE(client.ShouldAssignSiteForURL(GURL("chrome-native://test")));
31 EXPECT_TRUE(client.ShouldAssignSiteForURL(GURL("http://www.google.com"))); 34 EXPECT_TRUE(client.ShouldAssignSiteForURL(GURL("http://www.google.com")));
32 EXPECT_TRUE(client.ShouldAssignSiteForURL(GURL("https://www.google.com"))); 35 EXPECT_TRUE(client.ShouldAssignSiteForURL(GURL("https://www.google.com")));
33 } 36 }
(...skipping 160 matching lines...) Expand 10 before | Expand all | Expand 10 after
194 } // namespace content 197 } // namespace content
195 #endif // !defined(OS_IOS) && !defined(OS_ANDROID) 198 #endif // !defined(OS_IOS) && !defined(OS_ANDROID)
196 199
197 namespace chrome { 200 namespace chrome {
198 201
199 // For testing permissions related functionality. 202 // For testing permissions related functionality.
200 class PermissionBrowserClientTest : public testing::Test { 203 class PermissionBrowserClientTest : public testing::Test {
201 public: 204 public:
202 PermissionBrowserClientTest() : url_("https://www.google.com") {} 205 PermissionBrowserClientTest() : url_("https://www.google.com") {}
203 206
204 void CheckPermissionStatus(content::PermissionType type, 207 void CheckPermissionStatus(PermissionType type,
205 content::PermissionStatus expected) { 208 content::PermissionStatus expected) {
206 EXPECT_EQ(expected, client_.GetPermissionStatus(type, &profile_, 209 EXPECT_EQ(expected, client_.GetPermissionStatus(type, &profile_,
207 url_.GetOrigin(), 210 url_.GetOrigin(),
208 url_.GetOrigin())); 211 url_.GetOrigin()));
209 } 212 }
210 213
211 void SetPermission(ContentSettingsType type, ContentSetting value) { 214 void SetPermission(ContentSettingsType type, ContentSetting value) {
212 profile_.GetHostContentSettingsMap()->SetContentSetting( 215 profile_.GetHostContentSettingsMap()->SetContentSetting(
213 ContentSettingsPattern::FromURLNoWildcard(url_), 216 ContentSettingsPattern::FromURLNoWildcard(url_),
214 ContentSettingsPattern::FromURLNoWildcard(url_), 217 ContentSettingsPattern::FromURLNoWildcard(url_),
215 type, std::string(), value); 218 type, std::string(), value);
216 } 219 }
217 220
218 private: 221 private:
219 content::TestBrowserThreadBundle thread_bundle_; 222 content::TestBrowserThreadBundle thread_bundle_;
220 ChromeContentBrowserClient client_; 223 ChromeContentBrowserClient client_;
221 TestingProfile profile_; 224 TestingProfile profile_;
222 GURL url_; 225 GURL url_;
223 }; 226 };
224 227
225 TEST_F(PermissionBrowserClientTest, GetPermissionStatusDefault) { 228 TEST_F(PermissionBrowserClientTest, GetPermissionStatusDefault) {
226 using namespace content; 229 using namespace content;
227 CheckPermissionStatus(PERMISSION_MIDI_SYSEX, PERMISSION_STATUS_ASK); 230 CheckPermissionStatus(PermissionType::MIDI_SYSEX, PERMISSION_STATUS_ASK);
228 CheckPermissionStatus(PERMISSION_PUSH_MESSAGING, PERMISSION_STATUS_ASK); 231 CheckPermissionStatus(PermissionType::PUSH_MESSAGING, PERMISSION_STATUS_ASK);
229 CheckPermissionStatus(PERMISSION_NOTIFICATIONS, PERMISSION_STATUS_ASK); 232 CheckPermissionStatus(PermissionType::NOTIFICATIONS, PERMISSION_STATUS_ASK);
230 CheckPermissionStatus(PERMISSION_GEOLOCATION, PERMISSION_STATUS_ASK); 233 CheckPermissionStatus(PermissionType::GEOLOCATION, PERMISSION_STATUS_ASK);
231 #if defined(OS_ANDROID) 234 #if defined(OS_ANDROID)
232 CheckPermissionStatus(PERMISSION_PROTECTED_MEDIA_IDENTIFIER, 235 CheckPermissionStatus(PermissionType::PROTECTED_MEDIA_IDENTIFIER,
233 PERMISSION_STATUS_ASK); 236 PERMISSION_STATUS_ASK);
234 #endif 237 #endif
235 } 238 }
236 239
237 TEST_F(PermissionBrowserClientTest, GetPermissionStatusAfterSet) { 240 TEST_F(PermissionBrowserClientTest, GetPermissionStatusAfterSet) {
238 using namespace content; 241 using namespace content;
239 SetPermission(CONTENT_SETTINGS_TYPE_GEOLOCATION, CONTENT_SETTING_ALLOW); 242 SetPermission(CONTENT_SETTINGS_TYPE_GEOLOCATION, CONTENT_SETTING_ALLOW);
240 CheckPermissionStatus(PERMISSION_GEOLOCATION, PERMISSION_STATUS_GRANTED); 243 CheckPermissionStatus(PermissionType::GEOLOCATION, PERMISSION_STATUS_GRANTED);
241 244
242 SetPermission(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, CONTENT_SETTING_ALLOW); 245 SetPermission(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, CONTENT_SETTING_ALLOW);
243 CheckPermissionStatus(PERMISSION_NOTIFICATIONS, PERMISSION_STATUS_GRANTED); 246 CheckPermissionStatus(PermissionType::NOTIFICATIONS,
247 PERMISSION_STATUS_GRANTED);
244 248
245 SetPermission(CONTENT_SETTINGS_TYPE_MIDI_SYSEX, CONTENT_SETTING_ALLOW); 249 SetPermission(CONTENT_SETTINGS_TYPE_MIDI_SYSEX, CONTENT_SETTING_ALLOW);
246 CheckPermissionStatus(PERMISSION_MIDI_SYSEX, PERMISSION_STATUS_GRANTED); 250 CheckPermissionStatus(PermissionType::MIDI_SYSEX, PERMISSION_STATUS_GRANTED);
247 251
248 SetPermission(CONTENT_SETTINGS_TYPE_PUSH_MESSAGING, CONTENT_SETTING_ALLOW); 252 SetPermission(CONTENT_SETTINGS_TYPE_PUSH_MESSAGING, CONTENT_SETTING_ALLOW);
249 CheckPermissionStatus(PERMISSION_PUSH_MESSAGING, PERMISSION_STATUS_GRANTED); 253 CheckPermissionStatus(PermissionType::PUSH_MESSAGING,
254 PERMISSION_STATUS_GRANTED);
250 255
251 #if defined(OS_ANDROID) 256 #if defined(OS_ANDROID)
252 SetPermission(CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER, 257 SetPermission(CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER,
253 CONTENT_SETTING_ALLOW); 258 CONTENT_SETTING_ALLOW);
254 CheckPermissionStatus(PERMISSION_PROTECTED_MEDIA_IDENTIFIER, 259 CheckPermissionStatus(PermissionType::PROTECTED_MEDIA_IDENTIFIER,
255 PERMISSION_STATUS_GRANTED); 260 PERMISSION_STATUS_GRANTED);
256 #endif 261 #endif
257 } 262 }
258 263
259 } // namespace chrome 264 } // namespace chrome
OLDNEW
« no previous file with comments | « chrome/browser/chrome_content_browser_client.cc ('k') | chrome/browser/content_settings/permission_context_uma_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698