Reland "Fix reload after back/forward navigation setting the wrong cache policy.""

Reland "Fix reload after back/forward navigation setting the wrong cache policy.""

This reverts commit c9cfbdff188ab7fc4c74a73c7f1e9e1010efde7d.

BUG=248145
TEST=WebFrameTest.BackToReload

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

[Nate Chapin, 2013-06-18 18:17:55]

On 2013/06/18 18:17:33, Nate Chapin wrote:

There was a spurious registerMockedHttpURLLoad() call that caused this test to
assert in debug.

[abarth-chromium, 2013-06-19 16:47:26]

On 2013/06/18 18:17:55, Nate Chapin wrote:
> On 2013/06/18 18:17:33, Nate Chapin wrote:
> 
> There was a spurious registerMockedHttpURLLoad() call that caused this test to
> assert in debug.

Should we make that a RELEASE_ASSERT so that it's easier to catch?

[abarth-chromium, 2013-06-19 16:47:29]

lgtm

[Nate Chapin, 2013-06-19 16:48:44]

On 2013/06/19 16:47:26, abarth wrote:
> On 2013/06/18 18:17:55, Nate Chapin wrote:
> > On 2013/06/18 18:17:33, Nate Chapin wrote:
> > 
> > There was a spurious registerMockedHttpURLLoad() call that caused this test
to
> > assert in debug.
> 
> Should we make that a RELEASE_ASSERT so that it's easier to catch?

Perhaps, though that question is probably better for someone who knows the
implementation details of registerMockedHttpURLLoad()?

[commit-bot: I haz the power, 2013-06-19 16:49:03]

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

[commit-bot: I haz the power, 2013-06-19 18:31:46]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_layout...

[commit-bot: I haz the power, 2013-06-19 18:33:42]

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

[commit-bot: I haz the power, 2013-06-20 05:51:20]

Change committed as 152752