[wpt-sync] Sync PR 33033 - [FedCM] Make privacy policy url required
Categories
(Core :: DOM: Security, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox100 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream][domsecurity-backlog])
Sync web-platform-tests PR 33033 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/33033
Details from upstream follow.
Nicolás Peña Moreno <npm@chromium.org> wrote:
[FedCM] Make privacy policy url required
Currently we allow no privacy policy url to be specified in the client
metadata. This CL changes that by surfacing an error if the JSON parser
returns an empty value for |privacy_policy_url|. Note that currently we
are not sanity checking the strings themselves to ensure the URLs
provided are actually valid URLs, and this CL does not change that. In
order to properly surface the problem about missing privacy policy, we
add a specific console error message. We also add a new value to the
FederatedAuthRequestIssueReason so that a similar message can be
displayed in the Issues panel.Bug: 1293913
Change-Id: I5dbb6e6a1005db33dccafb7977fb7cced8f871d3
Reviewed-on: https://chromium-review.googlesource.com/3499232
WPT-Export-Revision: 689a9935eeea65249fd0edd1c6f196da181b01b1
Assignee | ||
Comment 1•2 years ago
|
||
PR 33033 applied with additional changes from upstream: a5996ace0256d702602318cd824c816066eb792b
Assignee | ||
Updated•2 years ago
|
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 2•2 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=None
Updated•2 years ago
|
Assignee | ||
Comment 3•2 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=6b72eef721e355d2ab838e7d5fdcdb426c14276d
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/edce89881ec6 [wpt PR 33033] - [FedCM] Make privacy policy url required, a=testonly
Assignee | ||
Comment 5•2 years ago
|
||
Test result changes from PR not available.
Comment 6•2 years ago
|
||
bugherder |
Description
•