summaryrefslogtreecommitdiffstats
path: root/net/cookies/cookie_store_unittest.h
diff options
context:
space:
mode:
authortyoshino@chromium.org <tyoshino@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-05-22 12:43:40 +0000
committertyoshino@chromium.org <tyoshino@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-05-22 12:43:40 +0000
commitf6c58823df3449679e4f266ba0c612f11e9594cb (patch)
treef0ebb507382e651ecb5f7f1bb22d974ffd89e69e /net/cookies/cookie_store_unittest.h
parent05e78a90a52f8af3b66b4991f67684d008abc6de (diff)
downloadchromium_src-f6c58823df3449679e4f266ba0c612f11e9594cb.zip
chromium_src-f6c58823df3449679e4f266ba0c612f11e9594cb.tar.gz
chromium_src-f6c58823df3449679e4f266ba0c612f11e9594cb.tar.bz2
Rename test callbacks for CookieStore to represent the argument type.
These callbacks are not tightly connected to each method but are made to save a result value of the specified type. Rename them to represent the type. Actually, SetCookieCallback is used for testing Delete method which passes a bool result to the done callback. Review URL: https://chromiumcodereview.appspot.com/15380003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@201497 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/cookies/cookie_store_unittest.h')
-rw-r--r--net/cookies/cookie_store_unittest.h73
1 files changed, 39 insertions, 34 deletions
diff --git a/net/cookies/cookie_store_unittest.h b/net/cookies/cookie_store_unittest.h
index f209ab7..bb0d42e 100644
--- a/net/cookies/cookie_store_unittest.h
+++ b/net/cookies/cookie_store_unittest.h
@@ -90,26 +90,27 @@ class CookieStoreTest : public testing::Test {
CookieOptions options;
if (!CookieStoreTestTraits::supports_http_only)
options.set_include_httponly();
- GetCookieStringCallback callback;
+ StringResultCookieCallback callback;
cs->GetCookiesWithOptionsAsync(
url, options,
- base::Bind(&GetCookieStringCallback::Run, base::Unretained(&callback)));
+ base::Bind(&StringResultCookieCallback::Run,
+ base::Unretained(&callback)));
RunFor(kTimeout);
EXPECT_TRUE(callback.did_run());
- return callback.cookie();
+ return callback.result();
}
std::string GetCookiesWithOptions(CookieStore* cs,
const GURL& url,
const CookieOptions& options) {
DCHECK(cs);
- GetCookieStringCallback callback;
+ StringResultCookieCallback callback;
cs->GetCookiesWithOptionsAsync(
- url, options, base::Bind(&GetCookieStringCallback::Run,
+ url, options, base::Bind(&StringResultCookieCallback::Run,
base::Unretained(&callback)));
RunFor(kTimeout);
EXPECT_TRUE(callback.did_run());
- return callback.cookie();
+ return callback.result();
}
bool SetCookieWithOptions(CookieStore* cs,
@@ -117,10 +118,11 @@ class CookieStoreTest : public testing::Test {
const std::string& cookie_line,
const CookieOptions& options) {
DCHECK(cs);
- SetCookieCallback callback;
+ BoolResultCookieCallback callback;
cs->SetCookieWithOptionsAsync(
url, cookie_line, options,
- base::Bind(&SetCookieCallback::Run, base::Unretained(&callback)));
+ base::Bind(&BoolResultCookieCallback::Run,
+ base::Unretained(&callback)));
RunFor(kTimeout);
EXPECT_TRUE(callback.did_run());
return callback.result();
@@ -150,10 +152,10 @@ class CookieStoreTest : public testing::Test {
const GURL& url,
const std::string& cookie_name) {
DCHECK(cs);
- DeleteCookieCallback callback;
+ NoResultCookieCallback callback;
cs->DeleteCookieAsync(
url, cookie_name,
- base::Bind(&DeleteCookieCallback::Run, base::Unretained(&callback)));
+ base::Bind(&NoResultCookieCallback::Run, base::Unretained(&callback)));
RunFor(kTimeout);
EXPECT_TRUE(callback.did_run());
}
@@ -162,23 +164,23 @@ class CookieStoreTest : public testing::Test {
const base::Time& delete_begin,
const base::Time& delete_end) {
DCHECK(cs);
- DeleteCallback callback;
+ IntResultCookieCallback callback;
cs->DeleteAllCreatedBetweenAsync(
delete_begin, delete_end,
- base::Bind(&DeleteCallback::Run, base::Unretained(&callback)));
+ base::Bind(&IntResultCookieCallback::Run, base::Unretained(&callback)));
RunFor(kTimeout);
EXPECT_TRUE(callback.did_run());
- return callback.num_deleted();
+ return callback.result();
}
int DeleteSessionCookies(CookieStore* cs) {
DCHECK(cs);
- DeleteCallback callback;
+ IntResultCookieCallback callback;
cs->DeleteSessionCookiesAsync(
- base::Bind(&DeleteCallback::Run, base::Unretained(&callback)));
+ base::Bind(&IntResultCookieCallback::Run, base::Unretained(&callback)));
RunFor(kTimeout);
EXPECT_TRUE(callback.did_run());
- return callback.num_deleted();
+ return callback.result();
}
void RunFor(int ms) {
@@ -942,46 +944,49 @@ class MultiThreadedCookieStoreTest :
void GetCookiesTask(CookieStore* cs,
const GURL& url,
- GetCookieStringCallback* callback) {
+ StringResultCookieCallback* callback) {
CookieOptions options;
if (!CookieStoreTestTraits::supports_http_only)
options.set_include_httponly();
cs->GetCookiesWithOptionsAsync(
url, options,
- base::Bind(&GetCookieStringCallback::Run, base::Unretained(callback)));
+ base::Bind(&StringResultCookieCallback::Run,
+ base::Unretained(callback)));
}
void GetCookiesWithOptionsTask(CookieStore* cs,
const GURL& url,
const CookieOptions& options,
- GetCookieStringCallback* callback) {
+ StringResultCookieCallback* callback) {
cs->GetCookiesWithOptionsAsync(
url, options,
- base::Bind(&GetCookieStringCallback::Run, base::Unretained(callback)));
+ base::Bind(&StringResultCookieCallback::Run,
+ base::Unretained(callback)));
}
void SetCookieWithOptionsTask(CookieStore* cs,
const GURL& url,
const std::string& cookie_line,
const CookieOptions& options,
- SetCookieCallback* callback) {
+ BoolResultCookieCallback* callback) {
cs->SetCookieWithOptionsAsync(
url, cookie_line, options,
- base::Bind(&SetCookieCallback::Run, base::Unretained(callback)));
+ base::Bind(&BoolResultCookieCallback::Run, base::Unretained(callback)));
}
void DeleteCookieTask(CookieStore* cs,
const GURL& url,
const std::string& cookie_name,
- DeleteCookieCallback* callback) {
+ NoResultCookieCallback* callback) {
cs->DeleteCookieAsync(
url, cookie_name,
- base::Bind(&DeleteCookieCallback::Run, base::Unretained(callback)));
+ base::Bind(&NoResultCookieCallback::Run, base::Unretained(callback)));
}
- void DeleteSessionCookiesTask(CookieStore* cs, DeleteCallback* callback) {
+ void DeleteSessionCookiesTask(CookieStore* cs,
+ IntResultCookieCallback* callback) {
cs->DeleteSessionCookiesAsync(
- base::Bind(&DeleteCallback::Run, base::Unretained(callback)));
+ base::Bind(&IntResultCookieCallback::Run, base::Unretained(callback)));
}
protected:
@@ -1004,14 +1009,14 @@ TYPED_TEST_P(MultiThreadedCookieStoreTest, ThreadCheckGetCookies) {
scoped_refptr<CookieStore> cs(this->GetCookieStore());
EXPECT_TRUE(this->SetCookie(cs, this->url_google_, "A=B"));
this->MatchCookieLines("A=B", this->GetCookies(cs, this->url_google_));
- GetCookieStringCallback callback(&this->other_thread_);
+ StringResultCookieCallback callback(&this->other_thread_);
base::Closure task = base::Bind(
&net::MultiThreadedCookieStoreTest<TypeParam>::GetCookiesTask,
base::Unretained(this),
cs, this->url_google_, &callback);
this->RunOnOtherThread(task);
EXPECT_TRUE(callback.did_run());
- EXPECT_EQ("A=B", callback.cookie());
+ EXPECT_EQ("A=B", callback.result());
}
TYPED_TEST_P(MultiThreadedCookieStoreTest, ThreadCheckGetCookiesWithOptions) {
@@ -1022,14 +1027,14 @@ TYPED_TEST_P(MultiThreadedCookieStoreTest, ThreadCheckGetCookiesWithOptions) {
EXPECT_TRUE(this->SetCookie(cs, this->url_google_, "A=B"));
this->MatchCookieLines("A=B",
this->GetCookiesWithOptions(cs, this->url_google_, options));
- GetCookieStringCallback callback(&this->other_thread_);
+ StringResultCookieCallback callback(&this->other_thread_);
base::Closure task = base::Bind(
&net::MultiThreadedCookieStoreTest<TypeParam>::GetCookiesWithOptionsTask,
base::Unretained(this),
cs, this->url_google_, options, &callback);
this->RunOnOtherThread(task);
EXPECT_TRUE(callback.did_run());
- EXPECT_EQ("A=B", callback.cookie());
+ EXPECT_EQ("A=B", callback.result());
}
TYPED_TEST_P(MultiThreadedCookieStoreTest, ThreadCheckSetCookieWithOptions) {
@@ -1039,7 +1044,7 @@ TYPED_TEST_P(MultiThreadedCookieStoreTest, ThreadCheckSetCookieWithOptions) {
options.set_include_httponly();
EXPECT_TRUE(this->SetCookieWithOptions(cs, this->url_google_, "A=B",
options));
- SetCookieCallback callback(&this->other_thread_);
+ BoolResultCookieCallback callback(&this->other_thread_);
base::Closure task = base::Bind(
&net::MultiThreadedCookieStoreTest<TypeParam>::SetCookieWithOptionsTask,
base::Unretained(this),
@@ -1059,7 +1064,7 @@ TYPED_TEST_P(MultiThreadedCookieStoreTest, ThreadCheckDeleteCookie) {
this->DeleteCookie(cs, this->url_google_, "A");
EXPECT_TRUE(this->SetCookieWithOptions(cs, this->url_google_, "A=B",
options));
- DeleteCookieCallback callback(&this->other_thread_);
+ NoResultCookieCallback callback(&this->other_thread_);
base::Closure task = base::Bind(
&net::MultiThreadedCookieStoreTest<TypeParam>::DeleteCookieTask,
base::Unretained(this),
@@ -1081,14 +1086,14 @@ TYPED_TEST_P(MultiThreadedCookieStoreTest, ThreadCheckDeleteSessionCookies) {
EXPECT_EQ(0, this->DeleteSessionCookies(cs));
EXPECT_TRUE(this->SetCookieWithOptions(cs, this->url_google_,
"A=B", options));
- DeleteCallback callback(&this->other_thread_);
+ IntResultCookieCallback callback(&this->other_thread_);
base::Closure task = base::Bind(
&net::MultiThreadedCookieStoreTest<TypeParam>::DeleteSessionCookiesTask,
base::Unretained(this),
cs, &callback);
this->RunOnOtherThread(task);
EXPECT_TRUE(callback.did_run());
- EXPECT_EQ(1, callback.num_deleted());
+ EXPECT_EQ(1, callback.result());
}
REGISTER_TYPED_TEST_CASE_P(MultiThreadedCookieStoreTest,