base::Bind fixes

base::Bind fixes

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110402

[eroman, 2011-11-16 02:48:58]

lgtm

[wtc, 2011-11-16 06:30:18]

mattm: please do a second review of this CL because this
is origin-bound certificate code.

http://codereview.chromium.org/8573031/diff/2001/net/base/default_origin_boun...
File net/base/default_origin_bound_cert_store.h (right):

http://codereview.chromium.org/8573031/diff/2001/net/base/default_origin_boun...
net/base/default_origin_bound_cert_store.h:55: void FlushStore(const
base::Closure& completion_task);
Should we rename the function parameter because it is no longer
a Task?

[mattm, 2011-11-16 20:56:35]

lgtm

[groby-ooo-7-16, 2011-11-16 22:24:58]

http://codereview.chromium.org/8573031/diff/2001/net/base/default_origin_boun...
File net/base/default_origin_bound_cert_store.h (right):

http://codereview.chromium.org/8573031/diff/2001/net/base/default_origin_boun...
net/base/default_origin_bound_cert_store.h:55: void FlushStore(const
base::Closure& completion_task);
On 2011/11/16 06:30:18, wtc wrote:
> Should we rename the function parameter because it is no longer
> a Task?

We still call it a task pretty much everywhere else. (i.e. PostTask etc.)

[commit-bot: I haz the power, 2011-11-16 22:25:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/groby@chromium.org/8573031/2001

[commit-bot: I haz the power, 2011-11-17 00:48:24]

Change committed as 110402