Closed
Bug 1467921
Opened 6 years ago
Closed 6 years ago
Disable Web Push
Categories
(GeckoView :: General, enhancement, P1)
Tracking
(firefox61 wontfix, firefox62 fixed, firefox63 fixed)
RESOLVED
FIXED
mozilla63
People
(Reporter: snorp, Assigned: snorp)
References
Details
Attachments
(1 file)
(deleted),
text/x-review-board-request
|
jchen
:
review+
lizzard
:
approval-mozilla-beta+
|
Details |
It doesn't work right now and causes hangs if you try to use it.
Comment hidden (mozreview-request) |
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8984608 [details]
Bug 1467921 - Disable Web Push in GeckoView
https://reviewboard.mozilla.org/r/250492/#review256940
Attachment #8984608 -
Flags: review?(nchen) → review+
Comment 3•6 years ago
|
||
Assigning this bug to James because he's already working on it.
Assignee: nobody → snorp
status-firefox61:
--- → wontfix
status-firefox62:
--- → affected
OS: Unspecified → Android
Priority: -- → P1
Pushed by jwillcox@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f55e78d65936
Disable Web Push in GeckoView r=jchen
Comment 5•6 years ago
|
||
Backed out for xpcshell failures
Backout link: https://hg.mozilla.org/integration/mozilla-inbound/rev/d5f39774d8f1f37814824016ee716846edae4df5
Push link: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=f55e78d6593656aff9f61a684bfbc7ba1f3ce264
Log link1: https://treeherder.mozilla.org/logviewer.html#?job_id=185433695&repo=mozilla-inbound&lineNumber=1885
Log link2: https://treeherder.mozilla.org/logviewer.html#?job_id=185430765&repo=mozilla-inbound&lineNumber=2065
Flags: needinfo?(snorp)
Pushed by jwillcox@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8bf2b990c788
Disable Web Push in GeckoView r=jchen
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(snorp)
Comment 7•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Comment 8•6 years ago
|
||
bugherder |
Comment 9•6 years ago
|
||
James is this something we should uplift to beta or can we just let it ride the trains?
Flags: needinfo?(snorp)
Assignee | ||
Comment 11•6 years ago
|
||
Comment on attachment 8984608 [details]
Bug 1467921 - Disable Web Push in GeckoView
Approval Request Comment
[Feature/Bug causing the regression]: GeckoView
[User impact if declined]: Hangs in GeckoView
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: There isn't really a Nightly
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: Bug 1467919
[Is the change risky?]: No
[Why is the change risky/not risky?]: Only changes a pref for GeckoView
[String changes made/needed]: None
Attachment #8984608 -
Flags: approval-mozilla-beta?
Can you request uplift in bug 1467919 too? Thanks!
Flags: needinfo?(snorp)
Comment on attachment 8984608 [details]
Bug 1467921 - Disable Web Push in GeckoView
Should only affect GeckoView, let's uplift for beta 7.
Attachment #8984608 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 14•6 years ago
|
||
bugherder uplift |
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(snorp)
Updated•6 years ago
|
Product: Firefox for Android → GeckoView
Updated•6 years ago
|
Version: Firefox 59 → 59 Branch
Updated•6 years ago
|
Target Milestone: Firefox 63 → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•