(Finally) Remove NetLog::EventParameters.

(Finally) Remove NetLog::EventParameters.

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

[mmenke, 2012-06-15 20:09:30]

I'll want to run this through all the trybots (linux, mac, android, chromeos)
once the other CL lands, just in case, but think this is it.

[eroman, 2012-06-16 01:19:15]

lgtm

http://codereview.chromium.org/10565009/diff/7001/net/base/net_log.h
File net/base/net_log.h (right):

http://codereview.chromium.org/10565009/diff/7001/net/base/net_log.h#newcode14
net/base/net_log.h:14: #include "base/memory/ref_counted.h"
is this still needed?

[mmenke, 2012-06-16 02:09:25]

http://codereview.chromium.org/10565009/diff/7001/net/base/net_log.h
File net/base/net_log.h (right):

http://codereview.chromium.org/10565009/diff/7001/net/base/net_log.h#newcode14
net/base/net_log.h:14: #include "base/memory/ref_counted.h"
On 2012/06/16 01:19:15, eroman wrote:
> is this still needed?

Nope.

[mmenke, 2012-06-16 02:20:23]

Thanks so much for all the speedy reviews.

[eroman, 2012-06-16 23:54:14]

Thanks for making this change :)

[commit-bot: I haz the power, 2012-06-19 05:57:49]

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

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

Try job failure for 10565009-1006 (retry) on mac_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[mmenke, 2012-06-19 13:19:45]

On 2012/06/19 06:12:46, I haz the power (commit-bot) wrote:
> Try job failure for 10565009-1006 (retry) on mac_rel for step "compile"
(clobber
> build).
> It's a second try, previously, step "compile" failed.
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

Ran into a pair of include-what-you-use violations (And the tryjobs seem to have
been run without my previously landed patch applied, oddly).  Adding headers
where necessary any trying again.

[commit-bot: I haz the power, 2012-06-19 13:20:45]

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

[commit-bot: I haz the power, 2012-06-19 13:30:07]

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

[commit-bot: I haz the power, 2012-06-19 14:58:37]

Change committed as 142953