[wpt-sync] Sync PR 19308 - Fix for AllowPaymentRequest Bug(972089)
Categories
(Testing :: web-platform-tests, task, P4)
Tracking
(firefox72 fixed)
Tracking | Status | |
---|---|---|
firefox72 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 19308 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/19308
Details from upstream follow.
Charlie Hu <chenleihu@google.com> wrote:
Fix for AllowPaymentRequest Bug(972089)
The solution is to snapshot the value of frame policy at the beginning of
a local_frame_client navigation and use it during document initialization.
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::InitializeFeaturePolicyMore detailed information could be found on crbug page.
Bug: 972089
Change-Id: I694aa46b6e8e04bebc86b0e7886b856a9a302905Reviewed-on: https://chromium-review.googlesource.com/1796119
WPT-Export-Revision: 2e711f96aac182e9796ba1471c69fd8b520acd8f
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=1931847f146d2f4cb86d161ecc39cfe0516d4aaa
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=23bf0b73fe017851c7f2a03de027f209cd5ffeb1
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=021378e232ee13f57cab2abe31e115e2f43b44ab
Assignee | ||
Comment 4•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=eaea561dc7c0f2077dca1eac9e9d015656b3ac2d
Assignee | ||
Comment 5•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=aed66a32bcfafd3b62a3f5775f1c243159c64996
Assignee | ||
Comment 6•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=8a713c15ccb94d9d8804fe04361c7cc749e7f990
Assignee | ||
Comment 7•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=caa5830dbc35e9a6c636c826837256796b0fe08c
Assignee | ||
Comment 8•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=870cc196d491e504bbaa81860e6af015d9a4d084
Assignee | ||
Comment 9•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=b91dba7bf1c2ba94b3bfbe09eeccdc6502281b4b
Assignee | ||
Comment 10•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=9caaff8fb8ef2cea0a28c0cda80229e9e260a6d4
Assignee | ||
Comment 11•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=f4cf45792e7d1fabe1379147e9d3c609ac159ef5
Assignee | ||
Comment 12•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=9b9d691b48fd204fb5aa51e0110704303197f5b2
Assignee | ||
Comment 13•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=250b8cfa3ab33e11621ac20da4e3ae5626244622
Assignee | ||
Comment 14•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=09253c3f757763394dd8ee1cd853b2bcb5518027
Assignee | ||
Comment 15•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=d1d788f93d41e759dc3e74f6a44b0cc2fdf852c4
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 16•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=e59dfe224939fd91bda1a92e5529420a4f047e43
Assignee | ||
Comment 17•5 years ago
|
||
GitHub CI Results
wpt.fyi PR Results Base Results
Ran 1 tests and 4 subtests
Firefox
OK : 1
PASS: 2
FAIL: 2
Chrome
OK : 1
PASS: 3
FAIL: 1
Safari
OK : 1
PASS: 3
FAIL: 1
Firefox-only failures
/service-workers/service-worker/sandboxed-iframe-navigator-serviceworker.https.html
Accessing navigator.serviceWorker in sandboxed iframe should throw.: Firefox: FAIL
Gecko CI Results
Ran 1 tests and 4 subtests
OK : 1
PASS : 2
FAIL : 2
Comment 18•4 years ago
|
||
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/03831919b661 [wpt PR 19308] - Capture frame policy at the beginning of navigation, a=testonly
Comment 19•4 years ago
|
||
bugherder |
Description
•