Closed Bug 1159812 Opened 5 years ago Closed 4 years ago
"Bookmark All Tabs" is broken
[Tracking Requested - why for this release]: Steps to reproduce 1. Right click on a tab 2. Choose "Bookmark All Tabs" 3. Expand tree view 4. Expand Bookmarks Menu 5. Select "Mozilla Firefox" folder 6. Click [Add Bookmarks] to save them Actual Results: When I select sub-folder, the select folder is not expanded and new folder appears under the special bookmark(such as "Bookmarks Toolbar" "Bookmarks Menu" "Unsorted Bookmarks"). And Bookmarks are always created under the special bookmark. Expected Results: When I select sub-folder, the select folder should be expanded automatically and new folder should appear under the selected folder. And [Add Bookmarks] is successfully performed Error in Browser console: ReferenceError: container is not defined editBookmarkOverlay.js:721:10 ReferenceError: menupopup is not defined editBookmarkOverlay.js:666:0 I wonder there are no automate tests... Too much regression due to refactoring Bug 951651.
Unfortunately it's very hard to reach 100% tests coverage with such huge refactorings, we have some tests, clearly not enough, but we have few devs and moreover the tests we have need some refactoring too :/ But I'm trying to be responsive on regressions.
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Points: --- → 3
I'm going to file a bug to refactor browser_bookmarkProperties and add more tests to it.
Comment on attachment 8600087 [details] [diff] [review] patch v1 Review of attachment 8600087 [details] [diff] [review]: ----------------------------------------------------------------- Wow, so many typos. We seriously need more tests here.
Attachment #8600087 - Flags: review?(ttaubert) → review+
I'll leave this untracked for now since it's already fixed. Please renominate if it needs it! Thanks.
4 years ago
Verified fixed on Windows 7 64bit, Mac OSX 10.9.5 and Ubuntu 13.10 32bit using latest Aurora 40.0a2 (buildID: 20150526004004).
You need to log in before you can comment on or make changes to this bug.