49974 - vlink attribute of Body tag fails.

Status: VERIFIED DUPLICATE of bug 12493

(Core :: Layout, defect, P3)

Description

[Nasiruddin M.S]

After clicking the link, it takes user to the specified url. when we return to
from there, the visited link doesn't show up in purple color.

Comment 1

[Kevin McCluskey (gone)]

Dividing up claytons bugs to triage.

Comment 2

[waqar]

This bug has been marked "future" because the original netscape engineer working
on this is over-burdened. If you feel this is an error, that you or another
known resource will be working on this bug,or if it blocks your work in some way
-- please attach your concern to the bug for reconsideration.

Comment 3

[gerardok]

massive update for QA contact.

Comment 4

[timeless]

Hixie: can you fix this w/ a ua.css?

Comment 5

[Hixie (not reading bugmail)]

Nope. But since :visited is acting up as well, I think this is probably a bug
with the Session History.

Comment 6

[Keyser Sose]

I am seeing this on the PC, Windows 98 with Mozilla #2000120604.

Comment 7

[Gregory Labzovsky]

Attachment: proposed fix

Comment 8

[Gregory Labzovsky]

I'm attaching a patch for file nsStdURL.cpp. This seems to fix the bug. 
	The problem was in different URL representation in href's. All host URL's
stored in history have slashes on the end. A document may have link to this host
without the closing slash. So when URL has "/" on the end, vlink attribute works
well. The patch makes mozilla consider both URL spellings as the same (i.e.
"http://www.cnn.com" and "http://www.cnn.com/").
   On the other hand there is still a problem with href's when they point to
host subdirectories  (as  http://www.cnn.com/countries). This problem seems to
be inherited from Netscape 4.x. I don't know if to consider links like
"http://www.cnn.com/countries" and "http://www.cnn.com/countries/" the same.

Comment 9

[timeless]

Attachment: revised version. no tabs.

Comment 10

[timeless]

Your patch seems to use tabs. We don't like tabs. 
here is the modeline:
/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*-
This means that indents should be four spaces.

Attached is a revised version. [Taking bug because waqar is overburdened]. 
This patch shares a variable w/ a later part of the same function and uses '/' 
instead of "/" which is consistent with the function.

This patch is untested and assumes that the previous one works.

Seeking testing, review and approval.

Comment 11

[timeless]

fwiw countries and countries/ are not equivalent.  If any patch makes them 
equivalent it should not be accepted.

Actually, we probably need to just mark both links as visited. Either we make 
link mappings a => a/ and then resolve(a) => a/; check(a/) => visited, or just 
mark(a)=visited when we visit it.

Come to think of it, I think this is a dupe.

Comment 12

[Loco]

Timeless, thanks for bringing that up.  Yes, Its a DUPLICATE of #12493.  -d


*** This bug has been marked as a duplicate of 12493 ***

Comment 13

[timeless]

vrfy dupe. All status moved to original.

Comment 14

[Heikki Toivonen (remove -bugzilla when emailing directly)]

SPAM. HTML Element component is deprecated, changing to Layout component. See
bug 88132 for details.

Comment 15

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Come on Bugzilla, you can do it...