CapturingNetLog - remove maximum entries constructor argument.

CapturingNetLog - remove maximum entries constructor argument.
Also replace with a NULL NetLog in a couple callsites where it
wasn't being used.

R=eroman@chromium.org
BUG=none

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

[mmenke, 2012-06-08 16:09:34]

I also removed unneeded use of CapturingNetLogs in a couple of tests, but left
those in the http_network_transaction_* tests alone, since a large number of
tests in those file do use them, and it didn't really seem like it would
accomplish much.  More to the point, keeping the two files in sync really seemed
like a waste of time.

[mmenke, 2012-06-08 16:10:34]

On 2012/06/08 16:09:34, Matt Menke wrote:
> I also removed unneeded use of CapturingNetLogs in a couple of tests, but left
> those in the http_network_transaction_* tests alone, since a large number of
> tests in those file do use them, and it didn't really seem like it would
> accomplish much.  More to the point, keeping the two files in sync really
seemed
> like a waste of time.

Oh, and this includes the other CL's changes in capturing_net_log, but not they
gyp.  Figure I'll land the other one first, wait a couple cycles, and then land
this.

[eroman, 2012-06-08 18:24:41]

lgtm

http://codereview.chromium.org/10546071/diff/1/net/base/capturing_net_log.h
File net/base/capturing_net_log.h (left):

http://codereview.chromium.org/10546071/diff/1/net/base/capturing_net_log.h#o...
net/base/capturing_net_log.h:31: // TODO(mmenke):  Move CapturingNetLog to
net_unittests project, once the CL
Oh I see, you removed the comment as part of this CL.

http://codereview.chromium.org/10546071/diff/1/net/base/capturing_net_log.h
File net/base/capturing_net_log.h (right):

http://codereview.chromium.org/10546071/diff/1/net/base/capturing_net_log.h#n...
net/base/capturing_net_log.h:69: enum { kUnbounded = -1 };
Delete

[mmenke, 2012-06-08 18:28:11]

Thanks.

http://codereview.chromium.org/10546071/diff/1/net/base/capturing_net_log.h
File net/base/capturing_net_log.h (left):

http://codereview.chromium.org/10546071/diff/1/net/base/capturing_net_log.h#o...
net/base/capturing_net_log.h:31: // TODO(mmenke):  Move CapturingNetLog to
net_unittests project, once the CL
On 2012/06/08 18:24:41, eroman wrote:
> Oh I see, you removed the comment as part of this CL.

Just forgot to copy it to the other one (Was thinking one CL, then decided two).
 Now done.

http://codereview.chromium.org/10546071/diff/1/net/base/capturing_net_log.h
File net/base/capturing_net_log.h (right):

http://codereview.chromium.org/10546071/diff/1/net/base/capturing_net_log.h#n...
net/base/capturing_net_log.h:69: enum { kUnbounded = -1 };
On 2012/06/08 18:24:41, eroman wrote:
> Delete

Thanks, done.

[commit-bot: I haz the power, 2012-06-08 22:09:36]

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

[commit-bot: I haz the power, 2012-06-08 22:57:34]

Change committed as 141315