139626 - Link Navigates to the Wrong Page

Status: RESOLVED DUPLICATE of bug 90664

(Core :: DOM: HTML Parser, defect, P3)

Description

[Nick Lerissa]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.0rc1)
Gecko/20020417
BuildID:    2002041711

The link labeled Management on the left side of the page
just refreshes the page even though it should be navigating
to another page.
http://www.abilizer.com/channel?channel=company

Reproducible: Always
Steps to Reproduce:
1.go to http://www.abilizer.com/channel?channel=company
2.look at source (notice link labeled Management)
3.click on Managment link

Actual Results:  It refreshes the page and does not navigate properly

Expected Results:  should navigate to:
http://www.abilizer.com/channel?channel=management

Comment 1

[Boris Zbarsky [:bzbarsky]]

worksforme, linux trunk build 2002-04-22-07.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Actually, I do see this.  The first click fails, the second succeeds...  Need to
figure out the right component here.

Comment 3

[Mike Brittain]

This appears to be an issue with the HTML code for the page and not a bug with
Mozilla.  

I've reduced the code to a test case, but left enough for you to see where this
is an issue.  The A tag for the "company" link is left hanging open and finally
closed AFTER the "management", "careers", "investors", and "contact" links.

See the test case at
http://www.embimedia.com/temp/mozilla/139626/abilizer_test2.html

If you look at the source, you'll see the comments that I've added showing the
opening and closing of that A tag.

Internet Explorer seems to render this code in a manner that allows the link to
work properly (in the sense that it goes to the expected location).  I don't
know whether "quirks" mode in Mozilla should be able to pick up something like
this or not.

Comment 4

[Boris Zbarsky [:bzbarsky]]

To parser.  <a> is not allowed to contain other <a> tags, so we should be
closing out that first <a>....  thanks for the testcase, Mike!

Comment 5

[Mike Brittain]

testcase keyword added.

Comment 6

[harishd]

Observation: When I shift+reload the testcase then the links work correctly!

Comment 7

[Patrick Xia]

*** Bug 157252 has been marked as a duplicate of this bug. ***

Comment 8

[Patrick Xia]

From bug 157252, marked as dupe.

If you right-click, and copy location of the link, the link turns out fine.

Comment 9

[Sean Harding]

Also, open in new window/tab opens the right link (at least in the example from
Bug 157252). Apparently, only a plain click on the link loads the wrong URL.
 

Comment 10

[Boris Zbarsky [:bzbarsky]]

Attachment: Mike's testcase

Comment 11

[Boris Zbarsky [:bzbarsky]]

Attachment: Minimal testcase

Note that removing the last </a> makes the bug go away.  Changing to standards
mode does not affect the bug.

Comment 12

[Andrew Schultz]

worksforme with linux trunk 2005061905, although I see the bug with 1.7.6

DOM inspector has the following content model for the second testcase with trunk:
<a/>
<table>...<a/>...</table>
<a/>

where the last <a/> has the same href as the first (google), but is empty.  Is
that correct?

Comment 13

[Blake Kaplan (:mrbkap) (inactive)]

It is correct. If the <table> was a <div>, it wouldn't, but hey, who said
compat. stuff had to make sense? Duping to the bug that fixed this.

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