| 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) | 
|  |