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

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

Issue 8383004: Adding CookieSettings for storing cookie content settings. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixing the rebase. Created 9 years, 1 month 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/cookies_tree_model.h" 5 #include "chrome/browser/cookies_tree_model.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "chrome/browser/content_settings/cookie_settings.h"
9 #include "chrome/browser/content_settings/host_content_settings_map.h" 10 #include "chrome/browser/content_settings/host_content_settings_map.h"
10 #include "chrome/browser/content_settings/mock_settings_observer.h" 11 #include "chrome/browser/content_settings/mock_settings_observer.h"
11 #include "chrome/browser/mock_browsing_data_appcache_helper.h" 12 #include "chrome/browser/mock_browsing_data_appcache_helper.h"
12 #include "chrome/browser/mock_browsing_data_cookie_helper.h" 13 #include "chrome/browser/mock_browsing_data_cookie_helper.h"
13 #include "chrome/browser/mock_browsing_data_database_helper.h" 14 #include "chrome/browser/mock_browsing_data_database_helper.h"
14 #include "chrome/browser/mock_browsing_data_file_system_helper.h" 15 #include "chrome/browser/mock_browsing_data_file_system_helper.h"
15 #include "chrome/browser/mock_browsing_data_indexed_db_helper.h" 16 #include "chrome/browser/mock_browsing_data_indexed_db_helper.h"
16 #include "chrome/browser/mock_browsing_data_quota_helper.h" 17 #include "chrome/browser/mock_browsing_data_quota_helper.h"
17 #include "chrome/browser/mock_browsing_data_local_storage_helper.h" 18 #include "chrome/browser/mock_browsing_data_local_storage_helper.h"
18 #include "chrome/test/base/testing_profile.h" 19 #include "chrome/test/base/testing_profile.h"
(...skipping 922 matching lines...) Expand 10 before | Expand all | Expand 10 after
941 mock_browsing_data_indexed_db_helper_, 942 mock_browsing_data_indexed_db_helper_,
942 mock_browsing_data_file_system_helper_, 943 mock_browsing_data_file_system_helper_,
943 mock_browsing_data_quota_helper_, 944 mock_browsing_data_quota_helper_,
944 false); 945 false);
945 mock_browsing_data_cookie_helper_->AddCookieSamples(host, "A=1"); 946 mock_browsing_data_cookie_helper_->AddCookieSamples(host, "A=1");
946 mock_browsing_data_cookie_helper_->Notify(); 947 mock_browsing_data_cookie_helper_->Notify();
947 948
948 TestingProfile profile; 949 TestingProfile profile;
949 HostContentSettingsMap* content_settings = 950 HostContentSettingsMap* content_settings =
950 profile.GetHostContentSettingsMap(); 951 profile.GetHostContentSettingsMap();
952 CookieSettings* cookie_settings = CookieSettings::GetForProfile(&profile);
951 MockSettingsObserver observer; 953 MockSettingsObserver observer;
952 954
953 CookieTreeRootNode* root = 955 CookieTreeRootNode* root =
954 static_cast<CookieTreeRootNode*>(cookies_model.GetRoot()); 956 static_cast<CookieTreeRootNode*>(cookies_model.GetRoot());
955 CookieTreeOriginNode* origin = root->GetOrCreateOriginNode(host); 957 CookieTreeOriginNode* origin = root->GetOrCreateOriginNode(host);
956 958
957 EXPECT_EQ(1, origin->child_count()); 959 EXPECT_EQ(1, origin->child_count());
958 EXPECT_TRUE(origin->CanCreateContentException()); 960 EXPECT_TRUE(origin->CanCreateContentException());
959 EXPECT_CALL(observer, 961 EXPECT_CALL(observer,
960 OnContentSettingsChanged( 962 OnContentSettingsChanged(
961 content_settings, 963 content_settings,
962 CONTENT_SETTINGS_TYPE_COOKIES, 964 CONTENT_SETTINGS_TYPE_COOKIES,
963 false, 965 false,
964 ContentSettingsPattern::FromURLNoWildcard(host), 966 ContentSettingsPattern::FromURLNoWildcard(host),
965 ContentSettingsPattern::Wildcard(), 967 ContentSettingsPattern::Wildcard(),
966 false)); 968 false));
967 EXPECT_CALL(observer, 969 EXPECT_CALL(observer,
968 OnContentSettingsChanged(content_settings, 970 OnContentSettingsChanged(content_settings,
969 CONTENT_SETTINGS_TYPE_COOKIES, 971 CONTENT_SETTINGS_TYPE_COOKIES,
970 false, 972 false,
971 ContentSettingsPattern::FromURL(host), 973 ContentSettingsPattern::FromURL(host),
972 ContentSettingsPattern::Wildcard(), 974 ContentSettingsPattern::Wildcard(),
973 false)); 975 false));
974 origin->CreateContentException( 976 origin->CreateContentException(
975 content_settings, CONTENT_SETTING_SESSION_ONLY); 977 cookie_settings, CONTENT_SETTING_SESSION_ONLY);
976 EXPECT_EQ(CONTENT_SETTING_SESSION_ONLY, 978 EXPECT_TRUE(cookie_settings->IsReadingCookieAllowed(host, host));
977 content_settings->GetCookieContentSetting(host, host, true)); 979 EXPECT_TRUE(cookie_settings->IsCookieSessionOnly(host));
978 } 980 }
979 981
980 TEST_F(CookiesTreeModelTest, FileSystemFilter) { 982 TEST_F(CookiesTreeModelTest, FileSystemFilter) {
981 scoped_ptr<CookiesTreeModel> cookies_model( 983 scoped_ptr<CookiesTreeModel> cookies_model(
982 CreateCookiesTreeModelWithInitialSample()); 984 CreateCookiesTreeModelWithInitialSample());
983 985
984 cookies_model->UpdateSearchResults(std::wstring(L"fshost1")); 986 cookies_model->UpdateSearchResults(std::wstring(L"fshost1"));
985 EXPECT_EQ("http://fshost1:1/", 987 EXPECT_EQ("http://fshost1:1/",
986 GetDisplayedFileSystems(cookies_model.get())); 988 GetDisplayedFileSystems(cookies_model.get()));
987 989
988 cookies_model->UpdateSearchResults(std::wstring(L"fshost2")); 990 cookies_model->UpdateSearchResults(std::wstring(L"fshost2"));
989 EXPECT_EQ("http://fshost2:2/", 991 EXPECT_EQ("http://fshost2:2/",
990 GetDisplayedFileSystems(cookies_model.get())); 992 GetDisplayedFileSystems(cookies_model.get()));
991 993
992 cookies_model->UpdateSearchResults(std::wstring(L"fshost3")); 994 cookies_model->UpdateSearchResults(std::wstring(L"fshost3"));
993 EXPECT_EQ("http://fshost3:3/", 995 EXPECT_EQ("http://fshost3:3/",
994 GetDisplayedFileSystems(cookies_model.get())); 996 GetDisplayedFileSystems(cookies_model.get()));
995 997
996 cookies_model->UpdateSearchResults(std::wstring()); 998 cookies_model->UpdateSearchResults(std::wstring());
997 EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/", 999 EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
998 GetDisplayedFileSystems(cookies_model.get())); 1000 GetDisplayedFileSystems(cookies_model.get()));
999 } 1001 }
1000 1002
1001 } // namespace 1003 } // namespace
OLDNEW
« no previous file with comments | « chrome/browser/cookies_tree_model.cc ('k') | chrome/browser/extensions/extension_content_settings_api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698