[e10s] dom/push/test/test_permissions.html | {"action":1,"state":"granted"} - got "denied", expected "granted"

RESOLVED FIXED

Status

()

Core
DOM: Push Notifications
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jgriffin, Unassigned)

Tracking

(Blocks: 1 bug)

unspecified
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(e10s+)

Details

(Whiteboard: dom-triaged)

(Reporter)

Description

2 years ago
On Windows 7, mochitest-push in e10s mode is permafail with:

15 INFO TEST-UNEXPECTED-FAIL | dom/push/test/test_permissions.html | {"action":1,"state":"granted"} - got "denied", expected "granted"
17 INFO TEST-UNEXPECTED-FAIL | dom/push/test/test_permissions.html | {"action":3,"state":"prompt"} - got "denied", expected "prompt"
18 INFO TEST-UNEXPECTED-FAIL | dom/push/test/test_permissions.html | {"action":0,"state":"prompt"} - got "denied", expected "prompt" 

log: https://treeherder.mozilla.org/logviewer.html#?job_id=16372051&repo=try
(Reporter)

Updated

2 years ago
Blocks: 1215233
No longer depends on: 1215233
Hmm, I've seen this fail on OS X with `mach mochitest --e10s`, too. I thought the `SpecialPowers.pushPermissions` wrapper from bug 1191931 would've helped, but apparently not.

Is this all I need to run mochitest-push under e10s? https://hg.mozilla.org/try/rev/6fe342dcb884

Updated

2 years ago
Whiteboard: dom-triaged
(Reporter)

Comment 2

2 years ago
(In reply to Kit Cambridge [:kitcambridge] from comment #1)
> Hmm, I've seen this fail on OS X with `mach mochitest --e10s`, too. I
> thought the `SpecialPowers.pushPermissions` wrapper from bug 1191931
> would've helped, but apparently not.
> 
> Is this all I need to run mochitest-push under e10s?
> https://hg.mozilla.org/try/rev/6fe342dcb884

Yes, that's all that's needed.
Blocks: 984139
tracking-e10s: --- → +
Looks like it's green now:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b0ec9e74fd06&exclusion_profile=false
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

2 years ago
(In reply to Kit Cambridge [:kitcambridge] from comment #3)
> Looks like it's green now:
> 
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=b0ec9e74fd06&exclusion_profile=false

No, that's the non-e10s version. The e10s version is still broken.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Sorry about that. I changed `win_unittest.py` in https://hg.mozilla.org/try/rev/b0ec9e74fd06; what else do I need to do to run the e10s version?
(Reporter)

Comment 6

2 years ago
(In reply to Kit Cambridge [:kitcambridge] from comment #5)
> Sorry about that. I changed `win_unittest.py` in
> https://hg.mozilla.org/try/rev/b0ec9e74fd06; what else do I need to do to
> run the e10s version?

My bad, you did the right thing!
Status: REOPENED → RESOLVED
Last Resolved: 2 years ago2 years ago
Resolution: --- → FIXED
(Reporter)

Updated

2 years ago
Blocks: 1248729
You need to log in before you can comment on or make changes to this bug.