DescriptionRevert of OpenURL post data handling. (patchset #9 id:160001 of https://codereview.chromium.org/2004653002/ )
Reason for revert:
Reverting due to https://bugs.chromium.org/p/chromium/issues/detail?id=621473
Original issue's description:
> Passing ResourceRequestBody via FrameHostMsg_OpenURL_Params.
>
> This CL adds support for passing ResourceRequestBody (i.e. POST data)
> inside the payload of FrameHostMsg_OpenURL IPC message.
>
> This CL adds |uses_post| and |resource_request_body| to
> FrameHostMsg_OpenURL_Params and plumbs through these 2 values into
> OpenURLParams passed to //content embedder.
>
> This CL wraps-up the work started in 2 earlier CLs:
>
> - https://crrev.com/2038813003 (Making ResourceRequestBody part of
> //content's public API.)
>
> - https://crrev.com/2038233002 (Using ResourceRequestBody as the type of
> HTTP body outside of //content)
>
> After this CL scenarios and tests that requires passing of POST data
> via OpenURL code path should start working:
>
> - HTTP POST requests where decidePolicyForNavigation tells that the
> request should be handled by the //content embedder. This scenario
> is covered by a new CrossSiteTransferTest.PostViaOpenUrlMsg content
> browser test added in this CL.
>
> - HTTP POST requests handled by RenderFrame::navigate (i.e. form
> submissions targeting remote frame). This scenario is covered
> by a layout test added in a recent CL:
> http/tests/navigation/form-targets-cross-site-frame-post.html
>
> BUG=344348
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
>
> Committed: https://crrev.com/bb2101b8d45775fef16a0ce4af1ae71f35749e76
> Cr-Commit-Position: refs/heads/master@{#400442}
TBR=creis@chromium.org,nasko@chromium.org,dcheng@chromium.org,lukasza@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=344348
Patch Set 1 #Messages
Total messages: 6 (3 generated)
|