Closed Bug 1623642 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 22333 - Deduplicate document policy violation report

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox79 fixed)

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 22333 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/22333
Details from upstream follow.

Charlie Hu <chenleihu@google.com> wrote:

Deduplicate document policy violation report

Most image policies can report multiple violations for a single cause(same image not following the rule) during layout, e.g. An unoptimized-lossless-image's bpp(byte per pixel) value can change multiple times if the image is being scaled in an animation.

To avoid unnecessary duplicated reports being generated. This CL adds Hash method for DocumentPolicyViolationReportBody to uniquely identify each report and avoid the duplication by remembering these hash values in ExecutionContext and filter out reports that are in record.

Bug: 924684
Change-Id: I230e32801c77980573b5ed55064d46f94aed3060
Reviewed-on: https://chromium-review.googlesource.com/2078837
WPT-Export-Revision: 2aa06aa29464f351bc9d8120646fe85cfdd4db75

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 2 tests and 1 subtests

Status Summary

Firefox

TIMEOUT: 4

Chrome

OK : 2
PASS : 2

Safari

TIMEOUT: 4

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/document-policy/font-display/font-display-document-policy-reporting.tentative.html: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
font-display-late-swap Report Format: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
/document-policy/font-display/font-display-document-policy-report-only.tentative.html: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
font-display-late-swap Report Format: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7d6a86f73cde [wpt PR 22333] - Deduplicate document policy violation report, a=testonly
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
You need to log in before you can comment on or make changes to this bug.