Match definition and declaration order of CookieMonster methods.

Match definition and declaration order of CookieMonster methods.

BUG=none

Committed: https://crrev.com/74bcbd548897b81ad7867001e634d6025d2d073f
Cr-Commit-Position: refs/heads/master@{#370720}

[mmenke, 2016-01-20 22:14:03]

mmenke@chromium.org changed reviewers:
+ mkwst@chromium.org

[mmenke, 2016-01-20 22:14:03]

Not sure if I should be sending these to you or Randy - may split them between
the two of you.

https://codereview.chromium.org/1607473010/diff/1/net/cookies/cookie_monster.cc
File net/cookies/cookie_monster.cc (right):

https://codereview.chromium.org/1607473010/diff/1/net/cookies/cookie_monster....
net/cookies/cookie_monster.cc:358: : CookieMonster(store, delegate,
kDefaultAccessUpdateThresholdSeconds) {
Other than this and the change in the header, all changes to this file are just
cut and paste.  Deliberately did not run "git cl format" on this.

https://codereview.chromium.org/1607473010/diff/1/net/cookies/cookie_monster.h
File net/cookies/cookie_monster.h (left):

https://codereview.chromium.org/1607473010/diff/1/net/cookies/cookie_monster....
net/cookies/cookie_monster.h:291: void ValidateMap(int arg);
I removed this, since this method doesn't actually exist.

[Mike West, 2016-01-21 09:28:36]

Copy/paste LGTM. If you'd like to run `git cl format` on the code and upload
that, I'd be happy to stamp it.

https://codereview.chromium.org/1607473010/diff/1/net/cookies/cookie_monster.h
File net/cookies/cookie_monster.h (left):

https://codereview.chromium.org/1607473010/diff/1/net/cookies/cookie_monster....
net/cookies/cookie_monster.h:291: void ValidateMap(int arg);
On 2016/01/20 at 22:14:03, mmenke wrote:
> I removed this, since this method doesn't actually exist.

These are the best. :)

[mmenke, 2016-01-21 16:28:49]

On 2016/01/21 09:28:36, Mike West wrote:
> Copy/paste LGTM. If you'd like to run `git cl format` on the code and upload
> that, I'd be happy to stamp it.

Think I'll land it as-is, just to keep it simple.  Didn't notice anything that
desperately needs to be reformatted when doing this.

> https://codereview.chromium.org/1607473010/diff/1/net/cookies/cookie_monster.h
> File net/cookies/cookie_monster.h (left):
> 
>
https://codereview.chromium.org/1607473010/diff/1/net/cookies/cookie_monster....
> net/cookies/cookie_monster.h:291: void ValidateMap(int arg);
> On 2016/01/20 at 22:14:03, mmenke wrote:
> > I removed this, since this method doesn't actually exist.
> 
> These are the best. :)

[mmenke, 2016-01-21 16:28:54]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2016-01-21 16:29:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1607473010/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1607473010/1

[commit-bot: I haz the power, 2016-01-21 17:18:06]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-21 17:19:23]

Description was changed from

==========
Match definition and declaration order of CookieMonster methods.

BUG=none
==========

to

==========
Match definition and declaration order of CookieMonster methods.

BUG=none

Committed: https://crrev.com/74bcbd548897b81ad7867001e634d6025d2d073f
Cr-Commit-Position: refs/heads/master@{#370720}
==========

[commit-bot: I haz the power, 2016-01-21 17:19:24]

Patchset 1 (id:??) landed as
https://crrev.com/74bcbd548897b81ad7867001e634d6025d2d073f
Cr-Commit-Position: refs/heads/master@{#370720}