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

Side by Side Diff: chrome/browser/net/sqlite_persistent_cookie_store_unittest.cc

Issue 5430004: Refactored cookies persistent store clean-up on shutdown. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Lock protected the clear on exit flag. Created 10 years 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
« no previous file with comments | « chrome/browser/net/sqlite_persistent_cookie_store.cc ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "base/file_util.h"
6 #include "base/message_loop.h"
7 #include "base/ref_counted.h"
8 #include "base/scoped_temp_dir.h"
9 #include "base/stl_util-inl.h"
10 #include "base/time.h"
11 #include "chrome/browser/browser_thread.h"
12 #include "chrome/browser/net/sqlite_persistent_cookie_store.h"
13 #include "chrome/common/chrome_constants.h"
14 #include "chrome/test/thread_test_helper.h"
15 #include "googleurl/src/gurl.h"
16 #include "testing/gtest/include/gtest/gtest.h"
17
18 class SQLitePersistentCookieStoreTest : public testing::Test {
19 public:
20 SQLitePersistentCookieStoreTest()
21 : ui_thread_(BrowserThread::UI),
22 db_thread_(BrowserThread::DB) {
23 }
24
25 protected:
26 virtual void SetUp() {
27 ui_thread_.Start();
28 db_thread_.Start();
29 ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
30 store_ = new SQLitePersistentCookieStore(
31 temp_dir_.path().Append(chrome::kCookieFilename));
32 std::vector<net::CookieMonster::CanonicalCookie*> cookies;
33 ASSERT_TRUE(store_->Load(&cookies));
34 ASSERT_TRUE(0 == cookies.size());
35 // Make sure the store gets written at least once.
36 store_->AddCookie(
37 net::CookieMonster::CanonicalCookie("A", "B", "http://foo.bar", "/",
38 false, false,
39 base::Time::Now(),
40 base::Time::Now(),
41 true, base::Time::Now()));
42 }
43
44 BrowserThread ui_thread_;
45 BrowserThread db_thread_;
46 scoped_refptr<SQLitePersistentCookieStore> store_;
47 ScopedTempDir temp_dir_;
48 };
49
50 TEST_F(SQLitePersistentCookieStoreTest, KeepOnDestruction) {
51 store_->SetClearLocalStateOnExit(false);
52 store_ = NULL;
53 // Make sure we wait until the destructor has run.
54 scoped_refptr<ThreadTestHelper> helper(
55 new ThreadTestHelper(BrowserThread::DB));
56 ASSERT_TRUE(helper->Run());
57
58 ASSERT_TRUE(file_util::PathExists(
59 temp_dir_.path().Append(chrome::kCookieFilename)));
60 ASSERT_TRUE(file_util::Delete(
61 temp_dir_.path().Append(chrome::kCookieFilename), false));
62 }
63
64 TEST_F(SQLitePersistentCookieStoreTest, RemoveOnDestruction) {
65 store_->SetClearLocalStateOnExit(true);
66 // Replace the store effectively destroying the current one and forcing it
67 // to write it's data to disk. Then we can see if after loading it again it
68 // is still there.
69 store_ = NULL;
70 // Make sure we wait until the destructor has run.
71 scoped_refptr<ThreadTestHelper> helper(
72 new ThreadTestHelper(BrowserThread::DB));
73 ASSERT_TRUE(helper->Run());
74
75 ASSERT_FALSE(file_util::PathExists(
76 temp_dir_.path().Append(chrome::kCookieFilename)));
77 }
78
79 // Test if data is stored as expected in the SQLite database.
80 TEST_F(SQLitePersistentCookieStoreTest, TestPersistance) {
81 std::vector<net::CookieMonster::CanonicalCookie*> cookies;
82 // Replace the store effectively destroying the current one and forcing it
83 // to write it's data to disk. Then we can see if after loading it again it
84 // is still there.
85 store_ = NULL;
86 scoped_refptr<ThreadTestHelper> helper(
87 new ThreadTestHelper(BrowserThread::DB));
88 // Make sure we wait until the destructor has run.
89 ASSERT_TRUE(helper->Run());
90 store_ = new SQLitePersistentCookieStore(
91 temp_dir_.path().Append(chrome::kCookieFilename));
92
93 // Reload and test for persistence
94 ASSERT_TRUE(store_->Load(&cookies));
95 ASSERT_EQ(1U, cookies.size());
96 ASSERT_STREQ("http://foo.bar", cookies[0]->Domain().c_str());
97 ASSERT_STREQ("A", cookies[0]->Name().c_str());
98 ASSERT_STREQ("B", cookies[0]->Value().c_str());
99
100 // Now delete the cookie and check persistence again.
101 store_->DeleteCookie(*cookies[0]);
102 store_ = NULL;
103 // Make sure we wait until the destructor has run.
104 ASSERT_TRUE(helper->Run());
105 STLDeleteContainerPointers(cookies.begin(), cookies.end());
106 cookies.clear();
107 store_ = new SQLitePersistentCookieStore(
108 temp_dir_.path().Append(chrome::kCookieFilename));
109
110 // Reload and check if the cookie has been removed.
111 ASSERT_TRUE(store_->Load(&cookies));
112 ASSERT_EQ(0U, cookies.size());
113 }
OLDNEW
« no previous file with comments | « chrome/browser/net/sqlite_persistent_cookie_store.cc ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698