1984443 - Storage-Access-Header: Handle ABA case

Status: RESOLVED FIXED

(Core :: Privacy: Anti-Tracking, enhancement, P2)

Description

[Manuel Bucher [:manuel]]

There are two web-platform tests: https://wpt.fyi/results/storage-access-api/storage-access-headers.tentative.https.sub.window.html?label=master&label=experimental&aligned

Both with ABA in description. They are not covered by Bug 1968722. Handle them separately.

Comment 1

[Manuel Bucher [:manuel]]

This is the line where I need to add the ABA check: https://searchfox.org/firefox-main/rev/8a50d94240fb8f312d8b061a425ff0755bcbaac4/toolkit/components/antitracking/AntiTrackingUtils.cpp#638

Comment 2

[Manuel Bucher [:manuel]]

Attachment: Bug 1984443 - Storage-Access-Headers: Handle ABA case

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/829b49bfafea
https://hg.mozilla.org/integration/autoland/rev/965be1c9e487
Storage-Access-Headers: Handle ABA case r=timhuang

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/9659ebc5ac4b
https://hg.mozilla.org/integration/autoland/rev/a17c0949b7a3
Revert "Bug 1984443 - Storage-Access-Headers: Handle ABA case r=timhuang" for causing build bustages on AntiTrackingUtils.cpp

Comment 5

[Cosmin Sabou [:CosminS]]

Backed out for causing build bustages on AntiTrackingUtils.cpp

Push with failures

Failure log

Backout link

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/bd856895ca94
https://hg.mozilla.org/integration/autoland/rev/eac8c170d67d
Storage-Access-Headers: Handle ABA case r=timhuang

Comment 7

[Manuel Bucher [:manuel]]

Used deprecated function nsIPrincipal::GetURI.

Comment 8

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/eac8c170d67d