NetLogEventParameter to Callback refactoring 2.

NetLogEventParameter to Callback refactoring 2.
Get rid of all uses of NetLogEventParameters in net/disk_cache.

R=eroman@chromium.org
BUG=126243
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=141697

[eroman, 2012-06-12 16:54:07]

lgtm

http://codereview.chromium.org/10543114/diff/6002/net/disk_cache/net_log_para...
File net/disk_cache/net_log_parameters.cc (right):

http://codereview.chromium.org/10543114/diff/6002/net/disk_cache/net_log_para...
net/disk_cache/net_log_parameters.cc:128: }  // namespace disk_cache
I wonder why disk_cache isn't inside of the net:: namespace?

http://codereview.chromium.org/10543114/diff/6002/net/disk_cache/net_log_para...
File net/disk_cache/net_log_parameters.h (right):

http://codereview.chromium.org/10543114/diff/6002/net/disk_cache/net_log_para...
net/disk_cache/net_log_parameters.h:13: // This file contains a set of
NetLog::EventParameters shared by EntryImpls and
nit: update comment (EventParameters no longer current)

[mmenke, 2012-06-12 16:59:09]

Thanks

http://codereview.chromium.org/10543114/diff/6002/net/disk_cache/net_log_para...
File net/disk_cache/net_log_parameters.cc (right):

http://codereview.chromium.org/10543114/diff/6002/net/disk_cache/net_log_para...
net/disk_cache/net_log_parameters.cc:128: }  // namespace disk_cache
On 2012/06/12 16:54:07, eroman wrote:
> I wonder why disk_cache isn't inside of the net:: namespace?

I don't think I've seen any nested namespaces used in Chrome.  Suspect that's
why.

http://codereview.chromium.org/10543114/diff/6002/net/disk_cache/net_log_para...
File net/disk_cache/net_log_parameters.h (right):

http://codereview.chromium.org/10543114/diff/6002/net/disk_cache/net_log_para...
net/disk_cache/net_log_parameters.h:13: // This file contains a set of
NetLog::EventParameters shared by EntryImpls and
On 2012/06/12 16:54:07, eroman wrote:
> nit: update comment (EventParameters no longer current)

Done.

[commit-bot: I haz the power, 2012-06-12 16:59:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mmenke@chromium.org/10543114/10005

[commit-bot: I haz the power, 2012-06-12 19:07:22]

Change committed as 141697