Set speculative bool on dns-preresolves in //content

Set speculative bool on dns-preresolves in //content

This bit is only used to separate DNS requests in histograms.

BUG=650693
Committed: https://crrev.com/9d561b84341f15e7926fcdb5e762c51a996449bc
Cr-Commit-Position: refs/heads/master@{#421245}

[Charlie Harrison, 2016-09-27 16:25:59]

The CQ bit was checked by csharrison@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-27 16:26:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Charlie Harrison, 2016-09-27 16:27:49]

csharrison@chromium.org changed reviewers:
+ juliatuttle@chromium.org

[Charlie Harrison, 2016-09-27 16:27:49]

juliatuttle@, ptal?

[Randy Smith (Not in Mondays), 2016-09-27 16:29:28]

rdsmith@chromium.org changed reviewers:
+ rdsmith@chromium.org

[Randy Smith (Not in Mondays), 2016-09-27 16:29:29]

Drive by comment, mostly driven by curiosity (but I think the answer should be
in the CL description, so I'll let Rietveld "reviewer" me :-}): What's the
user-facing change in behavior expected from this CL?

[Charlie Harrison, 2016-09-27 16:33:40]

Description was changed from

==========
Set speculative bool on dns-preresolves in //content

BUG=650693
==========

to

==========
Set speculative bool on dns-preresolves in //content

This bit is only used to separate DNS requests in histograms.

BUG=650693
==========

[Charlie Harrison, 2016-09-27 16:34:12]

Updated the CL description. There aren't any user facing changes in this patch
(speculative dns requests are already lowest priority).

[Randy Smith (Not in Mondays), 2016-09-27 16:36:33]

TY.  LGTM (just to get myself out of the way :-}).

[commit-bot: I haz the power, 2016-09-27 17:05:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-27 17:05:45]

Dry run: This issue passed the CQ dry run.

[Julia Tuttle, 2016-09-27 17:10:16]

lgtm.

[Charlie Harrison, 2016-09-27 17:10:52]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2016-09-27 17:11:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 17:17:09]

Description was changed from

==========
Set speculative bool on dns-preresolves in //content

This bit is only used to separate DNS requests in histograms.

BUG=650693
==========

to

==========
Set speculative bool on dns-preresolves in //content

This bit is only used to separate DNS requests in histograms.

BUG=650693
==========

[commit-bot: I haz the power, 2016-09-27 17:17:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 17:21:01]

Description was changed from

==========
Set speculative bool on dns-preresolves in //content

This bit is only used to separate DNS requests in histograms.

BUG=650693
==========

to

==========
Set speculative bool on dns-preresolves in //content

This bit is only used to separate DNS requests in histograms.

BUG=650693

Committed: https://crrev.com/9d561b84341f15e7926fcdb5e762c51a996449bc
Cr-Commit-Position: refs/heads/master@{#421245}
==========

[commit-bot: I haz the power, 2016-09-27 17:21:02]

Patchset 1 (id:??) landed as
https://crrev.com/9d561b84341f15e7926fcdb5e762c51a996449bc
Cr-Commit-Position: refs/heads/master@{#421245}