| Index: net/cookies/cookie_monster.cc
|
| diff --git a/net/cookies/cookie_monster.cc b/net/cookies/cookie_monster.cc
|
| index 452ce68216d898f6c301865fb50f005673d6466a..7c2a14c6d60e96c3c41e6d8f4b1e0cf9e5fd76c5 100644
|
| --- a/net/cookies/cookie_monster.cc
|
| +++ b/net/cookies/cookie_monster.cc
|
| @@ -97,18 +97,18 @@ namespace net {
|
|
|
| // See comments at declaration of these variables in cookie_monster.h
|
| // for details.
|
| -const size_t CookieMonster::kDomainMaxCookies = 180;
|
| -const size_t CookieMonster::kDomainPurgeCookies = 30;
|
| -const size_t CookieMonster::kMaxCookies = 3300;
|
| -const size_t CookieMonster::kPurgeCookies = 300;
|
| +const size_t CookieMonster::kDomainMaxCookies = 180;
|
| +const size_t CookieMonster::kDomainPurgeCookies = 30;
|
| +const size_t CookieMonster::kMaxCookies = 3300;
|
| +const size_t CookieMonster::kPurgeCookies = 300;
|
|
|
| -const size_t CookieMonster::kDomainCookiesQuotaLow = 30;
|
| +const size_t CookieMonster::kDomainCookiesQuotaLow = 30;
|
| const size_t CookieMonster::kDomainCookiesQuotaMedium = 50;
|
| -const size_t CookieMonster::kDomainCookiesQuotaHigh =
|
| - kDomainMaxCookies - kDomainPurgeCookies
|
| - - kDomainCookiesQuotaLow - kDomainCookiesQuotaMedium;
|
| +const size_t CookieMonster::kDomainCookiesQuotaHigh =
|
| + kDomainMaxCookies - kDomainPurgeCookies - kDomainCookiesQuotaLow -
|
| + kDomainCookiesQuotaMedium;
|
|
|
| -const int CookieMonster::kSafeFromGlobalPurgeDays = 30;
|
| +const int CookieMonster::kSafeFromGlobalPurgeDays = 30;
|
|
|
| namespace {
|
|
|
| @@ -177,8 +177,7 @@ struct CookieSignature {
|
| CookieSignature(const std::string& name,
|
| const std::string& domain,
|
| const std::string& path)
|
| - : name(name), domain(domain), path(path) {
|
| - }
|
| + : name(name), domain(domain), path(path) {}
|
|
|
| // To be a key for a map this class needs to be assignable, copyable,
|
| // and have an operator<. The default assignment operator
|
| @@ -206,10 +205,9 @@ struct CookieSignature {
|
| // sorts the first |num_sort| + 1 elements by LastAccessDate().
|
| // The + 1 element exists so for any interval of length <= |num_sort| starting
|
| // from |cookies_its_begin|, a LastAccessDate() bound can be found.
|
| -void SortLeastRecentlyAccessed(
|
| - CookieMonster::CookieItVector::iterator it_begin,
|
| - CookieMonster::CookieItVector::iterator it_end,
|
| - size_t num_sort) {
|
| +void SortLeastRecentlyAccessed(CookieMonster::CookieItVector::iterator it_begin,
|
| + CookieMonster::CookieItVector::iterator it_end,
|
| + size_t num_sort) {
|
| DCHECK_LT(static_cast<int>(num_sort), it_end - it_begin);
|
| std::partial_sort(it_begin, it_begin + num_sort + 1, it_end, LRACookieSorter);
|
| }
|
| @@ -218,7 +216,7 @@ void SortLeastRecentlyAccessed(
|
| struct CookiePriorityEqualsTo
|
| : std::unary_function<const CookieMonster::CookieMap::iterator, bool> {
|
| explicit CookiePriorityEqualsTo(CookiePriority priority)
|
| - : priority_(priority) {}
|
| + : priority_(priority) {}
|
|
|
| bool operator()(const CookieMonster::CookieMap::iterator it) const {
|
| return it->second->Priority() == priority_;
|
| @@ -238,8 +236,8 @@ CookieMonster::CookieItVector::iterator PartitionCookieByPriority(
|
| return std::partition(it_begin, it_end, CookiePriorityEqualsTo(priority));
|
| }
|
|
|
| -bool LowerBoundAccessDateComparator(
|
| - const CookieMonster::CookieMap::iterator it, const Time& access_date) {
|
| +bool LowerBoundAccessDateComparator(const CookieMonster::CookieMap::iterator it,
|
| + const Time& access_date) {
|
| return it->second->LastAccessDate() < access_date;
|
| }
|
|
|
| @@ -263,33 +261,32 @@ typedef struct ChangeCausePair_struct {
|
| bool notify;
|
| } ChangeCausePair;
|
| ChangeCausePair ChangeCauseMapping[] = {
|
| - // DELETE_COOKIE_EXPLICIT
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT, true },
|
| - // DELETE_COOKIE_OVERWRITE
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_OVERWRITE, true },
|
| - // DELETE_COOKIE_EXPIRED
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EXPIRED, true },
|
| - // DELETE_COOKIE_EVICTED
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true },
|
| - // DELETE_COOKIE_DUPLICATE_IN_BACKING_STORE
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT, false },
|
| - // DELETE_COOKIE_DONT_RECORD
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT, false },
|
| - // DELETE_COOKIE_EVICTED_DOMAIN
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true },
|
| - // DELETE_COOKIE_EVICTED_GLOBAL
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true },
|
| - // DELETE_COOKIE_EVICTED_DOMAIN_PRE_SAFE
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true },
|
| - // DELETE_COOKIE_EVICTED_DOMAIN_POST_SAFE
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true },
|
| - // DELETE_COOKIE_EXPIRED_OVERWRITE
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EXPIRED_OVERWRITE, true },
|
| - // DELETE_COOKIE_CONTROL_CHAR
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true},
|
| - // DELETE_COOKIE_LAST_ENTRY
|
| - { CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT, false }
|
| -};
|
| + // DELETE_COOKIE_EXPLICIT
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT, true},
|
| + // DELETE_COOKIE_OVERWRITE
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_OVERWRITE, true},
|
| + // DELETE_COOKIE_EXPIRED
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EXPIRED, true},
|
| + // DELETE_COOKIE_EVICTED
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true},
|
| + // DELETE_COOKIE_DUPLICATE_IN_BACKING_STORE
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT, false},
|
| + // DELETE_COOKIE_DONT_RECORD
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT, false},
|
| + // DELETE_COOKIE_EVICTED_DOMAIN
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true},
|
| + // DELETE_COOKIE_EVICTED_GLOBAL
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true},
|
| + // DELETE_COOKIE_EVICTED_DOMAIN_PRE_SAFE
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true},
|
| + // DELETE_COOKIE_EVICTED_DOMAIN_POST_SAFE
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true},
|
| + // DELETE_COOKIE_EXPIRED_OVERWRITE
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EXPIRED_OVERWRITE, true},
|
| + // DELETE_COOKIE_CONTROL_CHAR
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EVICTED, true},
|
| + // DELETE_COOKIE_LAST_ENTRY
|
| + {CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT, false}};
|
|
|
| std::string BuildCookieLine(const CanonicalCookieVector& cookies) {
|
| std::string cookie_line;
|
| @@ -347,7 +344,6 @@ CookieMonster::CookieMonster(PersistentCookieStore* store,
|
| SetDefaultCookieableSchemes();
|
| }
|
|
|
| -
|
| // Task classes for queueing the coming request.
|
|
|
| class CookieMonster::CookieMonsterTask
|
| @@ -367,9 +363,7 @@ class CookieMonster::CookieMonsterTask
|
| // instance until the callback completes.
|
| void InvokeCallback(base::Closure callback);
|
|
|
| - CookieMonster* cookie_monster() {
|
| - return cookie_monster_;
|
| - }
|
| + CookieMonster* cookie_monster() { return cookie_monster_; }
|
|
|
| private:
|
| friend class base::RefCountedThreadSafe<CookieMonsterTask>;
|
| @@ -386,7 +380,8 @@ CookieMonster::CookieMonsterTask::CookieMonsterTask(
|
| thread_(base::MessageLoopProxy::current()) {
|
| }
|
|
|
| -CookieMonster::CookieMonsterTask::~CookieMonsterTask() {}
|
| +CookieMonster::CookieMonsterTask::~CookieMonsterTask() {
|
| +}
|
|
|
| // Unfortunately, one cannot re-bind a Callback with parameters into a closure.
|
| // Therefore, the closure passed to InvokeCallback is a clumsy binding of
|
| @@ -402,8 +397,8 @@ void CookieMonster::CookieMonsterTask::InvokeCallback(base::Closure callback) {
|
| if (thread_->BelongsToCurrentThread()) {
|
| callback.Run();
|
| } else {
|
| - thread_->PostTask(FROM_HERE, base::Bind(
|
| - &CookieMonsterTask::InvokeCallback, this, callback));
|
| + thread_->PostTask(FROM_HERE, base::Bind(&CookieMonsterTask::InvokeCallback,
|
| + this, callback));
|
| }
|
| }
|
|
|
| @@ -431,8 +426,7 @@ class CookieMonster::SetCookieWithDetailsTask : public CookieMonsterTask {
|
| secure_(secure),
|
| http_only_(http_only),
|
| priority_(priority),
|
| - callback_(callback) {
|
| - }
|
| + callback_(callback) {}
|
|
|
| // CookieMonsterTask:
|
| void Run() override;
|
| @@ -456,9 +450,9 @@ class CookieMonster::SetCookieWithDetailsTask : public CookieMonsterTask {
|
| };
|
|
|
| void CookieMonster::SetCookieWithDetailsTask::Run() {
|
| - bool success = this->cookie_monster()->
|
| - SetCookieWithDetails(url_, name_, value_, domain_, path_,
|
| - expiration_time_, secure_, http_only_, priority_);
|
| + bool success = this->cookie_monster()->SetCookieWithDetails(
|
| + url_, name_, value_, domain_, path_, expiration_time_, secure_,
|
| + http_only_, priority_);
|
| if (!callback_.is_null()) {
|
| this->InvokeCallback(base::Bind(&SetCookiesCallback::Run,
|
| base::Unretained(&callback_), success));
|
| @@ -470,9 +464,7 @@ class CookieMonster::GetAllCookiesTask : public CookieMonsterTask {
|
| public:
|
| GetAllCookiesTask(CookieMonster* cookie_monster,
|
| const GetCookieListCallback& callback)
|
| - : CookieMonsterTask(cookie_monster),
|
| - callback_(callback) {
|
| - }
|
| + : CookieMonsterTask(cookie_monster), callback_(callback) {}
|
|
|
| // CookieMonsterTask
|
| void Run() override;
|
| @@ -491,23 +483,21 @@ void CookieMonster::GetAllCookiesTask::Run() {
|
| CookieList cookies = this->cookie_monster()->GetAllCookies();
|
| this->InvokeCallback(base::Bind(&GetCookieListCallback::Run,
|
| base::Unretained(&callback_), cookies));
|
| - }
|
| + }
|
| }
|
|
|
| // Task class for GetAllCookiesForURLWithOptions call.
|
| class CookieMonster::GetAllCookiesForURLWithOptionsTask
|
| : public CookieMonsterTask {
|
| public:
|
| - GetAllCookiesForURLWithOptionsTask(
|
| - CookieMonster* cookie_monster,
|
| - const GURL& url,
|
| - const CookieOptions& options,
|
| - const GetCookieListCallback& callback)
|
| + GetAllCookiesForURLWithOptionsTask(CookieMonster* cookie_monster,
|
| + const GURL& url,
|
| + const CookieOptions& options,
|
| + const GetCookieListCallback& callback)
|
| : CookieMonsterTask(cookie_monster),
|
| url_(url),
|
| options_(options),
|
| - callback_(callback) {
|
| - }
|
| + callback_(callback) {}
|
|
|
| // CookieMonsterTask:
|
| void Run() override;
|
| @@ -525,18 +515,20 @@ class CookieMonster::GetAllCookiesForURLWithOptionsTask
|
|
|
| void CookieMonster::GetAllCookiesForURLWithOptionsTask::Run() {
|
| if (!callback_.is_null()) {
|
| - CookieList cookies = this->cookie_monster()->
|
| - GetAllCookiesForURLWithOptions(url_, options_);
|
| + CookieList cookies =
|
| + this->cookie_monster()->GetAllCookiesForURLWithOptions(url_, options_);
|
| this->InvokeCallback(base::Bind(&GetCookieListCallback::Run,
|
| base::Unretained(&callback_), cookies));
|
| }
|
| }
|
|
|
| -template <typename Result> struct CallbackType {
|
| +template <typename Result>
|
| +struct CallbackType {
|
| typedef base::Callback<void(Result)> Type;
|
| };
|
|
|
| -template <> struct CallbackType<void> {
|
| +template <>
|
| +struct CallbackType<void> {
|
| typedef base::Closure Type;
|
| };
|
|
|
| @@ -546,9 +538,7 @@ class CookieMonster::DeleteTask : public CookieMonsterTask {
|
| public:
|
| DeleteTask(CookieMonster* cookie_monster,
|
| const typename CallbackType<Result>::Type& callback)
|
| - : CookieMonsterTask(cookie_monster),
|
| - callback_(callback) {
|
| - }
|
| + : CookieMonsterTask(cookie_monster), callback_(callback) {}
|
|
|
| // CookieMonsterTask:
|
| virtual void Run() override;
|
| @@ -588,9 +578,8 @@ base::Closure CookieMonster::DeleteTask<void>::RunDeleteTaskAndBindCallback() {
|
|
|
| template <typename Result>
|
| void CookieMonster::DeleteTask<Result>::Run() {
|
| - this->cookie_monster()->FlushStore(
|
| - base::Bind(&DeleteTask<Result>::FlushDone, this,
|
| - RunDeleteTaskAndBindCallback()));
|
| + this->cookie_monster()->FlushStore(base::Bind(
|
| + &DeleteTask<Result>::FlushDone, this, RunDeleteTaskAndBindCallback()));
|
| }
|
|
|
| template <typename Result>
|
| @@ -604,10 +593,8 @@ void CookieMonster::DeleteTask<Result>::FlushDone(
|
| // Task class for DeleteAll call.
|
| class CookieMonster::DeleteAllTask : public DeleteTask<int> {
|
| public:
|
| - DeleteAllTask(CookieMonster* cookie_monster,
|
| - const DeleteCallback& callback)
|
| - : DeleteTask<int>(cookie_monster, callback) {
|
| - }
|
| + DeleteAllTask(CookieMonster* cookie_monster, const DeleteCallback& callback)
|
| + : DeleteTask<int>(cookie_monster, callback) {}
|
|
|
| // DeleteTask:
|
| int RunDeleteTask() override;
|
| @@ -632,8 +619,7 @@ class CookieMonster::DeleteAllCreatedBetweenTask : public DeleteTask<int> {
|
| const DeleteCallback& callback)
|
| : DeleteTask<int>(cookie_monster, callback),
|
| delete_begin_(delete_begin),
|
| - delete_end_(delete_end) {
|
| - }
|
| + delete_end_(delete_end) {}
|
|
|
| // DeleteTask:
|
| int RunDeleteTask() override;
|
| @@ -649,8 +635,8 @@ class CookieMonster::DeleteAllCreatedBetweenTask : public DeleteTask<int> {
|
| };
|
|
|
| int CookieMonster::DeleteAllCreatedBetweenTask::RunDeleteTask() {
|
| - return this->cookie_monster()->
|
| - DeleteAllCreatedBetween(delete_begin_, delete_end_);
|
| + return this->cookie_monster()->DeleteAllCreatedBetween(delete_begin_,
|
| + delete_end_);
|
| }
|
|
|
| // Task class for DeleteAllForHost call.
|
| @@ -659,9 +645,7 @@ class CookieMonster::DeleteAllForHostTask : public DeleteTask<int> {
|
| DeleteAllForHostTask(CookieMonster* cookie_monster,
|
| const GURL& url,
|
| const DeleteCallback& callback)
|
| - : DeleteTask<int>(cookie_monster, callback),
|
| - url_(url) {
|
| - }
|
| + : DeleteTask<int>(cookie_monster, callback), url_(url) {}
|
|
|
| // DeleteTask:
|
| int RunDeleteTask() override;
|
| @@ -683,17 +667,15 @@ int CookieMonster::DeleteAllForHostTask::RunDeleteTask() {
|
| class CookieMonster::DeleteAllCreatedBetweenForHostTask
|
| : public DeleteTask<int> {
|
| public:
|
| - DeleteAllCreatedBetweenForHostTask(
|
| - CookieMonster* cookie_monster,
|
| - Time delete_begin,
|
| - Time delete_end,
|
| - const GURL& url,
|
| - const DeleteCallback& callback)
|
| + DeleteAllCreatedBetweenForHostTask(CookieMonster* cookie_monster,
|
| + Time delete_begin,
|
| + Time delete_end,
|
| + const GURL& url,
|
| + const DeleteCallback& callback)
|
| : DeleteTask<int>(cookie_monster, callback),
|
| delete_begin_(delete_begin),
|
| delete_end_(delete_end),
|
| - url_(url) {
|
| - }
|
| + url_(url) {}
|
|
|
| // DeleteTask:
|
| int RunDeleteTask() override;
|
| @@ -720,9 +702,7 @@ class CookieMonster::DeleteCanonicalCookieTask : public DeleteTask<bool> {
|
| DeleteCanonicalCookieTask(CookieMonster* cookie_monster,
|
| const CanonicalCookie& cookie,
|
| const DeleteCookieCallback& callback)
|
| - : DeleteTask<bool>(cookie_monster, callback),
|
| - cookie_(cookie) {
|
| - }
|
| + : DeleteTask<bool>(cookie_monster, callback), cookie_(cookie) {}
|
|
|
| // DeleteTask:
|
| bool RunDeleteTask() override;
|
| @@ -752,8 +732,7 @@ class CookieMonster::SetCookieWithOptionsTask : public CookieMonsterTask {
|
| url_(url),
|
| cookie_line_(cookie_line),
|
| options_(options),
|
| - callback_(callback) {
|
| - }
|
| + callback_(callback) {}
|
|
|
| // CookieMonsterTask:
|
| void Run() override;
|
| @@ -771,8 +750,8 @@ class CookieMonster::SetCookieWithOptionsTask : public CookieMonsterTask {
|
| };
|
|
|
| void CookieMonster::SetCookieWithOptionsTask::Run() {
|
| - bool result = this->cookie_monster()->
|
| - SetCookieWithOptions(url_, cookie_line_, options_);
|
| + bool result = this->cookie_monster()->SetCookieWithOptions(url_, cookie_line_,
|
| + options_);
|
| if (!callback_.is_null()) {
|
| this->InvokeCallback(base::Bind(&SetCookiesCallback::Run,
|
| base::Unretained(&callback_), result));
|
| @@ -789,8 +768,7 @@ class CookieMonster::GetCookiesWithOptionsTask : public CookieMonsterTask {
|
| : CookieMonsterTask(cookie_monster),
|
| url_(url),
|
| options_(options),
|
| - callback_(callback) {
|
| - }
|
| + callback_(callback) {}
|
|
|
| // CookieMonsterTask:
|
| void Run() override;
|
| @@ -807,8 +785,8 @@ class CookieMonster::GetCookiesWithOptionsTask : public CookieMonsterTask {
|
| };
|
|
|
| void CookieMonster::GetCookiesWithOptionsTask::Run() {
|
| - std::string cookie = this->cookie_monster()->
|
| - GetCookiesWithOptions(url_, options_);
|
| + std::string cookie =
|
| + this->cookie_monster()->GetCookiesWithOptions(url_, options_);
|
| if (!callback_.is_null()) {
|
| this->InvokeCallback(base::Bind(&GetCookiesCallback::Run,
|
| base::Unretained(&callback_), cookie));
|
| @@ -824,8 +802,7 @@ class CookieMonster::DeleteCookieTask : public DeleteTask<void> {
|
| const base::Closure& callback)
|
| : DeleteTask<void>(cookie_monster, callback),
|
| url_(url),
|
| - cookie_name_(cookie_name) {
|
| - }
|
| + cookie_name_(cookie_name) {}
|
|
|
| // DeleteTask:
|
| void RunDeleteTask() override;
|
| @@ -849,8 +826,7 @@ class CookieMonster::DeleteSessionCookiesTask : public DeleteTask<int> {
|
| public:
|
| DeleteSessionCookiesTask(CookieMonster* cookie_monster,
|
| const DeleteCallback& callback)
|
| - : DeleteTask<int>(cookie_monster, callback) {
|
| - }
|
| + : DeleteTask<int>(cookie_monster, callback) {}
|
|
|
| // DeleteTask:
|
| int RunDeleteTask() override;
|
| @@ -869,14 +845,12 @@ int CookieMonster::DeleteSessionCookiesTask::RunDeleteTask() {
|
| // Task class for HasCookiesForETLDP1Task call.
|
| class CookieMonster::HasCookiesForETLDP1Task : public CookieMonsterTask {
|
| public:
|
| - HasCookiesForETLDP1Task(
|
| - CookieMonster* cookie_monster,
|
| - const std::string& etldp1,
|
| - const HasCookiesForETLDP1Callback& callback)
|
| + HasCookiesForETLDP1Task(CookieMonster* cookie_monster,
|
| + const std::string& etldp1,
|
| + const HasCookiesForETLDP1Callback& callback)
|
| : CookieMonsterTask(cookie_monster),
|
| etldp1_(etldp1),
|
| - callback_(callback) {
|
| - }
|
| + callback_(callback) {}
|
|
|
| // CookieMonsterTask:
|
| void Run() override;
|
| @@ -894,9 +868,8 @@ class CookieMonster::HasCookiesForETLDP1Task : public CookieMonsterTask {
|
| void CookieMonster::HasCookiesForETLDP1Task::Run() {
|
| bool result = this->cookie_monster()->HasCookiesForETLDP1(etldp1_);
|
| if (!callback_.is_null()) {
|
| - this->InvokeCallback(
|
| - base::Bind(&HasCookiesForETLDP1Callback::Run,
|
| - base::Unretained(&callback_), result));
|
| + this->InvokeCallback(base::Bind(&HasCookiesForETLDP1Callback::Run,
|
| + base::Unretained(&callback_), result));
|
| }
|
| }
|
|
|
| @@ -913,21 +886,18 @@ void CookieMonster::SetCookieWithDetailsAsync(
|
| bool http_only,
|
| CookiePriority priority,
|
| const SetCookiesCallback& callback) {
|
| - scoped_refptr<SetCookieWithDetailsTask> task =
|
| - new SetCookieWithDetailsTask(this, url, name, value, domain, path,
|
| - expiration_time, secure, http_only, priority,
|
| - callback);
|
| + scoped_refptr<SetCookieWithDetailsTask> task = new SetCookieWithDetailsTask(
|
| + this, url, name, value, domain, path, expiration_time, secure, http_only,
|
| + priority, callback);
|
| DoCookieTaskForURL(task, url);
|
| }
|
|
|
| void CookieMonster::GetAllCookiesAsync(const GetCookieListCallback& callback) {
|
| - scoped_refptr<GetAllCookiesTask> task =
|
| - new GetAllCookiesTask(this, callback);
|
| + scoped_refptr<GetAllCookiesTask> task = new GetAllCookiesTask(this, callback);
|
|
|
| DoCookieTask(task);
|
| }
|
|
|
| -
|
| void CookieMonster::GetAllCookiesForURLWithOptionsAsync(
|
| const GURL& url,
|
| const CookieOptions& options,
|
| @@ -939,7 +909,8 @@ void CookieMonster::GetAllCookiesForURLWithOptionsAsync(
|
| }
|
|
|
| void CookieMonster::GetAllCookiesForURLAsync(
|
| - const GURL& url, const GetCookieListCallback& callback) {
|
| + const GURL& url,
|
| + const GetCookieListCallback& callback) {
|
| CookieOptions options;
|
| options.set_include_httponly();
|
| scoped_refptr<GetAllCookiesForURLWithOptionsTask> task =
|
| @@ -958,18 +929,17 @@ void CookieMonster::HasCookiesForETLDP1Async(
|
| }
|
|
|
| void CookieMonster::DeleteAllAsync(const DeleteCallback& callback) {
|
| - scoped_refptr<DeleteAllTask> task =
|
| - new DeleteAllTask(this, callback);
|
| + scoped_refptr<DeleteAllTask> task = new DeleteAllTask(this, callback);
|
|
|
| DoCookieTask(task);
|
| }
|
|
|
| void CookieMonster::DeleteAllCreatedBetweenAsync(
|
| - const Time& delete_begin, const Time& delete_end,
|
| + const Time& delete_begin,
|
| + const Time& delete_end,
|
| const DeleteCallback& callback) {
|
| scoped_refptr<DeleteAllCreatedBetweenTask> task =
|
| - new DeleteAllCreatedBetweenTask(this, delete_begin, delete_end,
|
| - callback);
|
| + new DeleteAllCreatedBetweenTask(this, delete_begin, delete_end, callback);
|
|
|
| DoCookieTask(task);
|
| }
|
| @@ -980,14 +950,14 @@ void CookieMonster::DeleteAllCreatedBetweenForHostAsync(
|
| const GURL& url,
|
| const DeleteCallback& callback) {
|
| scoped_refptr<DeleteAllCreatedBetweenForHostTask> task =
|
| - new DeleteAllCreatedBetweenForHostTask(
|
| - this, delete_begin, delete_end, url, callback);
|
| + new DeleteAllCreatedBetweenForHostTask(this, delete_begin, delete_end,
|
| + url, callback);
|
|
|
| DoCookieTaskForURL(task, url);
|
| }
|
|
|
| -void CookieMonster::DeleteAllForHostAsync(
|
| - const GURL& url, const DeleteCallback& callback) {
|
| +void CookieMonster::DeleteAllForHostAsync(const GURL& url,
|
| + const DeleteCallback& callback) {
|
| scoped_refptr<DeleteAllForHostTask> task =
|
| new DeleteAllForHostTask(this, url, callback);
|
|
|
| @@ -1065,16 +1035,19 @@ void CookieMonster::DoCookieTaskForURL(
|
| // then run the task, otherwise load from DB.
|
| if (!loaded_) {
|
| // Checks if the domain key has been loaded.
|
| - std::string key(cookie_util::GetEffectiveDomain(url.scheme(),
|
| - url.host()));
|
| + std::string key(
|
| + cookie_util::GetEffectiveDomain(url.scheme(), url.host()));
|
| if (keys_loaded_.find(key) == keys_loaded_.end()) {
|
| - std::map<std::string, std::deque<scoped_refptr<CookieMonsterTask> > >
|
| - ::iterator it = tasks_pending_for_key_.find(key);
|
| + std::map<std::string,
|
| + std::deque<scoped_refptr<CookieMonsterTask>>>::iterator it =
|
| + tasks_pending_for_key_.find(key);
|
| if (it == tasks_pending_for_key_.end()) {
|
| - store_->LoadCookiesForKey(key,
|
| - base::Bind(&CookieMonster::OnKeyLoaded, this, key));
|
| - it = tasks_pending_for_key_.insert(std::make_pair(key,
|
| - std::deque<scoped_refptr<CookieMonsterTask> >())).first;
|
| + store_->LoadCookiesForKey(
|
| + key, base::Bind(&CookieMonster::OnKeyLoaded, this, key));
|
| + it = tasks_pending_for_key_
|
| + .insert(std::make_pair(
|
| + key, std::deque<scoped_refptr<CookieMonsterTask>>()))
|
| + .first;
|
| }
|
| it->second.push_back(task_item);
|
| return;
|
| @@ -1103,8 +1076,8 @@ bool CookieMonster::SetCookieWithDetails(const GURL& url,
|
|
|
| scoped_ptr<CanonicalCookie> cc;
|
| cc.reset(CanonicalCookie::Create(url, name, value, domain, path,
|
| - creation_time, expiration_time,
|
| - secure, http_only, priority));
|
| + creation_time, expiration_time, secure,
|
| + http_only, priority));
|
|
|
| if (!cc.get())
|
| return false;
|
| @@ -1117,8 +1090,8 @@ bool CookieMonster::SetCookieWithDetails(const GURL& url,
|
| bool CookieMonster::ImportCookies(const CookieList& list) {
|
| base::AutoLock autolock(lock_);
|
| InitIfNecessary();
|
| - for (net::CookieList::const_iterator iter = list.begin();
|
| - iter != list.end(); ++iter) {
|
| + for (net::CookieList::const_iterator iter = list.begin(); iter != list.end();
|
| + ++iter) {
|
| scoped_ptr<CanonicalCookie> cookie(new CanonicalCookie(*iter));
|
| net::CookieOptions options;
|
| options.set_include_httponly();
|
| @@ -1139,9 +1112,8 @@ CookieList CookieMonster::GetAllCookies() {
|
| //
|
| // Note that this does not prune cookies to be below our limits (if we've
|
| // exceeded them) the way that calling GarbageCollect() would.
|
| - GarbageCollectExpired(Time::Now(),
|
| - CookieMapItPair(cookies_.begin(), cookies_.end()),
|
| - NULL);
|
| + GarbageCollectExpired(
|
| + Time::Now(), CookieMapItPair(cookies_.begin(), cookies_.end()), NULL);
|
|
|
| // Copy the CanonicalCookie pointers from the map so that we can use the same
|
| // sorter as elsewhere, then copy the result out.
|
| @@ -1193,8 +1165,9 @@ int CookieMonster::DeleteAll(bool sync_to_store) {
|
| CookieMap::iterator curit = it;
|
| ++it;
|
| InternalDeleteCookie(curit, sync_to_store,
|
| - sync_to_store ? DELETE_COOKIE_EXPLICIT :
|
| - DELETE_COOKIE_DONT_RECORD /* Destruction. */);
|
| + sync_to_store
|
| + ? DELETE_COOKIE_EXPLICIT
|
| + : DELETE_COOKIE_DONT_RECORD /* Destruction. */);
|
| ++num_deleted;
|
| }
|
|
|
| @@ -1213,8 +1186,7 @@ int CookieMonster::DeleteAllCreatedBetween(const Time& delete_begin,
|
|
|
| if (cc->CreationDate() >= delete_begin &&
|
| (delete_end.is_null() || cc->CreationDate() < delete_end)) {
|
| - InternalDeleteCookie(curit,
|
| - true, /*sync_to_store*/
|
| + InternalDeleteCookie(curit, true, /*sync_to_store*/
|
| DELETE_COOKIE_EXPLICIT);
|
| ++num_deleted;
|
| }
|
| @@ -1262,7 +1234,6 @@ int CookieMonster::DeleteAllForHost(const GURL& url) {
|
| return DeleteAllCreatedBetweenForHost(Time(), Time::Max(), url);
|
| }
|
|
|
| -
|
| bool CookieMonster::DeleteCanonicalCookie(const CanonicalCookie& cookie) {
|
| base::AutoLock autolock(lock_);
|
|
|
| @@ -1285,15 +1256,15 @@ void CookieMonster::SetCookieableSchemes(const char* const schemes[],
|
| DCHECK(!initialized_);
|
|
|
| cookieable_schemes_.clear();
|
| - cookieable_schemes_.insert(cookieable_schemes_.end(),
|
| - schemes, schemes + num_schemes);
|
| + cookieable_schemes_.insert(cookieable_schemes_.end(), schemes,
|
| + schemes + num_schemes);
|
| }
|
|
|
| void CookieMonster::SetEnableFileScheme(bool accept) {
|
| // This assumes "file" is always at the end of the array. See the comment
|
| // above kDefaultCookieableSchemes.
|
| - int num_schemes = accept ? kDefaultCookieableSchemesCount :
|
| - kDefaultCookieableSchemesCount - 1;
|
| + int num_schemes = accept ? kDefaultCookieableSchemesCount
|
| + : kDefaultCookieableSchemesCount - 1;
|
| SetCookieableSchemes(kDefaultCookieableSchemes, num_schemes);
|
| }
|
|
|
| @@ -1381,8 +1352,7 @@ int CookieMonster::DeleteSessionCookies() {
|
| ++it;
|
|
|
| if (!cc->IsPersistent()) {
|
| - InternalDeleteCookie(curit,
|
| - true, /*sync_to_store*/
|
| + InternalDeleteCookie(curit, true, /*sync_to_store*/
|
| DELETE_COOKIE_EXPIRED);
|
| ++num_deleted;
|
| }
|
| @@ -1467,7 +1437,7 @@ void CookieMonster::OnKeyLoaded(const std::string& key,
|
| // This function does its own separate locking.
|
| StoreLoadedCookies(cookies);
|
|
|
| - std::deque<scoped_refptr<CookieMonsterTask> > tasks_pending_for_key;
|
| + std::deque<scoped_refptr<CookieMonsterTask>> tasks_pending_for_key;
|
|
|
| // We need to do this repeatedly until no more tasks were added to the queue
|
| // during the period where we release the lock.
|
| @@ -1478,8 +1448,9 @@ void CookieMonster::OnKeyLoaded(const std::string& key,
|
| FROM_HERE_WITH_EXPLICIT_FUNCTION("456373 CookieMonster::OnKeyLoaded1"));
|
| {
|
| base::AutoLock autolock(lock_);
|
| - std::map<std::string, std::deque<scoped_refptr<CookieMonsterTask> > >
|
| - ::iterator it = tasks_pending_for_key_.find(key);
|
| + std::map<std::string,
|
| + std::deque<scoped_refptr<CookieMonsterTask>>>::iterator it =
|
| + tasks_pending_for_key_.find(key);
|
| if (it == tasks_pending_for_key_.end()) {
|
| keys_loaded_.insert(key);
|
| return;
|
| @@ -1531,15 +1502,15 @@ void CookieMonster::StoreLoadedCookies(
|
|
|
| if (ContainsControlCharacter((*it)->Name()) ||
|
| ContainsControlCharacter((*it)->Value())) {
|
| - cookies_with_control_chars.push_back(inserted);
|
| + cookies_with_control_chars.push_back(inserted);
|
| }
|
| } else {
|
| - LOG(ERROR) << base::StringPrintf("Found cookies with duplicate creation "
|
| - "times in backing store: "
|
| - "{name='%s', domain='%s', path='%s'}",
|
| - (*it)->Name().c_str(),
|
| - (*it)->Domain().c_str(),
|
| - (*it)->Path().c_str());
|
| + LOG(ERROR) << base::StringPrintf(
|
| + "Found cookies with duplicate creation "
|
| + "times in backing store: "
|
| + "{name='%s', domain='%s', path='%s'}",
|
| + (*it)->Name().c_str(), (*it)->Domain().c_str(),
|
| + (*it)->Path().c_str());
|
| // We've been given ownership of the cookie and are throwing it
|
| // away; reclaim the space.
|
| delete (*it);
|
| @@ -1610,10 +1581,9 @@ void CookieMonster::EnsureCookiesMapIsValid() {
|
| histogram_cookie_deletion_cause_->Add(num_duplicates_trimmed);
|
| }
|
|
|
| -int CookieMonster::TrimDuplicateCookiesForKey(
|
| - const std::string& key,
|
| - CookieMap::iterator begin,
|
| - CookieMap::iterator end) {
|
| +int CookieMonster::TrimDuplicateCookiesForKey(const std::string& key,
|
| + CookieMap::iterator begin,
|
| + CookieMap::iterator end) {
|
| lock_.AssertAcquired();
|
|
|
| // Set of cookies ordered by creation time.
|
| @@ -1632,8 +1602,7 @@ int CookieMonster::TrimDuplicateCookiesForKey(
|
| DCHECK_EQ(key, it->first);
|
| CanonicalCookie* cookie = it->second;
|
|
|
| - CookieSignature signature(cookie->Name(), cookie->Domain(),
|
| - cookie->Path());
|
| + CookieSignature signature(cookie->Name(), cookie->Domain(), cookie->Path());
|
| CookieSet& set = equivalent_cookies[signature];
|
|
|
| // We found a duplicate!
|
| @@ -1643,8 +1612,8 @@ int CookieMonster::TrimDuplicateCookiesForKey(
|
| // We save the iterator into |cookies_| rather than the actual cookie
|
| // pointer, since we may need to delete it later.
|
| bool insert_success = set.insert(it).second;
|
| - DCHECK(insert_success) <<
|
| - "Duplicate creation times found in duplicate cookie name scan.";
|
| + DCHECK(insert_success)
|
| + << "Duplicate creation times found in duplicate cookie name scan.";
|
| }
|
|
|
| // If there were no duplicates, we are done!
|
| @@ -1657,8 +1626,7 @@ int CookieMonster::TrimDuplicateCookiesForKey(
|
| // Otherwise, delete all the duplicate cookies, both from our in-memory store
|
| // and from the backing store.
|
| for (EquivalenceMap::iterator it = equivalent_cookies.begin();
|
| - it != equivalent_cookies.end();
|
| - ++it) {
|
| + it != equivalent_cookies.end(); ++it) {
|
| const CookieSignature& signature = it->first;
|
| CookieSet& dupes = it->second;
|
|
|
| @@ -1673,17 +1641,13 @@ int CookieMonster::TrimDuplicateCookiesForKey(
|
| LOG(ERROR) << base::StringPrintf(
|
| "Found %d duplicate cookies for host='%s', "
|
| "with {name='%s', domain='%s', path='%s'}",
|
| - static_cast<int>(dupes.size()),
|
| - key.c_str(),
|
| - signature.name.c_str(),
|
| - signature.domain.c_str(),
|
| - signature.path.c_str());
|
| + static_cast<int>(dupes.size()), key.c_str(), signature.name.c_str(),
|
| + signature.domain.c_str(), signature.path.c_str());
|
|
|
| // Remove all the cookies identified by |dupes|. It is valid to delete our
|
| // list of iterators one at a time, since |cookies_| is a multimap (they
|
| // don't invalidate existing iterators following deletion).
|
| - for (CookieSet::iterator dupes_it = dupes.begin();
|
| - dupes_it != dupes.end();
|
| + for (CookieSet::iterator dupes_it = dupes.begin(); dupes_it != dupes.end();
|
| ++dupes_it) {
|
| InternalDeleteCookie(*dupes_it, true,
|
| DELETE_COOKIE_DUPLICATE_IN_BACKING_STORE);
|
| @@ -1695,8 +1659,11 @@ int CookieMonster::TrimDuplicateCookiesForKey(
|
| }
|
|
|
| // Note: file must be the last scheme.
|
| -const char* const CookieMonster::kDefaultCookieableSchemes[] =
|
| - { "http", "https", "ws", "wss", "file" };
|
| +const char* const CookieMonster::kDefaultCookieableSchemes[] = {"http",
|
| + "https",
|
| + "ws",
|
| + "wss",
|
| + "file"};
|
| const int CookieMonster::kDefaultCookieableSchemesCount =
|
| arraysize(kDefaultCookieableSchemes);
|
|
|
| @@ -1722,8 +1689,8 @@ void CookieMonster::FindCookiesForHostAndDomain(
|
|
|
| // Can just dispatch to FindCookiesForKey
|
| const std::string key(GetKey(url.host()));
|
| - FindCookiesForKey(key, url, options, current_time,
|
| - update_access_time, cookies);
|
| + FindCookiesForKey(key, url, options, current_time, update_access_time,
|
| + cookies);
|
| }
|
|
|
| void CookieMonster::FindCookiesForKey(const std::string& key,
|
| @@ -1735,7 +1702,7 @@ void CookieMonster::FindCookiesForKey(const std::string& key,
|
| lock_.AssertAcquired();
|
|
|
| for (CookieMapItPair its = cookies_.equal_range(key);
|
| - its.first != its.second; ) {
|
| + its.first != its.second;) {
|
| CookieMap::iterator curit = its.first;
|
| CanonicalCookie* cc = curit->second;
|
| ++its.first;
|
| @@ -1770,7 +1737,7 @@ bool CookieMonster::DeleteAnyEquivalentCookie(const std::string& key,
|
| bool found_equivalent_cookie = false;
|
| bool skipped_httponly = false;
|
| for (CookieMapItPair its = cookies_.equal_range(key);
|
| - its.first != its.second; ) {
|
| + its.first != its.second;) {
|
| CookieMap::iterator curit = its.first;
|
| CanonicalCookie* cc = curit->second;
|
| ++its.first;
|
| @@ -1778,13 +1745,14 @@ bool CookieMonster::DeleteAnyEquivalentCookie(const std::string& key,
|
| if (ecc.IsEquivalent(*cc)) {
|
| // We should never have more than one equivalent cookie, since they should
|
| // overwrite each other.
|
| - CHECK(!found_equivalent_cookie) <<
|
| - "Duplicate equivalent cookies found, cookie store is corrupted.";
|
| + CHECK(!found_equivalent_cookie)
|
| + << "Duplicate equivalent cookies found, cookie store is corrupted.";
|
| if (skip_httponly && cc->IsHttpOnly()) {
|
| skipped_httponly = true;
|
| } else {
|
| - InternalDeleteCookie(curit, true, already_expired ?
|
| - DELETE_COOKIE_EXPIRED_OVERWRITE : DELETE_COOKIE_OVERWRITE);
|
| + InternalDeleteCookie(curit, true, already_expired
|
| + ? DELETE_COOKIE_EXPIRED_OVERWRITE
|
| + : DELETE_COOKIE_OVERWRITE);
|
| }
|
| found_equivalent_cookie = true;
|
| }
|
| @@ -1804,8 +1772,8 @@ CookieMonster::CookieMap::iterator CookieMonster::InternalInsertCookie(
|
| CookieMap::iterator inserted =
|
| cookies_.insert(CookieMap::value_type(key, cc));
|
| if (delegate_.get()) {
|
| - delegate_->OnCookieChanged(
|
| - *cc, false, CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT);
|
| + delegate_->OnCookieChanged(*cc, false,
|
| + CookieMonsterDelegate::CHANGE_COOKIE_EXPLICIT);
|
| }
|
| RunCallbacks(*cc, false);
|
|
|
| @@ -1849,8 +1817,8 @@ bool CookieMonster::SetCanonicalCookie(scoped_ptr<CanonicalCookie>* cc,
|
| return false;
|
| }
|
|
|
| - VLOG(kVlogSetCookies) << "SetCookie() key: " << key << " cc: "
|
| - << (*cc)->DebugString();
|
| + VLOG(kVlogSetCookies) << "SetCookie() key: " << key
|
| + << " cc: " << (*cc)->DebugString();
|
|
|
| // Realize that we might be setting an expired cookie, and the only point
|
| // was to delete the cookie which we've already done.
|
| @@ -1935,21 +1903,19 @@ void CookieMonster::InternalDeleteCookie(CookieMap::iterator it,
|
|
|
| // Domain expiry behavior is unchanged by key/expiry scheme (the
|
| // meaning of the key is different, but that's not visible to this routine).
|
| -int CookieMonster::GarbageCollect(const Time& current,
|
| - const std::string& key) {
|
| +int CookieMonster::GarbageCollect(const Time& current, const std::string& key) {
|
| lock_.AssertAcquired();
|
|
|
| int num_deleted = 0;
|
| - Time safe_date(
|
| - Time::Now() - TimeDelta::FromDays(kSafeFromGlobalPurgeDays));
|
| + Time safe_date(Time::Now() - TimeDelta::FromDays(kSafeFromGlobalPurgeDays));
|
|
|
| // Collect garbage for this key, minding cookie priorities.
|
| if (cookies_.count(key) > kDomainMaxCookies) {
|
| VLOG(kVlogGarbageCollection) << "GarbageCollect() key: " << key;
|
|
|
| CookieItVector cookie_its;
|
| - num_deleted += GarbageCollectExpired(
|
| - current, cookies_.equal_range(key), &cookie_its);
|
| + num_deleted +=
|
| + GarbageCollectExpired(current, cookies_.equal_range(key), &cookie_its);
|
| if (cookie_its.size() > kDomainMaxCookies) {
|
| VLOG(kVlogGarbageCollection) << "Deep Garbage Collect domain.";
|
| size_t purge_goal =
|
| @@ -1962,15 +1928,13 @@ int CookieMonster::GarbageCollect(const Time& current,
|
| // Schematic: [MLLHMHHLMM] => [LLL|MMMM|HHH], with 4 boundaries.
|
| it_bdd[0] = cookie_its.begin();
|
| it_bdd[3] = cookie_its.end();
|
| - it_bdd[1] = PartitionCookieByPriority(it_bdd[0], it_bdd[3],
|
| - COOKIE_PRIORITY_LOW);
|
| + it_bdd[1] =
|
| + PartitionCookieByPriority(it_bdd[0], it_bdd[3], COOKIE_PRIORITY_LOW);
|
| it_bdd[2] = PartitionCookieByPriority(it_bdd[1], it_bdd[3],
|
| COOKIE_PRIORITY_MEDIUM);
|
| - size_t quota[3] = {
|
| - kDomainCookiesQuotaLow,
|
| - kDomainCookiesQuotaMedium,
|
| - kDomainCookiesQuotaHigh
|
| - };
|
| + size_t quota[3] = {kDomainCookiesQuotaLow,
|
| + kDomainCookiesQuotaMedium,
|
| + kDomainCookiesQuotaHigh};
|
|
|
| // Purge domain cookies in 3 rounds.
|
| // Round 1: consider low-priority cookies only: evict least-recently
|
| @@ -2000,15 +1964,11 @@ int CookieMonster::GarbageCollect(const Time& current,
|
| LowerBoundAccessDate(it_purge_begin, it_purge_end, safe_date);
|
| // Delete cookies accessed before |safe_date|.
|
| num_deleted += GarbageCollectDeleteRange(
|
| - current,
|
| - DELETE_COOKIE_EVICTED_DOMAIN_PRE_SAFE,
|
| - it_purge_begin,
|
| + current, DELETE_COOKIE_EVICTED_DOMAIN_PRE_SAFE, it_purge_begin,
|
| it_purge_middle);
|
| // Delete cookies accessed on or after |safe_date|.
|
| num_deleted += GarbageCollectDeleteRange(
|
| - current,
|
| - DELETE_COOKIE_EVICTED_DOMAIN_POST_SAFE,
|
| - it_purge_middle,
|
| + current, DELETE_COOKIE_EVICTED_DOMAIN_POST_SAFE, it_purge_middle,
|
| it_purge_end);
|
| it_purge_begin = it_purge_end;
|
| }
|
| @@ -2018,8 +1978,7 @@ int CookieMonster::GarbageCollect(const Time& current,
|
|
|
| // Collect garbage for everything. With firefox style we want to preserve
|
| // cookies accessed in kSafeFromGlobalPurgeDays, otherwise evict.
|
| - if (cookies_.size() > kMaxCookies &&
|
| - earliest_access_time_ < safe_date) {
|
| + if (cookies_.size() > kMaxCookies && earliest_access_time_ < safe_date) {
|
| VLOG(kVlogGarbageCollection) << "GarbageCollect() everything";
|
| CookieItVector cookie_its;
|
| num_deleted += GarbageCollectExpired(
|
| @@ -2035,16 +1994,12 @@ int CookieMonster::GarbageCollect(const Time& current,
|
| SortLeastRecentlyAccessed(cookie_its.begin(), cookie_its.end(),
|
| purge_goal);
|
| // Find boundary to cookies older than safe_date.
|
| - CookieItVector::iterator global_purge_it =
|
| - LowerBoundAccessDate(cookie_its.begin(),
|
| - cookie_its.begin() + purge_goal,
|
| - safe_date);
|
| + CookieItVector::iterator global_purge_it = LowerBoundAccessDate(
|
| + cookie_its.begin(), cookie_its.begin() + purge_goal, safe_date);
|
| // Only delete the old cookies.
|
| - num_deleted += GarbageCollectDeleteRange(
|
| - current,
|
| - DELETE_COOKIE_EVICTED_GLOBAL,
|
| - cookie_its.begin(),
|
| - global_purge_it);
|
| + num_deleted +=
|
| + GarbageCollectDeleteRange(current, DELETE_COOKIE_EVICTED_GLOBAL,
|
| + cookie_its.begin(), global_purge_it);
|
| // Set access day to the oldest cookie that wasn't deleted.
|
| earliest_access_time_ = (*global_purge_it)->second->LastAccessDate();
|
| }
|
| @@ -2053,10 +2008,9 @@ int CookieMonster::GarbageCollect(const Time& current,
|
| return num_deleted;
|
| }
|
|
|
| -int CookieMonster::GarbageCollectExpired(
|
| - const Time& current,
|
| - const CookieMapItPair& itpair,
|
| - CookieItVector* cookie_its) {
|
| +int CookieMonster::GarbageCollectExpired(const Time& current,
|
| + const CookieMapItPair& itpair,
|
| + CookieItVector* cookie_its) {
|
| if (keep_expired_cookies_)
|
| return 0;
|
|
|
| @@ -2078,11 +2032,10 @@ int CookieMonster::GarbageCollectExpired(
|
| return num_deleted;
|
| }
|
|
|
| -int CookieMonster::GarbageCollectDeleteRange(
|
| - const Time& current,
|
| - DeletionCause cause,
|
| - CookieItVector::iterator it_begin,
|
| - CookieItVector::iterator it_end) {
|
| +int CookieMonster::GarbageCollectDeleteRange(const Time& current,
|
| + DeletionCause cause,
|
| + CookieItVector::iterator it_begin,
|
| + CookieItVector::iterator it_end) {
|
| for (CookieItVector::iterator it = it_begin; it != it_end; it++) {
|
| histogram_evicted_last_access_minutes_->Add(
|
| (current - (*it)->second->LastAccessDate()).InMinutes());
|
| @@ -2146,8 +2099,8 @@ bool CookieMonster::HasCookieableScheme(const GURL& url) {
|
| }
|
|
|
| // The scheme didn't match any in our whitelist.
|
| - VLOG(kVlogPerCookieMonster) << "WARNING: Unsupported cookie scheme: "
|
| - << url.scheme();
|
| + VLOG(kVlogPerCookieMonster)
|
| + << "WARNING: Unsupported cookie scheme: " << url.scheme();
|
| return false;
|
| }
|
|
|
| @@ -2176,7 +2129,7 @@ void CookieMonster::RecordPeriodicStats(const base::Time& current_time) {
|
| TimeTicks beginning_of_time(TimeTicks::Now());
|
|
|
| for (CookieMap::const_iterator it_key = cookies_.begin();
|
| - it_key != cookies_.end(); ) {
|
| + it_key != cookies_.end();) {
|
| const std::string& key(it_key->first);
|
|
|
| int key_count = 0;
|
| @@ -2199,9 +2152,9 @@ void CookieMonster::RecordPeriodicStats(const base::Time& current_time) {
|
| it_key = its_cookies.second;
|
| }
|
|
|
| - VLOG(kVlogPeriodic)
|
| - << "Time for recording cookie stats (us): "
|
| - << (TimeTicks::Now() - beginning_of_time).InMicroseconds();
|
| + VLOG(kVlogPeriodic) << "Time for recording cookie stats (us): "
|
| + << (TimeTicks::Now() - beginning_of_time)
|
| + .InMicroseconds();
|
|
|
| last_statistic_record_time_ = current_time;
|
| }
|
| @@ -2232,55 +2185,49 @@ void CookieMonster::RecordPeriodicStats(const base::Time& current_time) {
|
| void CookieMonster::InitializeHistograms() {
|
| // From UMA_HISTOGRAM_CUSTOM_COUNTS
|
| histogram_expiration_duration_minutes_ = base::Histogram::FactoryGet(
|
| - "Cookie.ExpirationDurationMinutes",
|
| - 1, kMinutesInTenYears, 50,
|
| + "Cookie.ExpirationDurationMinutes", 1, kMinutesInTenYears, 50,
|
| base::Histogram::kUmaTargetedHistogramFlag);
|
| histogram_between_access_interval_minutes_ = base::Histogram::FactoryGet(
|
| - "Cookie.BetweenAccessIntervalMinutes",
|
| - 1, kMinutesInTenYears, 50,
|
| + "Cookie.BetweenAccessIntervalMinutes", 1, kMinutesInTenYears, 50,
|
| base::Histogram::kUmaTargetedHistogramFlag);
|
| histogram_evicted_last_access_minutes_ = base::Histogram::FactoryGet(
|
| - "Cookie.EvictedLastAccessMinutes",
|
| - 1, kMinutesInTenYears, 50,
|
| + "Cookie.EvictedLastAccessMinutes", 1, kMinutesInTenYears, 50,
|
| base::Histogram::kUmaTargetedHistogramFlag);
|
| histogram_count_ = base::Histogram::FactoryGet(
|
| - "Cookie.Count", 1, 4000, 50,
|
| - base::Histogram::kUmaTargetedHistogramFlag);
|
| - histogram_domain_count_ = base::Histogram::FactoryGet(
|
| - "Cookie.DomainCount", 1, 4000, 50,
|
| - base::Histogram::kUmaTargetedHistogramFlag);
|
| - histogram_etldp1_count_ = base::Histogram::FactoryGet(
|
| - "Cookie.Etldp1Count", 1, 4000, 50,
|
| - base::Histogram::kUmaTargetedHistogramFlag);
|
| - histogram_domain_per_etldp1_count_ = base::Histogram::FactoryGet(
|
| - "Cookie.DomainPerEtldp1Count", 1, 4000, 50,
|
| - base::Histogram::kUmaTargetedHistogramFlag);
|
| + "Cookie.Count", 1, 4000, 50, base::Histogram::kUmaTargetedHistogramFlag);
|
| + histogram_domain_count_ =
|
| + base::Histogram::FactoryGet("Cookie.DomainCount", 1, 4000, 50,
|
| + base::Histogram::kUmaTargetedHistogramFlag);
|
| + histogram_etldp1_count_ =
|
| + base::Histogram::FactoryGet("Cookie.Etldp1Count", 1, 4000, 50,
|
| + base::Histogram::kUmaTargetedHistogramFlag);
|
| + histogram_domain_per_etldp1_count_ =
|
| + base::Histogram::FactoryGet("Cookie.DomainPerEtldp1Count", 1, 4000, 50,
|
| + base::Histogram::kUmaTargetedHistogramFlag);
|
|
|
| // From UMA_HISTOGRAM_COUNTS_10000 & UMA_HISTOGRAM_CUSTOM_COUNTS
|
| - histogram_number_duplicate_db_cookies_ = base::Histogram::FactoryGet(
|
| - "Net.NumDuplicateCookiesInDb", 1, 10000, 50,
|
| - base::Histogram::kUmaTargetedHistogramFlag);
|
| + histogram_number_duplicate_db_cookies_ =
|
| + base::Histogram::FactoryGet("Net.NumDuplicateCookiesInDb", 1, 10000, 50,
|
| + base::Histogram::kUmaTargetedHistogramFlag);
|
|
|
| // From UMA_HISTOGRAM_ENUMERATION
|
| histogram_cookie_deletion_cause_ = base::LinearHistogram::FactoryGet(
|
| - "Cookie.DeletionCause", 1,
|
| - DELETE_COOKIE_LAST_ENTRY - 1, DELETE_COOKIE_LAST_ENTRY,
|
| - base::Histogram::kUmaTargetedHistogramFlag);
|
| + "Cookie.DeletionCause", 1, DELETE_COOKIE_LAST_ENTRY - 1,
|
| + DELETE_COOKIE_LAST_ENTRY, base::Histogram::kUmaTargetedHistogramFlag);
|
|
|
| // From UMA_HISTOGRAM_{CUSTOM_,}TIMES
|
| histogram_time_blocked_on_load_ = base::Histogram::FactoryTimeGet(
|
| - "Cookie.TimeBlockedOnLoad",
|
| - base::TimeDelta::FromMilliseconds(1), base::TimeDelta::FromMinutes(1),
|
| - 50, base::Histogram::kUmaTargetedHistogramFlag);
|
| + "Cookie.TimeBlockedOnLoad", base::TimeDelta::FromMilliseconds(1),
|
| + base::TimeDelta::FromMinutes(1), 50,
|
| + base::Histogram::kUmaTargetedHistogramFlag);
|
| }
|
|
|
| -
|
| // The system resolution is not high enough, so we can have multiple
|
| // set cookies that result in the same system time. When this happens, we
|
| // increment by one Time unit. Let's hope computers don't get too fast.
|
| Time CookieMonster::CurrentTime() {
|
| - return std::max(Time::Now(),
|
| - Time::FromInternalValue(last_time_seen_.ToInternalValue() + 1));
|
| + return std::max(Time::Now(), Time::FromInternalValue(
|
| + last_time_seen_.ToInternalValue() + 1));
|
| }
|
|
|
| bool CookieMonster::CopyCookiesForKeyToOtherCookieMonster(
|
| @@ -2295,8 +2242,7 @@ bool CookieMonster::CopyCookiesForKeyToOtherCookieMonster(
|
| return false;
|
|
|
| for (CookieMapItPair its = cookies_.equal_range(key);
|
| - its.first != its.second;
|
| - ++its.first) {
|
| + its.first != its.second; ++its.first) {
|
| CookieMap::iterator curit = its.first;
|
| CanonicalCookie* cc = curit->second;
|
|
|
| @@ -2317,8 +2263,7 @@ bool CookieMonster::CopyCookiesForKeyToOtherCookieMonster(
|
| // Store the copied cookies in |other|.
|
| for (ScopedVector<CanonicalCookie>::const_iterator it =
|
| duplicated_cookies.begin();
|
| - it != duplicated_cookies.end();
|
| - ++it) {
|
| + it != duplicated_cookies.end(); ++it) {
|
| other->InternalInsertCookie(key, *it, true);
|
| }
|
|
|
| @@ -2335,10 +2280,9 @@ bool CookieMonster::loaded() {
|
| }
|
|
|
| scoped_ptr<CookieStore::CookieChangedSubscription>
|
| -CookieMonster::AddCallbackForCookie(
|
| - const GURL& gurl,
|
| - const std::string& name,
|
| - const CookieChangedCallback& callback) {
|
| +CookieMonster::AddCallbackForCookie(const GURL& gurl,
|
| + const std::string& name,
|
| + const CookieChangedCallback& callback) {
|
| base::AutoLock autolock(lock_);
|
| std::pair<GURL, std::string> key(gurl, name);
|
| if (hook_map_.count(key) == 0)
|
|
|