[FIX] LINK element with href attribute should match :link

RESOLVED FIXED

Status

()

--
minor
RESOLVED FIXED
16 years ago
4 years ago

People

(Reporter: mats, Assigned: mats)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [patch], URL)

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
LINK element with href attribute should match :link

REPEAT BY:
1. load http://www.bath.ac.uk/%7Epy8ieh/internet/projects/mozilla/linktests.html

ACTUAL BEHAVIOUR:
The first list of links at the top are missing.

EXPECTED BEHAVIOUR:
Three list-items that works as links (same as for AREA and A below)

PLATFORM & BUILD:
Mozilla nightly build 2003-06-03-05 trunk Linux

ADDITIONAL INFORMATION:
It worked at the time bug 44396 was verified, 2000-08-25.
-> style system
Assignee: other → dbaron
Component: Layout → Style System
(Assignee)

Comment 2

16 years ago
FWIW, bug occurs in Mozilla 1.0.2 too.
(Assignee)

Comment 3

16 years ago
Created attachment 125161 [details] [diff] [review]
Patch rev. 1

It's the following call in nsStyleUtil::IsHTMLLink(...) that returns NULL
for all LINK elements:

    nsCOMPtr<nsILink> link( do_QueryInterface(aContent) );

http://lxr.mozilla.org/mozilla/source/content/html/style/src/nsStyleUtil.cpp#572


The patch adds the nsILink interface to the LINK element.
(Assignee)

Updated

16 years ago
Summary: LINK element with href attribute should match :link → [FIX] LINK element with href attribute should match :link
(Assignee)

Updated

16 years ago
Attachment #125161 - Flags: review?(dbaron)
Comment on attachment 125161 [details] [diff] [review]
Patch rev. 1

This was a regression from the changes in revision 1.30 of the file.  r=dbaron
Attachment #125161 - Flags: review?(dbaron) → review+
Fix checked in to trunk, 2003-06-08 12:53 -0700.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Updated

4 years ago
See Also: → bug 118064
You need to log in before you can comment on or make changes to this bug.