Closed Bug 1543720 Opened 6 years ago Closed 3 years ago

Enable the Storage Access API on GeckoView

Tracking

(geckoview66 wontfix, firefox-esr60 wontfix, firefox66 wontfix, firefox67 wontfix, firefox67.0.1 wontfix, firefox68 wontfix, firefox69 wontfix, firefox70 wontfix, firefox92 fixed)

Status:

RESOLVED FIXED

Milestone:

92 Branch

Tracking Flags:

Tracking

Status

geckoview66

---

wontfix

firefox-esr60

---

wontfix

firefox66

---

wontfix

firefox67

---

wontfix

firefox67.0.1

---

wontfix

firefox68

---

wontfix

firefox69

---

wontfix

firefox70

---

wontfix

firefox92

---

fixed

People

(Reporter: ehsan.akhgari, Assigned: bugzilla)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [geckoview:m87][geckoview:m88][geckoview:m89][geckoview:m90])

Attachments

(3 files)

translate_internals_detect_logs_dump.json 4 years ago u682424 2 bytes, patch		Details \| Diff \| Splinter Review
Bug 1543720 - Add support for Storage Access API in GeckoView. r=aklotz!,#geckoview-reviewers! 3 years ago Dylan Roeh (:droeh) (he/him) 48 bytes, text/x-phabricator-request		Details \| Review
Bug 1543720: Remove wpt Android exceptions for storage-access-api; r=jgraham 3 years ago (No longer employed by Mozilla) Aaron Klotz 48 bytes, text/x-phabricator-request		Details \| Review

(no longer active)

Reporter

Description

•

6 years ago

Fenix would like to use our cookie restrictions feature (part of Enhanced Tracking Protection). The Storage Access API is part of that feature and we need to enable it on GeckoView.

(no longer active)

Reporter

Updated

•

6 years ago

Blocks: 1469714

(no longer active)

Reporter

Updated

•

6 years ago

Depends on: 1543721

(no longer active)

Reporter

Updated

•

6 years ago

Type: defect → enhancement

Chris Peterson [:cpeterson]

Updated

•

6 years ago

status-firefox66: --- → wontfix

status-firefox67: --- → affected

status-firefox68: --- → affected

OS: All → Android

Chris Peterson [:cpeterson]

Comment 1

•

6 years ago

Fenix UX issue:
https://github.com/mozilla-mobile/fenix/issues/1749

Vesta says Enhanced TP is not necessary for Fenix MVP.

status-firefox67: affected → fix-optional

status-firefox68: affected → fix-optional

Priority: -- → P2

See Also: → https://github.com/mozilla-mobile/fenix/issues/1749

Whiteboard: [geckoview:fenix:p2]

Chris Peterson [:cpeterson]

Updated

•

6 years ago

status-firefox67: fix-optional → wontfix

status-firefox67.0.1: --- → wontfix

Whiteboard: [geckoview:fenix:p2] → [geckoview:fenix:my]

Chris Peterson [:cpeterson]

Updated

•

6 years ago

status-firefox-esr60: --- → wontfix

status-geckoview66: --- → wontfix

Whiteboard: [geckoview:fenix:my] → [geckoview:fenix:m7]

Chris Peterson [:cpeterson]

Comment 2

•

5 years ago

Deferring this bug from Fenix's M7 (July) milestone to the M8 backlog for later in Q3.

status-firefox69: --- → affected

Whiteboard: [geckoview:fenix:m7] → [geckoview:fenix:m8]

Chris Peterson [:cpeterson]

Comment 3

•

5 years ago

Moving to [geckoview:fenix:m9] because Fenix doesn't need the Storage Access API for their ETP MVP.

status-firefox68: fix-optional → wontfix

status-firefox69: affected → fix-optional

status-firefox70: --- → fix-optional

Whiteboard: [geckoview:fenix:m8] → [geckoview:fenix:m9]

(no longer active)

Reporter

Updated

•

5 years ago

Depends on: 1573236

(no longer active)

Reporter

Updated

•

5 years ago

Depends on: 1573238

Chris Peterson [:cpeterson]

Comment 4

•

5 years ago

I filed a Fenix issue: "Implement user permission prompt for ETP's Storage Access cookie API"

https://github.com/mozilla-mobile/fenix/issues/4916

status-firefox69: fix-optional → wontfix

See Also: https://github.com/mozilla-mobile/fenix/issues/1749 → https://github.com/mozilla-mobile/fenix/issues/4916

Chris Peterson [:cpeterson]

Updated

•

5 years ago

Rank: 11

Summary: Enable the storage access API on GeckoView → Enable the Storage Access API on GeckoView

Whiteboard: [geckoview:fenix:m9]

Chris Peterson [:cpeterson]

Comment 5

•

5 years ago

We want to work on this in November.

Whiteboard: [geckoview:m1911]

Chris Peterson [:cpeterson]

Updated

•

5 years ago

Whiteboard: [geckoview:m1911]

Johann Hofmann [:johannh]

Updated

•

4 years ago

Blocks: storage-access-experience

Dylan Roeh (:droeh) (he/him)

Updated

•

4 years ago

Assignee: nobody → droeh

Whiteboard: [geckoview:m87]

Emily Toop (:fluffyemily)

Updated

•

4 years ago

Priority: P2 → P1

Emily Toop (:fluffyemily)

Updated

•

4 years ago

Whiteboard: [geckoview:m87] → [geckoview:m87][geckoview:m88]

Dylan Roeh (:droeh) (he/him)

Updated

•

4 years ago

Blocks: 1654832

Emily Toop (:fluffyemily)

Updated

•

4 years ago

Whiteboard: [geckoview:m87][geckoview:m88] → [geckoview:m87][geckoview:m88][geckoview:m89]

u682424

Comment 6

•

4 years ago

Attached patch translate_internals_detect_logs_dump.json — Details — Splinter Review

Flags: needinfo?(lambeaceh22)

Dylan Roeh (:droeh) (he/him)

Updated

•

4 years ago

Whiteboard: [geckoview:m87][geckoview:m88][geckoview:m89] → [geckoview:m87][geckoview:m88][geckoview:m89][geckoview:m90?]

Emily Toop (:fluffyemily)

Updated

•

4 years ago

Whiteboard: [geckoview:m87][geckoview:m88][geckoview:m89][geckoview:m90?] → [geckoview:m87][geckoview:m88][geckoview:m89][geckoview:m90]

Dylan Roeh (:droeh) (he/him)

Comment 7

•

3 years ago

Attached file Bug 1543720 - Add support for Storage Access API in GeckoView. r=aklotz!,#geckoview-reviewers! — Details

(No longer employed by Mozilla) Aaron Klotz

Assignee

Updated

•

3 years ago

Blocks: 1716091

Pulsebot

Comment 9

•

3 years ago

Pushed by aklotz@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a6ec3acd5daf Add support for Storage Access API in GeckoView. r=aklotz,geckoview-reviewers

Atila Butkovits

Comment 10

•

3 years ago

Backed out for causing failures at dlharness.window.html.

Backout link: https://hg.mozilla.org/integration/autoland/rev/5a65ee9d697a02d20cb4359c573514438d38e5c6

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&selectedTaskRun=MzUcWG7bTrGZ5c3VQ8fpww.0&revision=a6ec3acd5daf3837939903c145553cab2d44cdcd&searchStr=android%2C7.0%2Cx86-64%2Cwebrender%2Copt%2Cweb%2Cplatform%2Ctests%2Ctest-android-em-7.0-x86_64-qr%2Fopt-geckoview-web-platform-tests-e10s%2Cwpt11

Failure log: https://treeherder.mozilla.org/logviewer?job_id=342569221&repo=autoland&lineNumber=4862

(No longer employed by Mozilla) Aaron Klotz

Assignee

Updated

•

3 years ago

Assignee: droeh → aklotz

(No longer employed by Mozilla) Aaron Klotz

Assignee

Comment 11

•

3 years ago

Looks like the WPT test exceptions weren't updated. I'm running a try push to verify.

(No longer employed by Mozilla) Aaron Klotz

Assignee

Comment 12

•

3 years ago

Attached file Bug 1543720: Remove wpt Android exceptions for storage-access-api; r=jgraham — Details

Depends on D116143

Pulsebot

Comment 13

•

3 years ago

Pushed by aklotz@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/53dfbd4776e8 Add support for Storage Access API in GeckoView. r=aklotz,geckoview-reviewers https://hg.mozilla.org/integration/autoland/rev/71451fcc878a Remove wpt Android exceptions for storage-access-api; r=jgraham

Sandor Molnar[:smolnar]

Comment 14

•

3 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/53dfbd4776e8
https://hg.mozilla.org/mozilla-central/rev/71451fcc878a

Status: NEW → RESOLVED

Closed: 3 years ago

status-firefox92: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → 92 Branch

Ryan VanderMeulen [:RyanVM]

Updated

•

3 years ago

status-firefox70: fix-optional → wontfix

Flags: needinfo?(lambeaceh22)

Petru-Mugurel Lingurar [:petru]

Updated

•

2 years ago