Do those test only check for the callback or is there any functionality that is expected to work? We can't really deliver redirect callbacks in the renderer process with PlzNavigate, so unless we are breaking actual functionality, we should mark those tests for deletion once PlzNavigate ships.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/878b2e67ac952b00b2fcafc54867f61b25507923

commit 878b2e67ac952b00b2fcafc54867f61b25507923
Author: ananta <ananta@chromium.org>
Date: Mon Mar 13 22:40:26 2017

PlzNavigate: Fix the http/tests/loading/307-after-303-after-post.html and the http/tests/loading/redirect-methods.html layout tests.

These tests fail because willSendRequest() occurs before didStartProvisionalLoad(). In PlzNavigate willSendRequest is called when
we send a navigation to the browser via BeginNavigation().

Proposed fix is to remember the navigation state in decidePolicyForNavigation() and invoke BeginNavigation() when we
receive the didStartProvisionalLoad() notification.

We pass the WebURLRequest in WebFrameClient::didStartProvisionalLoad()

The other change is to remember the navigation state in a PendingNavigationInfo structure in
RenderFrameImpl. This is populated in decidePolicyForNavigation() and released in didStartProvisionalLoad().

BUG= 647698 , 625765 

Review-Url: https://codereview.chromium.org/2734633002
Cr-Commit-Position: refs/heads/master@{#456510}

[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/content/renderer/render_frame_impl.cc
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/content/renderer/render_frame_impl.h
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/content/shell/test_runner/web_frame_test_client.cc
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/content/shell/test_runner/web_frame_test_client.h
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/content/shell/test_runner/web_frame_test_proxy.h
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/third_party/WebKit/Source/core/frame/LocalFrameClient.h
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/third_party/WebKit/Source/core/loader/EmptyClients.h
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/third_party/WebKit/Source/core/loader/FrameLoader.cpp
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/third_party/WebKit/Source/web/LocalFrameClientImpl.cpp
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/third_party/WebKit/Source/web/LocalFrameClientImpl.h
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/third_party/WebKit/Source/web/tests/WebDocumentSubresourceFilterTest.cpp
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/third_party/WebKit/Source/web/tests/WebFrameTest.cpp
[modify] https://crrev.com/878b2e67ac952b00b2fcafc54867f61b25507923/third_party/WebKit/public/web/WebFrameClient.h