Closed
Bug 1349612
Opened 7 years ago
Closed 7 years ago
Report site issue button disabled in Fennec Nightly and Aurora
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(fennec53+, firefox52 unaffected, firefox-esr52 unaffected, firefox53 fixed, firefox54 verified, firefox55 verified)
VERIFIED
FIXED
Firefox 55
Tracking | Status | |
---|---|---|
fennec | 53+ | --- |
firefox52 | --- | unaffected |
firefox-esr52 | --- | unaffected |
firefox53 | --- | fixed |
firefox54 | --- | verified |
firefox55 | --- | verified |
People
(Reporter: adamopenweb, Assigned: jchen)
References
Details
(Keywords: regression)
Attachments
(1 file)
1.39 KB,
patch
|
jchen
:
review+
lizzard
:
approval-mozilla-aurora+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Steps to reproduce: 1) Open Firefox for Android, Nightly (55) or Aurora (54) 2) Navigate to a website to report a bug eg. https://www.mozilla.org/ 3) Press 3 dot menu button Expected behavior: Report Site Issue button is enabled - can tap it Actual behavior Report Site Issue button is disabled - unable to report bugs
Reporter | ||
Updated•7 years ago
|
Summary: Report site issue button disabled in Nightly and Aurora → Report site issue button disabled in Fennec Nightly and Aurora
Comment 1•7 years ago
|
||
I wonder if mozregression can show what changed here. Unfortunately my phone exploded at dinner tonight... gonna see if I can get the screen repaired at a local shop tomorrow. >_<
Comment 2•7 years ago
|
||
It still happens both on latest nightly(55) and aurora(54).
Comment 3•7 years ago
|
||
mozregression pointed me at Bug 1321418... Jim, does anything come to mind why that changeset would bust the menu being enabled/disabled? The code that controls that is here: https://dxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/WebcompatReporter.js#41-59 (also yeah, I should have written a test for this... >_>)
Blocks: 1321418
Flags: needinfo?(nchen)
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → nchen
Status: NEW → ASSIGNED
Flags: needinfo?(nchen)
Assignee | ||
Comment 4•7 years ago
|
||
I accidentally landed this patch over in bug 1321418, but it really belongs in this bug.
Attachment #8854989 -
Flags: review+
Assignee | ||
Updated•7 years ago
|
Product: Web Compatibility → Firefox for Android
Version: unspecified → Trunk
Assignee | ||
Updated•7 years ago
|
tracking-fennec: --- → ?
status-firefox53:
--- → affected
status-firefox54:
--- → affected
status-firefox55:
--- → affected
Assignee | ||
Comment 5•7 years ago
|
||
Comment on attachment 8854989 [details] [diff] [review] Update menu items using the correct ID (v1) Approval Request Comment [Feature/Bug causing the regression]: Bug 1321418 [User impact if declined]: Addons are not able to update the state of custom menu items; some addons like the webcompat reporter rely on this feature. [Is this code covered by automated tests?]: No [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: No [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: No [Why is the change risky/not risky?]: It's a small patch that only fixes the regression [String changes made/needed]: None
Attachment #8854989 -
Flags: approval-mozilla-beta?
Attachment #8854989 -
Flags: approval-mozilla-aurora?
Updated•7 years ago
|
Comment 6•7 years ago
|
||
(In reply to Jim Chen [:jchen] [:darchons] from comment #4) > I accidentally landed this patch over in bug 1321418, but it really belongs > in this bug. https://hg.mozilla.org/mozilla-central/rev/84abf2ef2fab
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
tracking-fennec: ? → 53+
Comment 7•7 years ago
|
||
Hi :adamopenweb, Can you help check if this issue is fixed in the latest nightly?
Flags: qe-verify+
Flags: needinfo?(astevenson)
Comment 8•7 years ago
|
||
Comment on attachment 8854989 [details] [diff] [review] Update menu items using the correct ID (v1) Last minute fix on beta 53 for a race condition, along with work from bug 1351964.
Attachment #8854989 -
Flags: approval-mozilla-beta?
Attachment #8854989 -
Flags: approval-mozilla-beta+
Attachment #8854989 -
Flags: approval-mozilla-aurora?
Attachment #8854989 -
Flags: approval-mozilla-aurora+
Comment 9•7 years ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/92ba21762445 https://hg.mozilla.org/releases/mozilla-beta/rev/da09e3a76155 This'll be in tomorrow's Aurora nightly too now for verification purposes.
Reporter | ||
Comment 10•7 years ago
|
||
I can verify the issue is resolved in Nightly. The report site issue button is working, screenshot upload is as well. Thanks!
Status: RESOLVED → VERIFIED
Flags: needinfo?(astevenson)
Comment 11•7 years ago
|
||
I can confirm this issue is verified for Nightly 55.0a1 and Aurora 54.0a2, the "report site issue" button works as expected. On Beta 53.0b11 and the release build, the button is not present in the menu.
This is broken again for me on Nightly.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 13•7 years ago
|
||
Caused by bug 1331742, which has since been backed out.
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•