Favicon (error icon) in Bookmark not updated

RESOLVED FIXED

Status

SeaMonkey
Bookmarks & History
RESOLVED FIXED
11 years ago
9 years ago

People

(Reporter: Boris 'pi' Piwinger, Assigned: neil@parkwaycc.co.uk)

Tracking

({fixed-seamonkey1.1.10})

1.8 Branch
x86
Windows 98
fixed-seamonkey1.1.10

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

1.20 KB, patch
jag (Peter Annema)
: review+
jag (Peter Annema)
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

11 years ago
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

11 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

Comment 2

11 years ago
Reassigning as per Bug #32644
Assignee: p_ch → nobody
(Reporter)

Comment 3

11 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

11 years ago
In 1.1b this problem just went away. No clue which bug fixed it.

pi
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → WORKSFORME
(Reporter)

Comment 5

10 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

9 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

9 years ago
Created attachment 313562 [details] [diff] [review]
Proposed patch
Assignee: nobody → neil
Status: REOPENED → ASSIGNED
Attachment #313562 - Flags: superreview?(jag)
Attachment #313562 - Flags: review?(jag)

Comment 8

9 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

9 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

9 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

9 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago9 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.