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

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. 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"
10 #include "chrome/browser/content_settings/cookie_settings_factory.h"
9 #include "chrome/browser/content_settings/host_content_settings_map.h" 11 #include "chrome/browser/content_settings/host_content_settings_map.h"
10 #include "chrome/browser/content_settings/mock_settings_observer.h" 12 #include "chrome/browser/content_settings/mock_settings_observer.h"
11 #include "chrome/browser/mock_browsing_data_appcache_helper.h" 13 #include "chrome/browser/mock_browsing_data_appcache_helper.h"
12 #include "chrome/browser/mock_browsing_data_cookie_helper.h" 14 #include "chrome/browser/mock_browsing_data_cookie_helper.h"
13 #include "chrome/browser/mock_browsing_data_database_helper.h" 15 #include "chrome/browser/mock_browsing_data_database_helper.h"
14 #include "chrome/browser/mock_browsing_data_file_system_helper.h" 16 #include "chrome/browser/mock_browsing_data_file_system_helper.h"
15 #include "chrome/browser/mock_browsing_data_indexed_db_helper.h" 17 #include "chrome/browser/mock_browsing_data_indexed_db_helper.h"
16 #include "chrome/browser/mock_browsing_data_quota_helper.h" 18 #include "chrome/browser/mock_browsing_data_quota_helper.h"
17 #include "chrome/browser/mock_browsing_data_local_storage_helper.h" 19 #include "chrome/browser/mock_browsing_data_local_storage_helper.h"
18 #include "chrome/test/base/testing_profile.h" 20 #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_, 943 mock_browsing_data_indexed_db_helper_,
942 mock_browsing_data_file_system_helper_, 944 mock_browsing_data_file_system_helper_,
943 mock_browsing_data_quota_helper_, 945 mock_browsing_data_quota_helper_,
944 false); 946 false);
945 mock_browsing_data_cookie_helper_->AddCookieSamples(host, "A=1"); 947 mock_browsing_data_cookie_helper_->AddCookieSamples(host, "A=1");
946 mock_browsing_data_cookie_helper_->Notify(); 948 mock_browsing_data_cookie_helper_->Notify();
947 949
948 TestingProfile profile; 950 TestingProfile profile;
949 HostContentSettingsMap* content_settings = 951 HostContentSettingsMap* content_settings =
950 profile.GetHostContentSettingsMap(); 952 profile.GetHostContentSettingsMap();
953 CookieSettings* cookie_settings =
954 CookieSettingsFactory::GetForProfile(&profile);
951 MockSettingsObserver observer; 955 MockSettingsObserver observer;
952 956
953 CookieTreeRootNode* root = 957 CookieTreeRootNode* root =
954 static_cast<CookieTreeRootNode*>(cookies_model.GetRoot()); 958 static_cast<CookieTreeRootNode*>(cookies_model.GetRoot());
955 CookieTreeOriginNode* origin = root->GetOrCreateOriginNode(host); 959 CookieTreeOriginNode* origin = root->GetOrCreateOriginNode(host);
956 960
957 EXPECT_EQ(1, origin->child_count()); 961 EXPECT_EQ(1, origin->child_count());
958 EXPECT_TRUE(origin->CanCreateContentException()); 962 EXPECT_TRUE(origin->CanCreateContentException());
959 EXPECT_CALL(observer, 963 EXPECT_CALL(observer,
960 OnContentSettingsChanged( 964 OnContentSettingsChanged(
961 content_settings, 965 content_settings,
962 CONTENT_SETTINGS_TYPE_COOKIES, 966 CONTENT_SETTINGS_TYPE_COOKIES,
963 false, 967 false,
964 ContentSettingsPattern::FromURLNoWildcard(host), 968 ContentSettingsPattern::FromURLNoWildcard(host),
965 ContentSettingsPattern::Wildcard(), 969 ContentSettingsPattern::Wildcard(),
966 false)); 970 false));
967 EXPECT_CALL(observer, 971 EXPECT_CALL(observer,
968 OnContentSettingsChanged(content_settings, 972 OnContentSettingsChanged(
973 content_settings,
969 CONTENT_SETTINGS_TYPE_COOKIES, 974 CONTENT_SETTINGS_TYPE_COOKIES,
970 false, 975 false,
971 ContentSettingsPattern::FromURL(host), 976 ContentSettingsPattern::FromURL(host),
972 ContentSettingsPattern::Wildcard(), 977 ContentSettingsPattern::Wildcard(),
973 false)); 978 false));
979 EXPECT_CALL(observer,
980 OnContentSettingsChanged(
981 content_settings,
982 CONTENT_SETTINGS_TYPE_COOKIES_SESSION_ONLY,
983 false,
984 ContentSettingsPattern::FromURLNoWildcard(host),
985 ContentSettingsPattern::Wildcard(),
986 false));
987 EXPECT_CALL(observer,
988 OnContentSettingsChanged(
989 content_settings,
990 CONTENT_SETTINGS_TYPE_COOKIES_SESSION_ONLY,
991 false,
992 ContentSettingsPattern::FromURL(host),
993 ContentSettingsPattern::Wildcard(),
994 false));
974 origin->CreateContentException( 995 origin->CreateContentException(
975 content_settings, CONTENT_SETTING_SESSION_ONLY); 996 cookie_settings, CONTENT_SETTING_SESSION_ONLY);
976 EXPECT_EQ(CONTENT_SETTING_SESSION_ONLY, 997 EXPECT_TRUE(cookie_settings->IsSettingCookieAllowed(host, host));
977 content_settings->GetCookieContentSetting(host, host, true)); 998 EXPECT_TRUE(cookie_settings->IsCookieSessionOnly(host));
978 } 999 }
979 1000
980 TEST_F(CookiesTreeModelTest, FileSystemFilter) { 1001 TEST_F(CookiesTreeModelTest, FileSystemFilter) {
981 scoped_ptr<CookiesTreeModel> cookies_model( 1002 scoped_ptr<CookiesTreeModel> cookies_model(
982 CreateCookiesTreeModelWithInitialSample()); 1003 CreateCookiesTreeModelWithInitialSample());
983 1004
984 cookies_model->UpdateSearchResults(std::wstring(L"fshost1")); 1005 cookies_model->UpdateSearchResults(std::wstring(L"fshost1"));
985 EXPECT_EQ("http://fshost1:1/", 1006 EXPECT_EQ("http://fshost1:1/",
986 GetDisplayedFileSystems(cookies_model.get())); 1007 GetDisplayedFileSystems(cookies_model.get()));
987 1008
988 cookies_model->UpdateSearchResults(std::wstring(L"fshost2")); 1009 cookies_model->UpdateSearchResults(std::wstring(L"fshost2"));
989 EXPECT_EQ("http://fshost2:2/", 1010 EXPECT_EQ("http://fshost2:2/",
990 GetDisplayedFileSystems(cookies_model.get())); 1011 GetDisplayedFileSystems(cookies_model.get()));
991 1012
992 cookies_model->UpdateSearchResults(std::wstring(L"fshost3")); 1013 cookies_model->UpdateSearchResults(std::wstring(L"fshost3"));
993 EXPECT_EQ("http://fshost3:3/", 1014 EXPECT_EQ("http://fshost3:3/",
994 GetDisplayedFileSystems(cookies_model.get())); 1015 GetDisplayedFileSystems(cookies_model.get()));
995 1016
996 cookies_model->UpdateSearchResults(std::wstring()); 1017 cookies_model->UpdateSearchResults(std::wstring());
997 EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/", 1018 EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
998 GetDisplayedFileSystems(cookies_model.get())); 1019 GetDisplayedFileSystems(cookies_model.get()));
999 } 1020 }
1000 1021
1001 } // namespace 1022 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698