From f26795ebb86f6bcd4da9d4971e252eea933cfdf3 Mon Sep 17 00:00:00 2001 From: "michaeln@chromium.org" Date: Fri, 26 Feb 2010 23:45:35 +0000 Subject: Teach the cookie tree view and model about appcaches. Not hooked up to real data yet, but the view and model pieces are in place for windows and gtk (not yet done for the mac). Also adds a 'name' attribute to database details pane, cleans up the layout of the detail panes on windows. BUG=25977 TEST=manual Review URL: http://codereview.chromium.org/650110 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40181 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/cookies_tree_model_unittest.cc | 32 ++++++++++++++++++++------- 1 file changed, 24 insertions(+), 8 deletions(-) (limited to 'chrome/browser/cookies_tree_model_unittest.cc') diff --git a/chrome/browser/cookies_tree_model_unittest.cc b/chrome/browser/cookies_tree_model_unittest.cc index 925a141..6e026b8 100644 --- a/chrome/browser/cookies_tree_model_unittest.cc +++ b/chrome/browser/cookies_tree_model_unittest.cc @@ -7,6 +7,7 @@ #include #include "app/l10n_util.h" +#include "chrome/browser/mock_browsing_data_appcache_helper.h" #include "chrome/browser/mock_browsing_data_database_helper.h" #include "chrome/browser/mock_browsing_data_local_storage_helper.h" #include "chrome/browser/net/url_request_context_getter.h" @@ -32,6 +33,8 @@ class CookiesTreeModelTest : public testing::Test { new MockBrowsingDataDatabaseHelper(profile_.get()); mock_browsing_data_local_storage_helper_ = new MockBrowsingDataLocalStorageHelper(profile_.get()); + mock_browsing_data_appcache_helper_ = + new MockBrowsingDataAppCacheHelper(profile_.get()); } CookiesTreeModel* CreateCookiesTreeModelWithInitialSample() { @@ -41,7 +44,8 @@ class CookiesTreeModelTest : public testing::Test { monster->SetCookie(GURL("http://foo3"), "C=1"); CookiesTreeModel* cookies_model = new CookiesTreeModel( profile_.get(), mock_browsing_data_database_helper_, - mock_browsing_data_local_storage_helper_); + mock_browsing_data_local_storage_helper_, + mock_browsing_data_appcache_helper_); mock_browsing_data_database_helper_->AddDatabaseSamples(); mock_browsing_data_database_helper_->Notify(); mock_browsing_data_local_storage_helper_->AddLocalStorageSamples(); @@ -91,6 +95,9 @@ class CookiesTreeModelTest : public testing::Test { return node->GetDetailedInfo().database_info->database_name + ","; case CookieTreeNode::DetailedInfo::TYPE_COOKIE: return node->GetDetailedInfo().cookie->second.Name() + ","; + case CookieTreeNode::DetailedInfo::TYPE_APPCACHE: + return node->GetDetailedInfo().appcache_info->manifest_url.spec() + + ","; default: return ""; } @@ -142,6 +149,11 @@ class CookiesTreeModelTest : public testing::Test { CookieTreeNode::DetailedInfo::TYPE_LOCAL_STORAGE); } + std::string GetDisplayedAppCaches(CookiesTreeModel* cookies_model) { + return GetDisplayedNodes(cookies_model, + CookieTreeNode::DetailedInfo::TYPE_APPCACHE); + } + // do not call on the root void DeleteStoredObjects(CookieTreeNode* node) { node->DeleteStoredObjects(); @@ -160,6 +172,8 @@ class CookiesTreeModelTest : public testing::Test { mock_browsing_data_database_helper_; scoped_refptr mock_browsing_data_local_storage_helper_; + scoped_refptr + mock_browsing_data_appcache_helper_; }; TEST_F(CookiesTreeModelTest, RemoveAll) { @@ -327,7 +341,8 @@ TEST_F(CookiesTreeModelTest, RemoveSingleCookieNode) { monster->SetCookie(GURL("http://foo3"), "D=1"); CookiesTreeModel cookies_model(profile_.get(), mock_browsing_data_database_helper_, - mock_browsing_data_local_storage_helper_); + mock_browsing_data_local_storage_helper_, + mock_browsing_data_appcache_helper_); mock_browsing_data_database_helper_->AddDatabaseSamples(); mock_browsing_data_database_helper_->Notify(); mock_browsing_data_local_storage_helper_->AddLocalStorageSamples(); @@ -365,7 +380,8 @@ TEST_F(CookiesTreeModelTest, RemoveSingleCookieNodeOf3) { monster->SetCookie(GURL("http://foo3"), "E=1"); CookiesTreeModel cookies_model(profile_.get(), mock_browsing_data_database_helper_, - mock_browsing_data_local_storage_helper_); + mock_browsing_data_local_storage_helper_, + mock_browsing_data_appcache_helper_); mock_browsing_data_database_helper_->AddDatabaseSamples(); mock_browsing_data_database_helper_->Notify(); mock_browsing_data_local_storage_helper_->AddLocalStorageSamples(); @@ -403,8 +419,9 @@ TEST_F(CookiesTreeModelTest, RemoveSecondOrigin) { monster->SetCookie(GURL("http://foo3"), "D=1"); monster->SetCookie(GURL("http://foo3"), "E=1"); CookiesTreeModel cookies_model(profile_.get(), - mock_browsing_data_database_helper_, - mock_browsing_data_local_storage_helper_); + mock_browsing_data_database_helper_, + mock_browsing_data_local_storage_helper_, + mock_browsing_data_appcache_helper_); { SCOPED_TRACE("Initial State 5 cookies"); // 11 because there's the root, then foo1 -> cookies -> a, @@ -437,7 +454,8 @@ TEST_F(CookiesTreeModelTest, OriginOrdering) { CookiesTreeModel cookies_model(profile_.get(), new MockBrowsingDataDatabaseHelper(profile_.get()), - new MockBrowsingDataLocalStorageHelper(profile_.get())); + new MockBrowsingDataLocalStorageHelper(profile_.get()), + new MockBrowsingDataAppCacheHelper(profile_.get())); { SCOPED_TRACE("Initial State 8 cookies"); @@ -454,6 +472,4 @@ TEST_F(CookiesTreeModelTest, OriginOrdering) { } } - - } // namespace -- cgit v1.1