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

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

Issue 7713034: HostContentSettingsMap refactoring. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Code review comments. Threading fixes. Mac + win fixes. Created 9 years, 3 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) 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 =
953 CookieSettings::GetForProfile(&profile);
951 MockSettingsObserver observer; 954 MockSettingsObserver observer;
952 955
953 CookieTreeRootNode* root = 956 CookieTreeRootNode* root =
954 static_cast<CookieTreeRootNode*>(cookies_model.GetRoot()); 957 static_cast<CookieTreeRootNode*>(cookies_model.GetRoot());
955 CookieTreeOriginNode* origin = root->GetOrCreateOriginNode(host); 958 CookieTreeOriginNode* origin = root->GetOrCreateOriginNode(host);
956 959
957 EXPECT_EQ(1, origin->child_count()); 960 EXPECT_EQ(1, origin->child_count());
958 EXPECT_TRUE(origin->CanCreateContentException()); 961 EXPECT_TRUE(origin->CanCreateContentException());
959 EXPECT_CALL(observer, 962 EXPECT_CALL(observer,
960 OnContentSettingsChanged( 963 OnContentSettingsChanged(
961 content_settings, 964 content_settings,
962 CONTENT_SETTINGS_TYPE_COOKIES, 965 CONTENT_SETTINGS_TYPE_COOKIES,
963 false, 966 false,
964 ContentSettingsPattern::FromURLNoWildcard(host), 967 ContentSettingsPattern::FromURLNoWildcard(host),
965 ContentSettingsPattern::Wildcard(), 968 ContentSettingsPattern::Wildcard(),
966 false)); 969 false));
967 EXPECT_CALL(observer, 970 EXPECT_CALL(observer,
968 OnContentSettingsChanged(content_settings, 971 OnContentSettingsChanged(
972 content_settings,
969 CONTENT_SETTINGS_TYPE_COOKIES, 973 CONTENT_SETTINGS_TYPE_COOKIES,
970 false, 974 false,
971 ContentSettingsPattern::FromURL(host), 975 ContentSettingsPattern::FromURL(host),
972 ContentSettingsPattern::Wildcard(), 976 ContentSettingsPattern::Wildcard(),
973 false)); 977 false));
978 EXPECT_CALL(observer,
979 OnContentSettingsChanged(
980 content_settings,
981 CONTENT_SETTINGS_TYPE_COOKIES_SESSION_ONLY,
982 false,
983 ContentSettingsPattern::FromURLNoWildcard(host),
984 ContentSettingsPattern::Wildcard(),
985 false));
986 EXPECT_CALL(observer,
987 OnContentSettingsChanged(
988 content_settings,
989 CONTENT_SETTINGS_TYPE_COOKIES_SESSION_ONLY,
990 false,
991 ContentSettingsPattern::FromURL(host),
992 ContentSettingsPattern::Wildcard(),
993 false));
974 origin->CreateContentException( 994 origin->CreateContentException(
975 content_settings, CONTENT_SETTING_SESSION_ONLY); 995 cookie_settings, CONTENT_SETTING_SESSION_ONLY);
976 EXPECT_EQ(CONTENT_SETTING_SESSION_ONLY, 996 EXPECT_TRUE(cookie_settings->IsSettingCookieAllowed(host, host));
977 content_settings->GetCookieContentSetting(host, host, true)); 997 EXPECT_TRUE(cookie_settings->IsCookieSessionOnly(host));
978 } 998 }
979 999
980 TEST_F(CookiesTreeModelTest, FileSystemFilter) { 1000 TEST_F(CookiesTreeModelTest, FileSystemFilter) {
981 scoped_ptr<CookiesTreeModel> cookies_model( 1001 scoped_ptr<CookiesTreeModel> cookies_model(
982 CreateCookiesTreeModelWithInitialSample()); 1002 CreateCookiesTreeModelWithInitialSample());
983 1003
984 cookies_model->UpdateSearchResults(std::wstring(L"fshost1")); 1004 cookies_model->UpdateSearchResults(std::wstring(L"fshost1"));
985 EXPECT_EQ("http://fshost1:1/", 1005 EXPECT_EQ("http://fshost1:1/",
986 GetDisplayedFileSystems(cookies_model.get())); 1006 GetDisplayedFileSystems(cookies_model.get()));
987 1007
988 cookies_model->UpdateSearchResults(std::wstring(L"fshost2")); 1008 cookies_model->UpdateSearchResults(std::wstring(L"fshost2"));
989 EXPECT_EQ("http://fshost2:2/", 1009 EXPECT_EQ("http://fshost2:2/",
990 GetDisplayedFileSystems(cookies_model.get())); 1010 GetDisplayedFileSystems(cookies_model.get()));
991 1011
992 cookies_model->UpdateSearchResults(std::wstring(L"fshost3")); 1012 cookies_model->UpdateSearchResults(std::wstring(L"fshost3"));
993 EXPECT_EQ("http://fshost3:3/", 1013 EXPECT_EQ("http://fshost3:3/",
994 GetDisplayedFileSystems(cookies_model.get())); 1014 GetDisplayedFileSystems(cookies_model.get()));
995 1015
996 cookies_model->UpdateSearchResults(std::wstring()); 1016 cookies_model->UpdateSearchResults(std::wstring());
997 EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/", 1017 EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
998 GetDisplayedFileSystems(cookies_model.get())); 1018 GetDisplayedFileSystems(cookies_model.get()));
999 } 1019 }
1000 1020
1001 } // namespace 1021 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698