264040 - Better handling of manually created hostname-only bookmarks

Status: RESOLVED WORKSFORME

(Firefox :: Bookmarks & History, defect)

Description

[Tracy Walker [:tracy]]

seen on FF branch on Win XP 2004-10-12-08-0.9

-Use Bookmarks | Bookmark Manager to add a new bookmark for a page known to have
a favicon. (www.cnn.com) (bug 264031 is blocking simple add bookmark)
-Click the -paper- icon for the bookmark just added.

tested results:  The icon for the bookmark remains the -paper-. It does not
update to the site favicon.

expected results: Upon visiting the page the associated bookmark icon updates to
the site favicon.

Note:  Many, but not all, of the default bookmarks have favicons associated with
them.  I assume that's because they're included with the install.  Clicking on
the icon for a default bookmark that doesn't have the site favicon, also doesn't
update to the site favicon as expected.

Comment 1

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Not a regression; creating a new bookmark and putting "www.cnn.com" results in a
URL of "http://www.cnn.com" being bookmarked.  The actual site that gets visited
is "http://www.cnn.com/" (note trailing /), so the favicon doesn't get updated.
 We should handle this case though, probably by changing how we mangle
hostname-only bookmarks when they're created..

Comment 2

[Tracy Walker [:tracy]]

Ah sneaky. Thanks for clarifying, Vlad. Updated summary.  

Comment 3

[Mike Connor [:mconnor]]

sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs, filter on "beltznerLovesGoats" to get rid of this mass change

Comment 4

[Marco Bonardo [:mak] (away Aug 5-12, Aug 19-21)]

WFM on current trunk
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090119 Minefield/3.2a1pre