Logging the NTP snippets request when verbose.

Logging the NTP snippets request when verbose.

BUG=621930
Committed: https://crrev.com/c53e56cc3afdbe74e918b83caa1fee5120b7c6c6
Cr-Commit-Position: refs/heads/master@{#401661}

[jkrcal, 2016-06-21 14:32:33]

jkrcal@chromium.org changed reviewers:
+ bauerb@chromium.org

[jkrcal, 2016-06-21 14:32:33]

PTAL

[Bernhard Bauer, 2016-06-21 16:33:06]

https://codereview.chromium.org/2083933004/diff/1/components/ntp_snippets/ntp...
File components/ntp_snippets/ntp_snippets_fetcher.cc (right):

https://codereview.chromium.org/2083933004/diff/1/components/ntp_snippets/ntp...
components/ntp_snippets/ntp_snippets_fetcher.cc:259: VLOG(1) << "Sendind a ntp
snippets request to " << url << ":";
Nit: "Sending", "NTP".

https://codereview.chromium.org/2083933004/diff/1/components/ntp_snippets/ntp...
components/ntp_snippets/ntp_snippets_fetcher.cc:260: VLOG(1) <<
headers.ToString();
You don't need a new VLOG() invocation; just keep the << chain (and align the
operators).

[jkrcal, 2016-06-23 08:21:36]

PTAL, again.

https://codereview.chromium.org/2083933004/diff/1/components/ntp_snippets/ntp...
File components/ntp_snippets/ntp_snippets_fetcher.cc (right):

https://codereview.chromium.org/2083933004/diff/1/components/ntp_snippets/ntp...
components/ntp_snippets/ntp_snippets_fetcher.cc:259: VLOG(1) << "Sendind a ntp
snippets request to " << url << ":";
On 2016/06/21 16:33:05, Bernhard Bauer wrote:
> Nit: "Sending", "NTP".

Done.

https://codereview.chromium.org/2083933004/diff/1/components/ntp_snippets/ntp...
components/ntp_snippets/ntp_snippets_fetcher.cc:260: VLOG(1) <<
headers.ToString();
On 2016/06/21 16:33:05, Bernhard Bauer wrote:
> You don't need a new VLOG() invocation; just keep the << chain (and align the
> operators).

Done (added some new lines, as well).

[Bernhard Bauer, 2016-06-23 11:37:04]

lgtm

[jkrcal, 2016-06-23 17:09:11]

The CQ bit was checked by jkrcal@chromium.org

[commit-bot: I haz the power, 2016-06-23 17:09:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2083933004/20001

[commit-bot: I haz the power, 2016-06-23 18:41:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-23 18:51:34]

Description was changed from

==========
Logging the NTP snippets request when verbose.

BUG=621930
==========

to

==========
Logging the NTP snippets request when verbose.

BUG=621930

Committed: https://crrev.com/c53e56cc3afdbe74e918b83caa1fee5120b7c6c6
Cr-Commit-Position: refs/heads/master@{#401661}
==========

[commit-bot: I haz the power, 2016-06-23 18:51:36]

Patchset 2 (id:??) landed as
https://crrev.com/c53e56cc3afdbe74e918b83caa1fee5120b7c6c6
Cr-Commit-Position: refs/heads/master@{#401661}