Closed
Bug 1342427
Opened 7 years ago
Closed 7 years ago
Setting Web Feeds in Application Preferences to "Add Live Bookmarks" then reselecting it appears to revert to "Preview in Nightly"
Categories
(Firefox :: Settings UI, defect)
Tracking
()
RESOLVED
FIXED
Firefox 54
People
(Reporter: standard8, Assigned: standard8)
References
Details
(Keywords: regression)
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
jaws
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details |
STR 1) Open Preferences 2) Select Applications 3) Next to "Web Feed" select "Add Live Bookmarks to Nightly" -> It is correctly shown as selected. 4) Click on any other item 5) Click back on "Web Feed" Expected Results -> "Add Live Bookmarks to Nightly" remains displayed Actual Results -> "Preview in Nightly" is displayed -> Console message: "ReferenceError: internalMenuItem is not defined" This is a regression from bug 1294989 that I've found when working on enabling the no-undef rule.
Assignee | ||
Updated•7 years ago
|
status-firefox51:
--- → wontfix
status-firefox52:
--- → affected
status-firefox53:
--- → affected
status-firefox54:
--- → affected
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8840893 [details] Bug 1342427 - Setting Web Feeds in Application Preferences to "Add Live Bookmarks" then reselecting it appears to revert to "Preview in Nightly". https://reviewboard.mozilla.org/r/115312/#review116808 We should probably have a test for this.
Attachment #8840893 -
Flags: review?(jaws) → review+
Comment hidden (mozreview-request) |
Assignee | ||
Comment 4•7 years ago
|
||
Comment on attachment 8840893 [details] Bug 1342427 - Setting Web Feeds in Application Preferences to "Add Live Bookmarks" then reselecting it appears to revert to "Preview in Nightly". Now with added unit test :-)
Attachment #8840893 -
Flags: review+ → review?(jaws)
Comment hidden (mozreview-request) |
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8840893 [details] Bug 1342427 - Setting Web Feeds in Application Preferences to "Add Live Bookmarks" then reselecting it appears to revert to "Preview in Nightly". https://reviewboard.mozilla.org/r/115312/#review117142 ::: browser/components/preferences/in-content/tests/browser_applications_selection.js:51 (Diff revision 3) > + win.document.getAnonymousElementByAttribute(feedItem, "class", "actionsMenu")); > + info("Got list after item was selected"); > + Assert.ok(list.selectedItem, "Should have a selected item."); > + Assert.equal(list.selectedItem.getAttribute("action"), > + Ci.nsIHandlerInfo.handleInternally, > + "Newly selected item should by the expected one."); s/should by/should be/
Attachment #8840893 -
Flags: review?(jaws) → review+
Comment hidden (mozreview-request) |
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/84a017610f03 Setting Web Feeds in Application Preferences to "Add Live Bookmarks" then reselecting it appears to revert to "Preview in Nightly". r=jaws
I had to back this out for failures like https://treeherder.mozilla.org/logviewer.html#?job_id=80463655&repo=autoland https://hg.mozilla.org/integration/autoland/rev/f30e28369136fcb28dddcd55a94d98e2816009e9
Flags: needinfo?(standard8)
Assignee | ||
Comment 10•7 years ago
|
||
It appears that the test was getting the favicon for mibbit for building the menulist. I've changed it to a different menu list so that doesn't happen.
Flags: needinfo?(standard8)
Comment hidden (mozreview-request) |
Comment 12•7 years ago
|
||
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a88a6717a2b7 Setting Web Feeds in Application Preferences to "Add Live Bookmarks" then reselecting it appears to revert to "Preview in Nightly". r=jaws
Comment 13•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a88a6717a2b7
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
Comment 14•7 years ago
|
||
Probably too late for Fx52 at this point, but were you thinking this should be uplifted or let it ride the trains?
Assignee | ||
Comment 15•7 years ago
|
||
Comment on attachment 8840893 [details] Bug 1342427 - Setting Web Feeds in Application Preferences to "Add Live Bookmarks" then reselecting it appears to revert to "Preview in Nightly". Approval Request Comment [Feature/Bug causing the regression]: Bug 1294989 [User impact if declined]: Strange effects if they have specific non-default settings in the application preferences. [Is this code covered by automated tests?]: This bug adds a test. [Has the fix been verified in Nightly?]: I've just verified it. [Needs manual test from QE? If yes, steps to reproduce]: See STR [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: No [Why is the change risky/not risky?]: Fixes scope of variables, unit test covers the changes. [String changes made/needed]: None
Flags: needinfo?(standard8)
Attachment #8840893 -
Flags: approval-mozilla-aurora?
Updated•7 years ago
|
Comment 16•7 years ago
|
||
Comment on attachment 8840893 [details] Bug 1342427 - Setting Web Feeds in Application Preferences to "Add Live Bookmarks" then reselecting it appears to revert to "Preview in Nightly". Odd. Let's fix it for 53 aurora. Thanks for including a new test!
Attachment #8840893 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 17•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/02bbbf7e0b0a
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•