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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/cookies_tree_model_unittest.cc

Issue 10536017: Refactoring CookiesTreeModel to support multiple data sources. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed all comments by Evan. Created 8 years, 6 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 side-by-side diff with in-line comments
Download patch
« chrome/browser/cookies_tree_model.cc ('K') | « chrome/browser/cookies_tree_model.cc ('k') | chrome/browser/local_data_container.h » ('j') | chrome/browser/local_data_container.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: 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 f1c47d5f90fe478b98a03084b6a6119b034d109d..0974a7d27a64421e4f7bd0fb89197bc8021c8bfa 100644
--- a/chrome/browser/cookies_tree_model_unittest.cc
+++ b/chrome/browser/cookies_tree_model_unittest.cc
@@ -22,6 +22,7 @@
#include "content/public/browser/notification_details.h"
#include "content/public/browser/notification_types.h"
#include "content/public/test/test_browser_thread.h"
+#include "net/base/net_util.h"
#include "net/url_request/url_request_context.h"
#include "net/url_request/url_request_context_getter.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -68,6 +69,16 @@ class CookiesTreeModelTest : public testing::Test {
new MockBrowsingDataQuotaHelper(profile_.get());
mock_browsing_data_server_bound_cert_helper_ =
new MockBrowsingDataServerBoundCertHelper();
+
+ // It is fine to reuse the profile request context for the app, since
+ // the mock cookie helper maintains its own list internally and doesn't
+ // really use the request context. Same is true for the rest.
+ mock_browsing_data_cookie_helper_app_ =
+ new MockBrowsingDataCookieHelper(profile_->GetRequestContext());
James Hawkins 2012/06/21 00:04:12 nit: Indentation is off by two.
nasko 2012/06/21 16:22:12 Done.
+ mock_browsing_data_database_helper_app_ =
+ new MockBrowsingDataDatabaseHelper(profile_.get());
+ mock_browsing_data_local_storage_helper_app_ =
+ new MockBrowsingDataLocalStorageHelper(profile_.get());
}
virtual void TearDown() OVERRIDE {
@@ -82,8 +93,11 @@ class CookiesTreeModelTest : public testing::Test {
message_loop_.RunAllPending();
}
- CookiesTreeModel* CreateCookiesTreeModelWithInitialSample() {
- CookiesTreeModel* cookies_model = new CookiesTreeModel(
+ CookiesTreeModel* CreateCookiesTreeModelWithInitialSample(bool add_app) {
+ ContainerMap containers_map;
+
+ containers_map[""] = new LocalDataContainer(
+ "Drive-By-Web", "",
mock_browsing_data_cookie_helper_,
mock_browsing_data_database_helper_,
mock_browsing_data_local_storage_helper_,
@@ -92,8 +106,26 @@ class CookiesTreeModelTest : public testing::Test {
mock_browsing_data_indexed_db_helper_,
mock_browsing_data_file_system_helper_,
mock_browsing_data_quota_helper_,
- mock_browsing_data_server_bound_cert_helper_,
- false);
+ mock_browsing_data_server_bound_cert_helper_);
+
+ if (add_app) {
+ std::string app_id = "some-random-id";
+ // The three helpers are mandatory, the rest can be NULL.
+ containers_map[app_id] = new LocalDataContainer(
+ "Isolated App", app_id,
+ mock_browsing_data_cookie_helper_app_,
+ mock_browsing_data_database_helper_app_,
+ mock_browsing_data_local_storage_helper_app_,
+ NULL,
James Hawkins 2012/06/21 00:04:12 Optional nit: Condense parameters to save lines.
nasko 2012/06/21 16:22:12 Done.
+ NULL,
+ NULL,
+ NULL,
+ NULL,
+ NULL);
+ }
+
+ CookiesTreeModel* cookies_model = new CookiesTreeModel(containers_map,
+ false);
mock_browsing_data_cookie_helper_->
AddCookieSamples(GURL("http://foo1"), "A=1");
mock_browsing_data_cookie_helper_->
@@ -118,28 +150,51 @@ class CookiesTreeModelTest : public testing::Test {
mock_browsing_data_server_bound_cert_helper_->AddServerBoundCertSample(
"sbc2");
mock_browsing_data_server_bound_cert_helper_->Notify();
+
+ if (add_app) {
+ mock_browsing_data_cookie_helper_app_->
+ AddCookieSamples(GURL("http://app1"), "Z=1");
+ mock_browsing_data_cookie_helper_app_->
+ AddCookieSamples(GURL("http://app2"), "Y=1");
+ mock_browsing_data_cookie_helper_app_->
+ AddCookieSamples(GURL("http://app3"), "X=1");
+ mock_browsing_data_cookie_helper_app_->Notify();
+ }
+
{
SCOPED_TRACE("Initial State 3 cookies, 2 databases, 2 local storages, "
"2 session storages, 2 indexed DBs, 3 filesystems, "
- "2 quotas");
- // 45 because there's the root, then foo1 -> cookies -> a,
- // foo2 -> cookies -> b, foo3 -> cookies -> c,
- // dbhost1 -> database -> db1, dbhost2 -> database -> db2,
- // fshost1 -> filesystem -> http://fshost1:1/,
- // fshost2 -> filesystem -> http://fshost2:1/,
- // fshost3 -> filesystem -> http://fshost3:1/,
+ "2 quotas, 2 server bound certs");
+ // 52 because there's the root, then the app node,
+ // foo1 -> cookies -> a,
+ // foo2 -> cookies -> b,
+ // foo3 -> cookies -> c,
+ // dbhost1 -> database -> db1,
+ // dbhost2 -> database -> db2,
// host1 -> localstorage -> http://host1:1/,
+ // -> sessionstorage -> http://host1:1/,
// host2 -> localstorage -> http://host2:2/.
- // host1 -> sessionstorage -> http://host1:1/,
- // host2 -> sessionstorage -> http://host2:2/,
+ // -> sessionstorage -> http://host2:2/,
// idbhost1 -> indexeddb -> http://idbhost1:1/,
// idbhost2 -> indexeddb -> http://idbhost2:2/,
+ // fshost1 -> filesystem -> http://fshost1:1/,
+ // fshost2 -> filesystem -> http://fshost2:1/,
+ // fshost3 -> filesystem -> http://fshost3:1/,
// quotahost1 -> quotahost1,
// quotahost2 -> quotahost2,
// sbc1 -> sbcerts -> sbc1,
// sbc2 -> sbcerts -> sbc2.
- EXPECT_EQ(51, cookies_model->GetRoot()->GetTotalNodeCount());
- EXPECT_EQ("A,B,C", GetDisplayedCookies(cookies_model));
+ if (!add_app) {
+ EXPECT_EQ(52, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ("A,B,C", GetDisplayedCookies(cookies_model));
+ } else {
+ // Once we add the app, we have 10 more nodes, one for the app, then
+ // app1 -> cookies -> z,
+ // app2 -> cookies -> y,
+ // app3 -> cookies -> x,
+ EXPECT_EQ(62, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ("A,B,C,Z,Y,X", GetDisplayedCookies(cookies_model));
+ }
EXPECT_EQ("db1,db2", GetDisplayedDatabases(cookies_model));
EXPECT_EQ("http://host1:1/,http://host2:2/",
GetDisplayedLocalStorages(cookies_model));
@@ -327,11 +382,19 @@ class CookiesTreeModelTest : public testing::Test {
mock_browsing_data_quota_helper_;
scoped_refptr<MockBrowsingDataServerBoundCertHelper>
mock_browsing_data_server_bound_cert_helper_;
+
+ // App helpers.
+ scoped_refptr<MockBrowsingDataCookieHelper>
+ mock_browsing_data_cookie_helper_app_;
+ scoped_refptr<MockBrowsingDataDatabaseHelper>
+ mock_browsing_data_database_helper_app_;
+ scoped_refptr<MockBrowsingDataLocalStorageHelper>
+ mock_browsing_data_local_storage_helper_app_;
};
TEST_F(CookiesTreeModelTest, RemoveAll) {
scoped_ptr<CookiesTreeModel> cookies_model(
- CreateCookiesTreeModelWithInitialSample());
+ CreateCookiesTreeModelWithInitialSample(false));
// Reset the selection of the first row.
{
@@ -365,12 +428,13 @@ TEST_F(CookiesTreeModelTest, RemoveAll) {
// Make sure the nodes are also deleted from the model's cache.
// http://crbug.com/43249
- cookies_model->UpdateSearchResults(std::wstring());
+ cookies_model->UpdateSearchResults(string16());
{
+ // 2 nodes - root and app
SCOPED_TRACE("After removing");
- EXPECT_EQ(1, cookies_model->GetRoot()->GetTotalNodeCount());
- EXPECT_EQ(0, cookies_model->GetRoot()->child_count());
+ EXPECT_EQ(2, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(1, cookies_model->GetRoot()->child_count());
EXPECT_EQ(std::string(""), GetDisplayedCookies(cookies_model.get()));
EXPECT_TRUE(mock_browsing_data_cookie_helper_->AllDeleted());
EXPECT_TRUE(mock_browsing_data_database_helper_->AllDeleted());
@@ -384,7 +448,7 @@ TEST_F(CookiesTreeModelTest, RemoveAll) {
TEST_F(CookiesTreeModelTest, Remove) {
scoped_ptr<CookiesTreeModel> cookies_model(
- CreateCookiesTreeModelWithInitialSample());
+ CreateCookiesTreeModelWithInitialSample(false));
// Children start out arranged as follows:
//
@@ -408,7 +472,7 @@ TEST_F(CookiesTreeModelTest, Remove) {
// Here, we'll remove them one by one, starting from the end, and
// check that the state makes sense.
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(15));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(15));
{
SCOPED_TRACE("`sbc2` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -425,9 +489,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
GetDisplayedQuotas(cookies_model.get()));
EXPECT_EQ("sbc1",
GetDisplayedServerBoundCerts(cookies_model.get()));
- EXPECT_EQ(48, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(49, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(14));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(14));
{
SCOPED_TRACE("`sbc1` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -442,9 +506,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
GetDisplayedIndexedDBs(cookies_model.get()));
EXPECT_EQ("quotahost1,quotahost2",
GetDisplayedQuotas(cookies_model.get()));
- EXPECT_EQ(45, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(46, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(13));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(13));
{
SCOPED_TRACE("`quotahost2` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -459,9 +523,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
GetDisplayedIndexedDBs(cookies_model.get()));
EXPECT_EQ("quotahost1",
GetDisplayedQuotas(cookies_model.get()));
- EXPECT_EQ(43, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(44, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(12));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(12));
{
SCOPED_TRACE("`quotahost1` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -474,9 +538,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("http://idbhost1:1/,http://idbhost2:2/",
GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(41, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(42, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(11));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(11));
{
SCOPED_TRACE("`idbhost2` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -489,9 +553,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("http://idbhost1:1/",
GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(38, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(39, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(10));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(10));
{
SCOPED_TRACE("`idbhost1` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -503,9 +567,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(35, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(36, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(9));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(9));
{
SCOPED_TRACE("`host2` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -517,9 +581,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(30, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(31, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(8));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(8));
{
SCOPED_TRACE("`host1` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -529,9 +593,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(25, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(26, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(7));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(7));
{
SCOPED_TRACE("`gdbhost2` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -541,9 +605,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(22, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(23, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(6));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(6));
{
SCOPED_TRACE("`gdbhost1` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -553,9 +617,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(19, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(20, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(5));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(5));
{
SCOPED_TRACE("`fshost3` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -565,9 +629,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("http://fshost1:1/,http://fshost2:2/",
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(16, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(17, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(4));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(4));
{
SCOPED_TRACE("`fshost2` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -577,9 +641,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("http://fshost1:1/",
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(13, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(14, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(3));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(3));
{
SCOPED_TRACE("`fshost1` removed.");
EXPECT_STREQ("A,B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -588,9 +652,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("", GetDisplayedSessionStorages(cookies_model.get()));
EXPECT_EQ("", GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(10, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(11, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(2));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(2));
{
SCOPED_TRACE("`foo3` removed.");
EXPECT_STREQ("A,B", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -599,9 +663,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("", GetDisplayedSessionStorages(cookies_model.get()));
EXPECT_EQ("", GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(7, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(8, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(1));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(1));
{
SCOPED_TRACE("`foo2` removed.");
EXPECT_STREQ("A", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -610,9 +674,9 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("", GetDisplayedSessionStorages(cookies_model.get()));
EXPECT_EQ("", GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(4, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(5, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0));
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(0));
{
SCOPED_TRACE("`foo1` removed.");
EXPECT_STREQ("", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -621,34 +685,22 @@ TEST_F(CookiesTreeModelTest, Remove) {
EXPECT_EQ("", GetDisplayedSessionStorages(cookies_model.get()));
EXPECT_EQ("", GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("", GetDisplayedIndexedDBs(cookies_model.get()));
- EXPECT_EQ(1, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(2, cookies_model->GetRoot()->GetTotalNodeCount());
}
}
TEST_F(CookiesTreeModelTest, RemoveCookiesNode) {
scoped_ptr<CookiesTreeModel> cookies_model(
- CreateCookiesTreeModelWithInitialSample());
+ CreateCookiesTreeModelWithInitialSample(false));
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(0)->GetChild(0));
+ DeleteStoredObjects(
+ cookies_model->GetRoot()->GetChild(0)->GetChild(0)->GetChild(0));
{
SCOPED_TRACE("First origin removed");
EXPECT_STREQ("B,C", GetDisplayedCookies(cookies_model.get()).c_str());
- // 43 because in this case, the origin remains, although the COOKIES
- // node beneath it has been deleted. So, we have
- // root -> foo1 -> cookies -> a, foo2, foo3 -> cookies -> c
- // dbhost1 -> database -> db1, dbhost2 -> database -> db2,
- // fshost1 -> filesystem -> http://fshost1:1/,
- // fshost2 -> filesystem -> http://fshost2:1/,
- // fshost3 -> filesystem -> http://fshost3:1/,
- // host1 -> localstorage -> http://host1:1/,
- // host2 -> localstorage -> http://host2:2/,
- // idbhost1 -> sessionstorage -> http://idbhost1:1/,
- // idbhost2 -> sessionstorage -> http://idbhost2:2/,
- // quotahost1 -> quotahost1,
- // quotahost2 -> quotahost1,
- // sbc1 -> sbcerts -> sbc1,
- // sbc2 -> sbcerts -> sbc2.
- EXPECT_EQ(49, cookies_model->GetRoot()->GetTotalNodeCount());
+ // 50 because in this case, the origin remains, although the COOKIES
+ // node beneath it has been deleted.
+ EXPECT_EQ(50, cookies_model->GetRoot()->GetTotalNodeCount());
EXPECT_EQ("db1,db2", GetDisplayedDatabases(cookies_model.get()));
EXPECT_EQ("http://host1:1/,http://host2:2/",
GetDisplayedLocalStorages(cookies_model.get()));
@@ -662,7 +714,8 @@ TEST_F(CookiesTreeModelTest, RemoveCookiesNode) {
EXPECT_EQ("sbc1,sbc2", GetDisplayedServerBoundCerts(cookies_model.get()));
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(6)->GetChild(0));
+ DeleteStoredObjects(
+ cookies_model->GetRoot()->GetChild(0)->GetChild(6)->GetChild(0));
{
SCOPED_TRACE("First database removed");
EXPECT_STREQ("B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -677,10 +730,11 @@ TEST_F(CookiesTreeModelTest, RemoveCookiesNode) {
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("quotahost1,quotahost2", GetDisplayedQuotas(cookies_model.get()));
EXPECT_EQ("sbc1,sbc2", GetDisplayedServerBoundCerts(cookies_model.get()));
- EXPECT_EQ(47, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(48, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(8)->GetChild(0));
+ DeleteStoredObjects(
+ cookies_model->GetRoot()->GetChild(0)->GetChild(8)->GetChild(0));
{
SCOPED_TRACE("First origin removed");
EXPECT_STREQ("B,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -695,15 +749,16 @@ TEST_F(CookiesTreeModelTest, RemoveCookiesNode) {
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("quotahost1,quotahost2", GetDisplayedQuotas(cookies_model.get()));
EXPECT_EQ("sbc1,sbc2", GetDisplayedServerBoundCerts(cookies_model.get()));
- EXPECT_EQ(45, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(46, cookies_model->GetRoot()->GetTotalNodeCount());
}
}
TEST_F(CookiesTreeModelTest, RemoveCookieNode) {
scoped_ptr<CookiesTreeModel> cookies_model(
- CreateCookiesTreeModelWithInitialSample());
+ CreateCookiesTreeModelWithInitialSample(false));
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(1)->GetChild(0));
+ DeleteStoredObjects(
+ cookies_model->GetRoot()->GetChild(0)->GetChild(1)->GetChild(0));
{
SCOPED_TRACE("Second origin COOKIES node removed");
EXPECT_STREQ("A,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -718,25 +773,13 @@ TEST_F(CookiesTreeModelTest, RemoveCookieNode) {
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("quotahost1,quotahost2", GetDisplayedQuotas(cookies_model.get()));
EXPECT_EQ("sbc1,sbc2", GetDisplayedServerBoundCerts(cookies_model.get()));
- // 43 because in this case, the origin remains, although the COOKIES
- // node beneath it has been deleted. So, we have
- // root -> foo1 -> cookies -> a, foo2, foo3 -> cookies -> c
- // dbhost1 -> database -> db1, dbhost2 -> database -> db2,
- // fshost1 -> filesystem -> http://fshost1:1/,
- // fshost2 -> filesystem -> http://fshost2:1/,
- // fshost3 -> filesystem -> http://fshost3:1/,
- // host1 -> localstorage -> http://host1:1/,
- // host2 -> localstorage -> http://host2:2/,
- // host1 -> sessionstorage -> http://host1:1/,
- // host2 -> sessionstorage -> http://host2:2/,
- // idbhost1 -> sessionstorage -> http://idbhost1:1/,
- // idbhost2 -> sessionstorage -> http://idbhost2:2/,
- // quotahost1 -> quotahost1,
- // quotahost2 -> quotahost2.
- EXPECT_EQ(49, cookies_model->GetRoot()->GetTotalNodeCount());
+ // 50 because in this case, the origin remains, although the COOKIES
+ // node beneath it has been deleted.
+ EXPECT_EQ(50, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(6)->GetChild(0));
+ DeleteStoredObjects(
+ cookies_model->GetRoot()->GetChild(0)->GetChild(6)->GetChild(0));
{
SCOPED_TRACE("First database removed");
EXPECT_STREQ("A,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -751,10 +794,11 @@ TEST_F(CookiesTreeModelTest, RemoveCookieNode) {
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("quotahost1,quotahost2", GetDisplayedQuotas(cookies_model.get()));
EXPECT_EQ("sbc1,sbc2", GetDisplayedServerBoundCerts(cookies_model.get()));
- EXPECT_EQ(47, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(48, cookies_model->GetRoot()->GetTotalNodeCount());
}
- DeleteStoredObjects(cookies_model->GetRoot()->GetChild(8)->GetChild(0));
+ DeleteStoredObjects(
+ cookies_model->GetRoot()->GetChild(0)->GetChild(8)->GetChild(0));
{
SCOPED_TRACE("First origin removed");
EXPECT_STREQ("A,C", GetDisplayedCookies(cookies_model.get()).c_str());
@@ -769,21 +813,26 @@ TEST_F(CookiesTreeModelTest, RemoveCookieNode) {
GetDisplayedFileSystems(cookies_model.get()));
EXPECT_EQ("quotahost1,quotahost2", GetDisplayedQuotas(cookies_model.get()));
EXPECT_EQ("sbc1,sbc2", GetDisplayedServerBoundCerts(cookies_model.get()));
- EXPECT_EQ(45, cookies_model->GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(46, cookies_model->GetRoot()->GetTotalNodeCount());
}
}
TEST_F(CookiesTreeModelTest, RemoveSingleCookieNode) {
- CookiesTreeModel cookies_model(mock_browsing_data_cookie_helper_,
- mock_browsing_data_database_helper_,
- mock_browsing_data_local_storage_helper_,
- mock_browsing_data_session_storage_helper_,
- mock_browsing_data_appcache_helper_,
- mock_browsing_data_indexed_db_helper_,
- mock_browsing_data_file_system_helper_,
- mock_browsing_data_quota_helper_,
- mock_browsing_data_server_bound_cert_helper_,
- false);
+ ContainerMap container_map;
+
+ container_map[""] = new LocalDataContainer(
+ "Drive-By-Web", "",
+ mock_browsing_data_cookie_helper_,
+ mock_browsing_data_database_helper_,
+ mock_browsing_data_local_storage_helper_,
+ mock_browsing_data_session_storage_helper_,
+ mock_browsing_data_appcache_helper_,
+ mock_browsing_data_indexed_db_helper_,
+ mock_browsing_data_file_system_helper_,
+ mock_browsing_data_quota_helper_,
+ mock_browsing_data_server_bound_cert_helper_);
+ CookiesTreeModel cookies_model(container_map, false);
+
mock_browsing_data_cookie_helper_->
AddCookieSamples(GURL("http://foo1"), "A=1");
mock_browsing_data_cookie_helper_->
@@ -810,20 +859,24 @@ TEST_F(CookiesTreeModelTest, RemoveSingleCookieNode) {
SCOPED_TRACE("Initial State 4 cookies, 2 databases, 2 local storages, "
"2 session storages, 2 indexed DBs, 3 file systems, "
"2 quotas.");
- // 42 because there's the root, then foo1 -> cookies -> a,
- // foo2 -> cookies -> b, foo3 -> cookies -> c,d
- // dbhost1 -> database -> db1, dbhost2 -> database -> db2,
- // fshost1 -> filesystem -> http://fshost1:1/,
- // fshost2 -> filesystem -> http://fshost2:1/,
- // fshost3 -> filesystem -> http://fshost3:1/,
+ // 47 because there's the root, the app, then
+ // foo1 -> cookies -> a,
+ // foo2 -> cookies -> b,
+ // foo3 -> cookies -> c,d
+ // dbhost1 -> database -> db1,
+ // dbhost2 -> database -> db2,
// host1 -> localstorage -> http://host1:1/,
- // host1 -> sessionstorage -> http://host1:1/,
- // host2 -> sessionstorage -> http://host2:2/,
+ // -> sessionstorage -> http://host1:1/,
+ // host2 -> localstorage -> http://host2:2/,
+ // -> sessionstorage -> http://host2:2/,
// idbhost1 -> sessionstorage -> http://idbhost1:1/,
// idbhost2 -> sessionstorage -> http://idbhost2:2/,
+ // fshost1 -> filesystem -> http://fshost1:1/,
+ // fshost2 -> filesystem -> http://fshost2:1/,
+ // fshost3 -> filesystem -> http://fshost3:1/,
// quotahost1 -> quotahost1,
// quotahost2 -> quotahost2.
- EXPECT_EQ(46, cookies_model.GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(47, cookies_model.GetRoot()->GetTotalNodeCount());
EXPECT_STREQ("A,B,C,D", GetDisplayedCookies(&cookies_model).c_str());
EXPECT_EQ("db1,db2", GetDisplayedDatabases(&cookies_model));
EXPECT_EQ("http://host1:1/,http://host2:2/",
@@ -836,7 +889,7 @@ TEST_F(CookiesTreeModelTest, RemoveSingleCookieNode) {
GetDisplayedFileSystems(&cookies_model));
EXPECT_EQ("quotahost1,quotahost2", GetDisplayedQuotas(&cookies_model));
}
- DeleteStoredObjects(cookies_model.GetRoot()->GetChild(2));
+ DeleteStoredObjects(cookies_model.GetRoot()->GetChild(0)->GetChild(2));
{
SCOPED_TRACE("Third origin removed");
EXPECT_STREQ("A,B", GetDisplayedCookies(&cookies_model).c_str());
@@ -850,21 +903,26 @@ TEST_F(CookiesTreeModelTest, RemoveSingleCookieNode) {
EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
GetDisplayedFileSystems(&cookies_model));
EXPECT_EQ("quotahost1,quotahost2", GetDisplayedQuotas(&cookies_model));
- EXPECT_EQ(42, cookies_model.GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(43, cookies_model.GetRoot()->GetTotalNodeCount());
}
}
TEST_F(CookiesTreeModelTest, RemoveSingleCookieNodeOf3) {
- CookiesTreeModel cookies_model(mock_browsing_data_cookie_helper_,
- mock_browsing_data_database_helper_,
- mock_browsing_data_local_storage_helper_,
- mock_browsing_data_session_storage_helper_,
- mock_browsing_data_appcache_helper_,
- mock_browsing_data_indexed_db_helper_,
- mock_browsing_data_file_system_helper_,
- mock_browsing_data_quota_helper_,
- mock_browsing_data_server_bound_cert_helper_,
- false);
+ ContainerMap container_map;
+
+ container_map[""] = new LocalDataContainer(
+ "Drive-By-Web", "",
+ mock_browsing_data_cookie_helper_,
+ mock_browsing_data_database_helper_,
+ mock_browsing_data_local_storage_helper_,
+ mock_browsing_data_session_storage_helper_,
+ mock_browsing_data_appcache_helper_,
+ mock_browsing_data_indexed_db_helper_,
+ mock_browsing_data_file_system_helper_,
+ mock_browsing_data_quota_helper_,
+ mock_browsing_data_server_bound_cert_helper_);
+ CookiesTreeModel cookies_model(container_map, false);
+
mock_browsing_data_cookie_helper_->
AddCookieSamples(GURL("http://foo1"), "A=1");
mock_browsing_data_cookie_helper_->
@@ -893,21 +951,24 @@ TEST_F(CookiesTreeModelTest, RemoveSingleCookieNodeOf3) {
SCOPED_TRACE("Initial State 5 cookies, 2 databases, 2 local storages, "
"2 session storages, 2 indexed DBs, 3 filesystems, "
"2 quotas.");
- // 43 because there's the root, then foo1 -> cookies -> a,
- // foo2 -> cookies -> b, foo3 -> cookies -> c,d,e
- // dbhost1 -> database -> db1, dbhost2 -> database -> db2,
- // fshost1 -> filesystem -> http://fshost1:1/,
- // fshost2 -> filesystem -> http://fshost2:1/,
- // fshost3 -> filesystem -> http://fshost3:1/,
+ // 48 because there's the root, then the app, then
+ // foo1 -> cookies -> a,
+ // foo2 -> cookies -> b,
+ // foo3 -> cookies -> c,d,e
+ // dbhost1 -> database -> db1,
+ // dbhost2 -> database -> db2,
// host1 -> localstorage -> http://host1:1/,
+ // -> sessionstorage -> http://host1:1/,
// host2 -> localstorage -> http://host2:2/,
- // host1 -> sessionstorage -> http://host1:1/,
- // host2 -> sessionstorage -> http://host2:2/,
+ // -> sessionstorage -> http://host2:2/,
// idbhost1 -> sessionstorage -> http://idbhost1:1/,
// idbhost2 -> sessionstorage -> http://idbhost2:2/,
+ // fshost1 -> filesystem -> http://fshost1:1/,
+ // fshost2 -> filesystem -> http://fshost2:1/,
+ // fshost3 -> filesystem -> http://fshost3:1/,
// quotahost1 -> quotahost1,
// quotahost2 -> quotahost2.
- EXPECT_EQ(47, cookies_model.GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(48, cookies_model.GetRoot()->GetTotalNodeCount());
EXPECT_STREQ("A,B,C,D,E", GetDisplayedCookies(&cookies_model).c_str());
EXPECT_EQ("db1,db2", GetDisplayedDatabases(&cookies_model));
EXPECT_EQ("http://host1:1/,http://host2:2/",
@@ -920,12 +981,12 @@ TEST_F(CookiesTreeModelTest, RemoveSingleCookieNodeOf3) {
GetDisplayedFileSystems(&cookies_model));
EXPECT_EQ("quotahost1,quotahost2", GetDisplayedQuotas(&cookies_model));
}
- DeleteStoredObjects(cookies_model.GetRoot()->GetChild(2)->GetChild(0)->
- GetChild(1));
+ DeleteStoredObjects(cookies_model.GetRoot()->GetChild(0)->GetChild(2)->
+ GetChild(0)->GetChild(1));
{
SCOPED_TRACE("Middle cookie in third origin removed");
EXPECT_STREQ("A,B,C,E", GetDisplayedCookies(&cookies_model).c_str());
- EXPECT_EQ(46, cookies_model.GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(47, cookies_model.GetRoot()->GetTotalNodeCount());
EXPECT_EQ("db1,db2", GetDisplayedDatabases(&cookies_model));
EXPECT_EQ("http://host1:1/,http://host2:2/",
GetDisplayedLocalStorages(&cookies_model));
@@ -940,16 +1001,21 @@ TEST_F(CookiesTreeModelTest, RemoveSingleCookieNodeOf3) {
}
TEST_F(CookiesTreeModelTest, RemoveSecondOrigin) {
- CookiesTreeModel cookies_model(mock_browsing_data_cookie_helper_,
- mock_browsing_data_database_helper_,
- mock_browsing_data_local_storage_helper_,
- mock_browsing_data_session_storage_helper_,
- mock_browsing_data_appcache_helper_,
- mock_browsing_data_indexed_db_helper_,
- mock_browsing_data_file_system_helper_,
- mock_browsing_data_quota_helper_,
- mock_browsing_data_server_bound_cert_helper_,
- false);
+ ContainerMap container_map;
+
+ container_map[""] = new LocalDataContainer(
+ "Drive-By-Web", "",
+ mock_browsing_data_cookie_helper_,
+ mock_browsing_data_database_helper_,
+ mock_browsing_data_local_storage_helper_,
+ mock_browsing_data_session_storage_helper_,
+ mock_browsing_data_appcache_helper_,
+ mock_browsing_data_indexed_db_helper_,
+ mock_browsing_data_file_system_helper_,
+ mock_browsing_data_quota_helper_,
+ mock_browsing_data_server_bound_cert_helper_);
+ CookiesTreeModel cookies_model(container_map, false);
+
mock_browsing_data_cookie_helper_->
AddCookieSamples(GURL("http://foo1"), "A=1");
mock_browsing_data_cookie_helper_->
@@ -964,31 +1030,36 @@ TEST_F(CookiesTreeModelTest, RemoveSecondOrigin) {
{
SCOPED_TRACE("Initial State 5 cookies");
- // 11 because there's the root, then foo1 -> cookies -> a,
+ // 13 because there's the root, the app, then foo1 -> cookies -> a,
// foo2 -> cookies -> b, foo3 -> cookies -> c,d,e
- EXPECT_EQ(12, cookies_model.GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(13, cookies_model.GetRoot()->GetTotalNodeCount());
EXPECT_STREQ("A,B,C,D,E", GetDisplayedCookies(&cookies_model).c_str());
}
- DeleteStoredObjects(cookies_model.GetRoot()->GetChild(1));
+ DeleteStoredObjects(cookies_model.GetRoot()->GetChild(0)->GetChild(1));
{
SCOPED_TRACE("Second origin removed");
EXPECT_STREQ("A,C,D,E", GetDisplayedCookies(&cookies_model).c_str());
- // Left with root -> foo1 -> cookies -> a, foo3 -> cookies -> c,d,e
- EXPECT_EQ(9, cookies_model.GetRoot()->GetTotalNodeCount());
+ // Left with root -> app -> foo1 -> cookies -> a, foo3 -> cookies -> c,d,e
+ EXPECT_EQ(10, cookies_model.GetRoot()->GetTotalNodeCount());
}
}
TEST_F(CookiesTreeModelTest, OriginOrdering) {
- CookiesTreeModel cookies_model(mock_browsing_data_cookie_helper_,
- mock_browsing_data_database_helper_,
- mock_browsing_data_local_storage_helper_,
- mock_browsing_data_session_storage_helper_,
- mock_browsing_data_appcache_helper_,
- mock_browsing_data_indexed_db_helper_,
- mock_browsing_data_file_system_helper_,
- mock_browsing_data_quota_helper_,
- mock_browsing_data_server_bound_cert_helper_,
- false);
+ ContainerMap container_map;
+
+ container_map[""] = new LocalDataContainer(
+ "Drive-By-Web", "",
+ mock_browsing_data_cookie_helper_,
+ mock_browsing_data_database_helper_,
+ mock_browsing_data_local_storage_helper_,
+ mock_browsing_data_session_storage_helper_,
+ mock_browsing_data_appcache_helper_,
+ mock_browsing_data_indexed_db_helper_,
+ mock_browsing_data_file_system_helper_,
+ mock_browsing_data_quota_helper_,
+ mock_browsing_data_server_bound_cert_helper_);
+ CookiesTreeModel cookies_model(container_map, false);
+
mock_browsing_data_cookie_helper_->
AddCookieSamples(GURL("http://a.foo2.com"), "A=1");
mock_browsing_data_cookie_helper_->
@@ -1010,11 +1081,12 @@ TEST_F(CookiesTreeModelTest, OriginOrdering) {
{
SCOPED_TRACE("Initial State 8 cookies");
- EXPECT_EQ(23, cookies_model.GetRoot()->GetTotalNodeCount());
+ EXPECT_EQ(24, cookies_model.GetRoot()->GetTotalNodeCount());
EXPECT_STREQ("F,E,C,B,A,G,D,H",
GetDisplayedCookies(&cookies_model).c_str());
}
- DeleteStoredObjects(cookies_model.GetRoot()->GetChild(1)); // Delete "E"
+ // Delete "E"
+ DeleteStoredObjects(cookies_model.GetRoot()->GetChild(0)->GetChild(1));
{
EXPECT_STREQ("F,C,B,A,G,D,H", GetDisplayedCookies(&cookies_model).c_str());
}
@@ -1022,16 +1094,23 @@ TEST_F(CookiesTreeModelTest, OriginOrdering) {
TEST_F(CookiesTreeModelTest, ContentSettings) {
GURL host("http://example.com/");
- CookiesTreeModel cookies_model(mock_browsing_data_cookie_helper_,
- mock_browsing_data_database_helper_,
- mock_browsing_data_local_storage_helper_,
- mock_browsing_data_session_storage_helper_,
- mock_browsing_data_appcache_helper_,
- mock_browsing_data_indexed_db_helper_,
- mock_browsing_data_file_system_helper_,
- mock_browsing_data_quota_helper_,
- mock_browsing_data_server_bound_cert_helper_,
- false);
+ std::string name = "Drive-By-Web";
+ std::string browser_id = "";
+ ContainerMap container_map;
+
+ container_map[browser_id] = new LocalDataContainer(
+ name, browser_id,
+ mock_browsing_data_cookie_helper_,
+ mock_browsing_data_database_helper_,
+ mock_browsing_data_local_storage_helper_,
+ mock_browsing_data_session_storage_helper_,
+ mock_browsing_data_appcache_helper_,
+ mock_browsing_data_indexed_db_helper_,
+ mock_browsing_data_file_system_helper_,
+ mock_browsing_data_quota_helper_,
+ mock_browsing_data_server_bound_cert_helper_);
+ CookiesTreeModel cookies_model(container_map, false);
+
mock_browsing_data_cookie_helper_->AddCookieSamples(host, "A=1");
mock_browsing_data_cookie_helper_->Notify();
@@ -1044,7 +1123,8 @@ TEST_F(CookiesTreeModelTest, ContentSettings) {
CookieTreeRootNode* root =
static_cast<CookieTreeRootNode*>(cookies_model.GetRoot());
- CookieTreeOriginNode* origin = root->GetOrCreateOriginNode(host);
+ CookieTreeAppNode* app_node = root->GetOrCreateAppNode(name, browser_id);
+ CookieTreeOriginNode* origin = app_node->GetOrCreateOriginNode(host);
EXPECT_EQ(1, origin->child_count());
EXPECT_TRUE(origin->CanCreateContentException());
@@ -1069,25 +1149,82 @@ TEST_F(CookiesTreeModelTest, ContentSettings) {
EXPECT_TRUE(cookie_settings->IsCookieSessionOnly(host));
}
+TEST_F(CookiesTreeModelTest, RemoveAppCookies) {
+ scoped_ptr<CookiesTreeModel> cookies_model(
+ CreateCookiesTreeModelWithInitialSample(true));
+
+ // Since we've added an app to the tree model, we expect two children of the
+ // root node. The app should have 3 children for the the 3 domains.
+ EXPECT_EQ(2, cookies_model->GetRoot()->child_count());
+ EXPECT_EQ(3, cookies_model->GetRoot()->GetChild(1)->child_count());
+
+ DeleteStoredObjects(cookies_model->GetRoot()->GetChild(1)->GetChild(1));
+ {
+ SCOPED_TRACE("Second origin for app removed");
+ EXPECT_STREQ("A,B,C,Z,X", GetDisplayedCookies(cookies_model.get()).c_str());
+ EXPECT_EQ(59, cookies_model->GetRoot()->GetTotalNodeCount());
+ }
+}
+
TEST_F(CookiesTreeModelTest, FileSystemFilter) {
scoped_ptr<CookiesTreeModel> cookies_model(
- CreateCookiesTreeModelWithInitialSample());
+ CreateCookiesTreeModelWithInitialSample(false));
- cookies_model->UpdateSearchResults(std::wstring(L"fshost1"));
+ cookies_model->UpdateSearchResults(string16(ASCIIToUTF16("fshost1")));
EXPECT_EQ("http://fshost1:1/",
GetDisplayedFileSystems(cookies_model.get()));
- cookies_model->UpdateSearchResults(std::wstring(L"fshost2"));
+ cookies_model->UpdateSearchResults(string16(ASCIIToUTF16("fshost2")));
EXPECT_EQ("http://fshost2:2/",
GetDisplayedFileSystems(cookies_model.get()));
- cookies_model->UpdateSearchResults(std::wstring(L"fshost3"));
+ cookies_model->UpdateSearchResults(string16(ASCIIToUTF16("fshost3")));
EXPECT_EQ("http://fshost3:3/",
GetDisplayedFileSystems(cookies_model.get()));
- cookies_model->UpdateSearchResults(std::wstring());
+ cookies_model->UpdateSearchResults(string16());
EXPECT_EQ("http://fshost1:1/,http://fshost2:2/,http://fshost3:3/",
GetDisplayedFileSystems(cookies_model.get()));
}
+TEST_F(CookiesTreeModelTest, String16Filter) {
+ ContainerMap container_map;
+
+ container_map[""] = new LocalDataContainer(
+ "Drive-By-Web", "",
+ mock_browsing_data_cookie_helper_,
+ mock_browsing_data_database_helper_,
+ mock_browsing_data_local_storage_helper_,
+ mock_browsing_data_session_storage_helper_,
+ mock_browsing_data_appcache_helper_,
+ mock_browsing_data_indexed_db_helper_,
+ mock_browsing_data_file_system_helper_,
+ mock_browsing_data_quota_helper_,
+ mock_browsing_data_server_bound_cert_helper_);
+ CookiesTreeModel cookies_model(container_map, false);
+
+ mock_browsing_data_cookie_helper_->
+ AddCookieSamples(GURL("http://123.com"), "A=1");
+ mock_browsing_data_cookie_helper_->
+ AddCookieSamples(GURL("http://foo1.com"), "B=1");
+ mock_browsing_data_cookie_helper_->
+ AddCookieSamples(GURL("http://foo2.com"), "C=1");
+ mock_browsing_data_cookie_helper_->
+ AddCookieSamples(GURL("http://foo3.com"), "D=1");
+ mock_browsing_data_cookie_helper_->Notify();
+ EXPECT_EQ("A,B,C,D", GetDisplayedCookies(&cookies_model));
+
+ cookies_model.UpdateSearchResults(string16(ASCIIToUTF16("foo")));
+ EXPECT_EQ("B,C,D", GetDisplayedCookies(&cookies_model));
+
+ cookies_model.UpdateSearchResults(string16(ASCIIToUTF16("2")));
+ EXPECT_EQ("A,C", GetDisplayedCookies(&cookies_model));
+
+ cookies_model.UpdateSearchResults(string16(ASCIIToUTF16("foo3")));
+ EXPECT_EQ("D", GetDisplayedCookies(&cookies_model));
+
+ cookies_model.UpdateSearchResults(string16());
+ EXPECT_EQ("A,B,C,D", GetDisplayedCookies(&cookies_model));
+}
+
} // namespace
« chrome/browser/cookies_tree_model.cc ('K') | « chrome/browser/cookies_tree_model.cc ('k') | chrome/browser/local_data_container.h » ('j') | chrome/browser/local_data_container.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698