PlzNavigate: fix RenderViewImplTest.OnNavigationHttpPost

PlzNavigate: fix RenderViewImplTest.OnNavigationHttpPost

This CL fixes RenderViewImplTest.OnNavigationHttpPost when PlzNavigate is
enabled by having RenderFrameImpl::NavigateInternal chose to use LoadDataURL
instead of WebLocalFrame::load in the same cases as what the current
architecture is doing.

It also ensures that the WebURLLoader will not replace the ResourceResponse it
receives with one provided by the browser when loading a data url. In this
case, the browser has no response to provide. so we should use the one
generated by the HandleDataURL task.

BUG=475027
Committed: https://crrev.com/269f92ce7d43d77d11674ea7898b6d826eb579d5
Cr-Commit-Position: refs/heads/master@{#398008}

[clamy, 2016-06-02 17:23:20]

Description was changed from

==========
PlzNavigate: fix RenderViewImplTest.OnNavigationHttpPost

This CL fixes RenderViewImplTest.OnNavigationHttpPost when PlzNavigate is
enabled by having RenderFrameImpl::NavigateInternal chose to use LoadDataURL
instead of WebLocalFrame::load in the same cases as what the current
architecture is doing.

It also ensures that the WebURLLoader will not replace the ResourceResponse it
receives with one provided by the browser when loading a data url. In this
case, the browser has no response to provide. so we should use the one
generated by the HandleDataURL task.

BUG=475027
==========

to

==========
PlzNavigate: fix RenderViewImplTest.OnNavigationHttpPost

This CL fixes RenderViewImplTest.OnNavigationHttpPost when PlzNavigate is
enabled by having RenderFrameImpl::NavigateInternal chose to use LoadDataURL
instead of WebLocalFrame::load in the same cases as what the current
architecture is doing.

It also ensures that the WebURLLoader will not replace the ResourceResponse it
receives with one provided by the browser when loading a data url. In this
case, the browser has no response to provide. so we should use the one
generated by the HandleDataURL task.

BUG=475027
==========

[clamy, 2016-06-02 17:23:20]

clamy@chromium.org changed reviewers:
+ nasko@chromium.org

[clamy, 2016-06-02 17:36:35]

@nasko:PTAL

[nasko, 2016-06-02 22:11:07]

LGTM

[clamy, 2016-06-06 11:30:02]

The CQ bit was checked by clamy@chromium.org

[clamy, 2016-06-06 11:30:02]

The patchset sent to the CQ was uploaded after l-g-t-m from nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/2037733002/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2016-06-06 11:30:14]

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

[commit-bot: I haz the power, 2016-06-06 12:24:41]

Description was changed from

==========
PlzNavigate: fix RenderViewImplTest.OnNavigationHttpPost

This CL fixes RenderViewImplTest.OnNavigationHttpPost when PlzNavigate is
enabled by having RenderFrameImpl::NavigateInternal chose to use LoadDataURL
instead of WebLocalFrame::load in the same cases as what the current
architecture is doing.

It also ensures that the WebURLLoader will not replace the ResourceResponse it
receives with one provided by the browser when loading a data url. In this
case, the browser has no response to provide. so we should use the one
generated by the HandleDataURL task.

BUG=475027
==========

to

==========
PlzNavigate: fix RenderViewImplTest.OnNavigationHttpPost

This CL fixes RenderViewImplTest.OnNavigationHttpPost when PlzNavigate is
enabled by having RenderFrameImpl::NavigateInternal chose to use LoadDataURL
instead of WebLocalFrame::load in the same cases as what the current
architecture is doing.

It also ensures that the WebURLLoader will not replace the ResourceResponse it
receives with one provided by the browser when loading a data url. In this
case, the browser has no response to provide. so we should use the one
generated by the HandleDataURL task.

BUG=475027
==========

[commit-bot: I haz the power, 2016-06-06 12:24:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-06 12:26:10]

Description was changed from

==========
PlzNavigate: fix RenderViewImplTest.OnNavigationHttpPost

This CL fixes RenderViewImplTest.OnNavigationHttpPost when PlzNavigate is
enabled by having RenderFrameImpl::NavigateInternal chose to use LoadDataURL
instead of WebLocalFrame::load in the same cases as what the current
architecture is doing.

It also ensures that the WebURLLoader will not replace the ResourceResponse it
receives with one provided by the browser when loading a data url. In this
case, the browser has no response to provide. so we should use the one
generated by the HandleDataURL task.

BUG=475027
==========

to

==========
PlzNavigate: fix RenderViewImplTest.OnNavigationHttpPost

This CL fixes RenderViewImplTest.OnNavigationHttpPost when PlzNavigate is
enabled by having RenderFrameImpl::NavigateInternal chose to use LoadDataURL
instead of WebLocalFrame::load in the same cases as what the current
architecture is doing.

It also ensures that the WebURLLoader will not replace the ResourceResponse it
receives with one provided by the browser when loading a data url. In this
case, the browser has no response to provide. so we should use the one
generated by the HandleDataURL task.

BUG=475027

Committed: https://crrev.com/269f92ce7d43d77d11674ea7898b6d826eb579d5
Cr-Commit-Position: refs/heads/master@{#398008}
==========

[commit-bot: I haz the power, 2016-06-06 12:26:11]

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