Closed
Bug 1253930
Opened 8 years ago
Closed 8 years ago
Default favicons of 5 recently bookmarked, while in smart folder favicons are OK
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
VERIFIED
FIXED
Firefox 48
People
(Reporter: krzysztof.glebowicz, Assigned: mak)
References
Details
(Keywords: regression)
Attachments
(2 files)
User Agent: Mozilla/5.0 (Windows NT 6.0; rv:47.0) Gecko/20100101 Firefox/47.0 Build ID: 20160305030241 Steps to reproduce: I've noticed a bug in the bookmarks menu. Actual results: As in attached screenshot: Favicons of 5 recently bookmarked pages are default (globes). But in smart folder Recently bookmarked and in real folders the same links have specific favicons.
Comment 1•8 years ago
|
||
[Tracking Requested - why for this release]: Regression window: https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=970aa793c5e2778172cd8e527f9dc04848400d40&tochange=f7149b0bcf7491e23edacbffce6fa418b05e776e Regressed by: f7149b0bcf74 Dão Gottwald — Bug 1250806 - Add menuitems for recent bookmarks synchronously so that they show up in the native menubar on OS X. r=mak
Status: UNCONFIRMED → NEW
status-firefox47:
--- → affected
tracking-firefox47:
--- → ?
Ever confirmed: true
Keywords: regression
Updated•8 years ago
|
Component: Untriaged → Bookmarks & History
Comment 2•8 years ago
|
||
Going to track this regression and see if Dao can take this
Flags: needinfo?(dao)
Assignee | ||
Comment 3•8 years ago
|
||
I can take this, the problem is that nsINavHistoryResult already provides a moz-anno:favicon: url, so we don't need to add the prefix.
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Flags: needinfo?(dao)
Assignee | ||
Comment 4•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/38451/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/38451/
Attachment #8727349 -
Flags: review?(dao)
Updated•8 years ago
|
Attachment #8727349 -
Flags: review?(dao) → review+
Comment 6•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4b613cad02de
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Comment 7•8 years ago
|
||
I managed to reproduce this issue on Firefox 47.0a1 (2016-03-05) and on Windows 10 x86. The issue is no longer reproducible on Firefox 48.0a1 (2016-03-08). The tests were performed on Windows 10 x86, Mac OS X 10.11.1 and on Ubuntu 14.04 x64. I am marking this issue Verified-Fixed.
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 8•8 years ago
|
||
Recent bookmarks have been disabled in 47, so we don't need this.
tracking-firefox47:
+ → ---
Updated•8 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•