net: Add namespace net to the remaining files under url_request directory.

net: Add namespace net to the remaining files under url_request directory.

It just adds the 'namespace net' to these files and a typedef for them, because there are many
entries to fix in one pass. They will be fixed later.

BUG=64263
TEST=trybots

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

[tfarina, 2011-01-04 17:38:53]

Please, take a look.

[willchan no longer on Chromium, 2011-01-04 18:36:26]

LGTM

On Tue, Jan 4, 2011 at 9:38 AM, <tfarina@chromium.org> wrote:

> Reviewers: willchan, wtc,
>
> Message:
> Please, take a look.
>
> Description:
> net: Add namespace net to the remaining files under url_request directory.
>
> It just adds the 'namespace net' to these files and a typedef for them,
> because
> there many
> entries to fix in one pass. They will be fixed later.
>
> BUG=64263
> TEST=trybots
>
> Please review this at http://codereview.chromium.org/6056007/
>
> Affected files:
>  M net/url_request/url_request_context.h
>  M net/url_request/url_request_context.cc
>  M net/url_request/url_request_status.h
>  M net/url_request/url_request_test_job.h
>  M net/url_request/url_request_test_job.cc
>
>
>

[wtc, 2011-01-05 00:19:40]

LGTM.

http://codereview.chromium.org/6056007/diff/6001/net/url_request/url_request_...
File net/url_request/url_request_test_job.cc (right):

http://codereview.chromium.org/6056007/diff/6001/net/url_request/url_request_...
net/url_request/url_request_test_job.cc:19: static std::vector<
scoped_refptr<URLRequestTestJob> > pending_jobs;
Nit: add a g_ prefix to the 'pending_jobs' variable name?

[tfarina, 2011-01-05 01:09:59]

Sent to the trybots.

http://codereview.chromium.org/6056007/diff/6001/net/url_request/url_request_...
File net/url_request/url_request_test_job.cc (right):

http://codereview.chromium.org/6056007/diff/6001/net/url_request/url_request_...
net/url_request/url_request_test_job.cc:19: static std::vector<
scoped_refptr<URLRequestTestJob> > pending_jobs;
On 2011/01/05 00:19:41, wtc wrote:
> Nit: add a g_ prefix to the 'pending_jobs' variable name?

Done.

[wtc, 2011-01-05 01:47:50]

Patch Set 4 LGTM.

[tfarina, 2011-01-05 01:53:51]

On 2011/01/05 01:47:50, wtc wrote:
> Patch Set 4 LGTM.

Please, take a look at patch set 5. Sending again to the trybots.

[wtc, 2011-01-05 02:07:33]

Patch Set 5 LGTM.