NetLogEventParameter to Callback refactoring 5.

NetLogEventParameter to Callback refactoring 5.
Get rid of all uses of NetLogEventParameters in net/http.

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

[eroman, 2012-06-13 18:44:06]

lgtm

http://codereview.chromium.org/10534129/diff/1/net/http/http_stream_factory_i...
File net/http/http_stream_factory_impl_job.cc (right):

http://codereview.chromium.org/10534129/diff/1/net/http/http_stream_factory_i...
net/http/http_stream_factory_impl_job.cc:48: dict->SetString("original_url",
original_url->spec());
This is different, is that intentional?

Before it was using url.GetOrigin().spec() -- in other words, stripping off the
path. 

Whereas now it is printing the entire URL.

http://codereview.chromium.org/10534129/diff/1/net/http/http_stream_factory_i...
net/http/http_stream_factory_impl_job.cc:49: dict->SetString("url",
url->spec());
ditto.

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

*sigh*.  My week of stupid mistakes.

http://codereview.chromium.org/10534129/diff/1/net/http/http_stream_factory_i...
File net/http/http_stream_factory_impl_job.cc (right):

http://codereview.chromium.org/10534129/diff/1/net/http/http_stream_factory_i...
net/http/http_stream_factory_impl_job.cc:48: dict->SetString("original_url",
original_url->spec());
On 2012/06/13 18:44:06, eroman wrote:
> This is different, is that intentional?
> 
> Before it was using url.GetOrigin().spec() -- in other words, stripping off
the
> path. 
> 
> Whereas now it is printing the entire URL.

Fixed.

http://codereview.chromium.org/10534129/diff/1/net/http/http_stream_factory_i...
net/http/http_stream_factory_impl_job.cc:49: dict->SetString("url",
url->spec());
On 2012/06/13 18:44:06, eroman wrote:
> ditto.

Fixed.

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

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

[commit-bot: I haz the power, 2012-06-13 23:14:43]

Change committed as 142009