Reland "Allow cache reuse of some requests with Cache-Control headers"

Reland "Allow cache reuse of some requests with Cache-Control headers"

Traditionally, we have carefully handled Cache-Control headers that were set on a ResourceResponse, but have pretended Cache-Control doesn't exist on ResourceRequests. As of http://src.chromium.org/viewvc/blink?view=rev&rev=165603, we do the smarter naive thing, and never allow resources to be reused from MemoryCache if the ResourceRequest includes a Cache-Control header.

This patch moves ResourceResponse's Cache-Control parsing to HTTPParsers and handles Cache-Control headers consistently, whether the header appears on a ResourceRequest or a ResourceResponse.

BUG=

This relands r171178.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=171375

[Nate Chapin, 2014-04-10 18:43:12]

Diff the two patches to see the fix for reland.

[abarth-chromium, 2014-04-10 22:18:57]

The only delta is in the test...

[Nate Chapin, 2014-04-10 22:20:22]

On 2014/04/10 22:18:57, abarth wrote:
> The only delta is in the test...

Correct. The test is the only part that compile-broke :D

[Nate Chapin, 2014-04-10 22:21:16]

On 2014/04/10 22:20:22, Nate Chapin wrote:
> On 2014/04/10 22:18:57, abarth wrote:
> > The only delta is in the test...
> 
> Correct. The test is the only part that compile-broke :D

This patch ran afoul of
https://code.google.com/p/chromium/issues/detail?id=361852 on the first landing
attempt.

[abarth-chromium, 2014-04-10 22:24:36]

LGTM

[Nate Chapin, 2014-04-10 22:25:41]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2014-04-10 22:25:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/japhet@chromium.org/232993004/20001

[commit-bot: I haz the power, 2014-04-10 23:07:20]

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

[commit-bot: I haz the power, 2014-04-10 23:07:20]

Try jobs failed on following builders:
  tryserver.blink on mac_blink_compile_dbg
  tryserver.blink on mac_blink_rel

[Nate Chapin, 2014-04-11 16:31:50]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2014-04-11 16:31:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/japhet@chromium.org/232993004/20001

[commit-bot: I haz the power, 2014-04-11 17:00:30]

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

[commit-bot: I haz the power, 2014-04-11 17:00:30]

Try jobs failed on following builders:
  tryserver.blink on mac_blink_compile_dbg

[Nate Chapin, 2014-04-11 20:31:30]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2014-04-11 20:31:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/japhet@chromium.org/232993004/40001

[commit-bot: I haz the power, 2014-04-11 21:42:30]

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

[commit-bot: I haz the power, 2014-04-11 21:42:30]

Try jobs failed on following builders:
  tryserver.blink on win_blink_rel

[Nate Chapin, 2014-04-11 21:44:02]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2014-04-11 21:44:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/japhet@chromium.org/232993004/40001

[commit-bot: I haz the power, 2014-04-11 22:19:44]

Change committed as 171375