tree 7315d78e4e06adf9154e1a182df460b95e64f44a
parent c7ee243f4fb817b9c9e8c58e9e35b9a9dfb39f20
author Charlie Hu <chenleihu@google.com> 1573571258 -0800
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1573575160 -0500

Capture frame policy at the beginning of navigation

Changing the frame policy when a navigation is already in-flight must not affect
the in-flight navigation because of
https://html.spec.whatwg.org/multipage/iframe-embed-object.html#attr-iframe-allow

The snapshot is made at the beginning of a local_frame_client navigation and used
on document initialization(after commit of navigation). If the frame owner is a
remote one, the old value is used.

- capture frame policy at the start of navigation
    - local_frame_client_impl
    - NavigationInfo
    - render_frame_impl.BeginNavigationInternal

- pass it from renderer process to browser process through
  CommonNavigationParams in navigation_params.mojom
  - MakeCommonNavigationParams

- use value in CommonNavigationParams to init navigation_params
  - render_frame_impl.CommitNavigationInternal
  - web_navigation_params
    -> DocumentInit
    -> Document::InitializeFeaturePolicy

More detailed information could be found on crbug page.

A document_loader_test is added to further strengthen the test on frame
policy value used. Verified that the added test would fail without the
patch.

Bug: 972089
Change-Id: I694aa46b6e8e04bebc86b0e7886b856a9a302905
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796119
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Reviewed-by: Ian Clelland <iclelland@chromium.org>
Commit-Queue: Charlie Hu <chenleihu@google.com>
Cr-Commit-Position: refs/heads/master@{#714475}