Closed
Bug 323508
Opened 19 years ago
Closed 17 years ago
Favicon (error icon) in Bookmark not updated
Categories
(SeaMonkey :: Bookmarks & History, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: 3.14, Assigned: neil)
References
Details
(Keywords: fixed-seamonkey1.1.10)
Attachments
(1 file)
(deleted),
patch
|
jag+mozilla
:
review+
jag+mozilla
:
superreview+
ajschult784
:
approval-seamonkey1.1.10+
|
Details | Diff | Splinter Review |
Hi!
Most likely a dupe, but I don't find it. It appears in Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8) Gecko/20051219 SeaMonkey/1.0b and previous versions.
I have a bookmark for a group of tabs. When I created it, some icons where available which changed in the meantime. In particular, one page did not load and had the warning symbol (exclamation point in yellow triangle). Those icons were saved and are displayed in the bookmark manager.
Now icons changed, in particular the one with the warning. They are always displayed correctly in the tabs when I call the bookmark. The icons shown in the bookmark manager are never updated, though. There also seems to be no way of manually updating.
pi
Reporter | ||
Comment 1•19 years ago
|
||
To add to this bug, I now observe the following (using WinXP):
There seem to be (at least) three different (!) icon caches:
1) for icons in "Bookmars" drop-down menu
2) for bookmark manager
3) for personal toolbar
I actually obeserved, that icons in the toolbar don't show (when not visiting the links). If I go in the drop-down menu and "Personal Toolbar" gets expanded, there the links are populated with icons, but they don't show up in the toolbar itself.
Actually, all three different caches should only be one since they refer to the exact same set of bookmarks.
pi
Reporter | ||
Comment 3•19 years ago
|
||
I think the real problem is that the error icon is actually written in form of data: to bookmarks.html. The icon attribute is then not updated.
pi
Summary: Favicon in Bookmark not update → Favicon (error icon) in Bookmark not updated
Reporter | ||
Comment 4•18 years ago
|
||
In 1.1b this problem just went away. No clue which bug fixed it.
pi
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Comment 5•18 years ago
|
||
The bug is back in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.4) Gecko/20070509 SeaMonkey/1.1.2 (also in 1.1.1 already). The warning sign is never updated. To test simply make a page unavailable (e.g. use invalid proxy setting), see icon in bookmarks. Regularly load the page, icon is shown in address bar, bookmark icon is not updated.
Resolution would be not to save the icon explicitly in the bookmark file, I believe.
pi
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Assignee | ||
Comment 6•17 years ago
|
||
I don't think that error icons should ever update bookmarks. The fact that they do can be traced to a combination of two bugs in the patch for bug 108809. The first is that the code in the onLinkIconAvailable handler always submits the icon to the bookmarks, even when the icon isn't actually available, although navigator.js already waits for the icon to load before submitting it to bookmarks. The second is that it determines the document's URI incorrectly.
Blocks: 108809
Assignee | ||
Comment 7•17 years ago
|
||
Assignee: nobody → neil
Status: REOPENED → ASSIGNED
Attachment #313562 -
Flags: superreview?(jag)
Attachment #313562 -
Flags: review?(jag)
Comment 8•17 years ago
|
||
Comment on attachment 313562 [details] [diff] [review]
Proposed patch
So this code is redundant because |onload| and |onerror| on the page-proxy-favicon element (updated right above the code you're removing) already do the right thing through |HandleBookmarkIcon(...)|.
r+sr=jag
That mehod, btw, has a rather ugly if/else. Feel free to fix that with rs=jag
Attachment #313562 -
Flags: superreview?(jag)
Attachment #313562 -
Flags: superreview+
Attachment #313562 -
Flags: review?(jag)
Attachment #313562 -
Flags: review+
Assignee | ||
Comment 9•17 years ago
|
||
Comment on attachment 313562 [details] [diff] [review]
Proposed patch
This patch should be simple enough for the branch too (obviously when applied against its old location).
Attachment #313562 -
Flags: approval-seamonkey1.1.10?
Comment 10•17 years ago
|
||
Comment on attachment 313562 [details] [diff] [review]
Proposed patch
a=ajschult
Attachment #313562 -
Flags: approval-seamonkey1.1.10? → approval-seamonkey1.1.10+
Assignee | ||
Comment 11•17 years ago
|
||
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago → 17 years ago
Keywords: fixed-seamonkey1.1.10
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•