predictors: Track whether resources have validators, and require validation.

predictors: Track whether resources have validators, and require validation.

This adds to the database whether resources have validators, and are
never fresh because they require revalidation each time. This will be
used later to adjust the priority.

Also adds tests for SummarizeResponse, and unifies (and fixes) the
content type handling in it.

BUG=631966
Committed: https://crrev.com/c65a3bf951519f90b6be30a1bded64a7001e59d0
Cr-Commit-Position: refs/heads/master@{#413202}

[Benoit L, 2016-08-18 16:28:57]

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

[commit-bot: I haz the power, 2016-08-18 16:29:25]

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

[Benoit L, 2016-08-18 16:35:15]

lizeb@chromium.org changed reviewers:
+ pasko@chromium.org

[Benoit L, 2016-08-19 15:57:40]

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

[commit-bot: I haz the power, 2016-08-19 15:57:54]

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

[pasko, 2016-08-19 16:17:42]

lgtm with bots passing

https://codereview.chromium.org/2260573002/diff/20001/chrome/browser/predicto...
File chrome/browser/predictors/resource_prefetch_predictor_unittest.cc (right):

https://codereview.chromium.org/2260573002/diff/20001/chrome/browser/predicto...
chrome/browser/predictors/resource_prefetch_predictor_unittest.cc:984:
response_info.headers = MakeResponseHeaders
does it compile without opening paren?

[Benoit L, 2016-08-19 16:26:27]

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

[commit-bot: I haz the power, 2016-08-19 16:26:43]

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

[Benoit L, 2016-08-19 16:34:21]

Thanks!
And sorry for the missing "(" :-/

[commit-bot: I haz the power, 2016-08-19 18:15:35]

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

[commit-bot: I haz the power, 2016-08-19 18:15:35]

Dry run: This issue passed the CQ dry run.

[Benoit L, 2016-08-19 18:58:18]

The CQ bit was checked by lizeb@chromium.org

[Benoit L, 2016-08-19 18:58:19]

The patchset sent to the CQ was uploaded after l-g-t-m from pasko@chromium.org
Link to the patchset: https://codereview.chromium.org/2260573002/#ps60001
(title: "Make Visual Studio happy.")

[commit-bot: I haz the power, 2016-08-19 18:59:06]

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-08-19 19:04:06]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-19 19:08:29]

Description was changed from

==========
predictors: Track whether resources have validators, and require validation.

This adds to the database whether resources have validators, and are
never fresh because they require revalidation each time. This will be
used later to adjust the priority.

Also adds tests for SummarizeResponse, and unifies (and fixes) the
content type handling in it.

BUG=631966
==========

to

==========
predictors: Track whether resources have validators, and require validation.

This adds to the database whether resources have validators, and are
never fresh because they require revalidation each time. This will be
used later to adjust the priority.

Also adds tests for SummarizeResponse, and unifies (and fixes) the
content type handling in it.

BUG=631966

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

[commit-bot: I haz the power, 2016-08-19 19:08:29]

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