Closed
Bug 1838048
Opened 2 years ago
Closed 2 years ago
Create a Test for validating the nsIIdentityCredentialPromptService.showAccountPrompt is handled in GeckoView
Categories
(GeckoView :: General, task, P1)
Tracking
(firefox117 fixed)
RESOLVED
FIXED
117 Branch
| Tracking | Status | |
|---|---|---|
| firefox117 | --- | fixed |
People
(Reporter: titouan, Assigned: titouan)
References
(Blocks 1 open bug)
Details
(Whiteboard: [foundation][fxdroid])
Attachments
(1 file)
|
48 bytes,
text/x-phabricator-request
|
Details |
We would like to add tests for validating that GeckoView is handling the showAccountPrompt for FedCM, in desktop we are testing it by calling directly the IdentityCredentialPromptService module but in GeckoView, at least most of the prompt tests don't have access to privileged APIs, we may need to explore both options of creating a local website or trying to access the browsers APIs.
Updated•2 years ago
|
Updated•2 years ago
|
Assignee: nobody → tthibaud
Severity: -- → N/A
Priority: -- → P1
| Assignee | ||
Comment 1•2 years ago
|
||
Pushed by tthibaud@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b2154b397ef7
Create a Test for validating the nsIIdentityCredentialPromptService.showAccountPrompt is handled in GeckoView r=geckoview-reviewers,amejiamarmol
Comment 3•2 years ago
|
||
Backed out for geckoview failures on onSelectIdentityCredentialAccount.
Failure log: https://treeherder.mozilla.org/logviewer?job_id=421591487&repo=autoland
Backout link: https://hg.mozilla.org/integration/autoland/rev/089e01b7b4406e2fedce4b8cbdd9a0e1709f4e47
Flags: needinfo?(tthibaud)
Pushed by tthibaud@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7313b6429e48
Create a Test for validating the nsIIdentityCredentialPromptService.showAccountPrompt is handled in GeckoView r=geckoview-reviewers,amejiamarmol
Comment 5•2 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox117:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
| Assignee | ||
Updated•2 years ago
|
Flags: needinfo?(tthibaud)
You need to log in
before you can comment on or make changes to this bug.
Description
•