Skip to content

Commit

Permalink
Bug 1543394 - Don't update visited state of all <html:link> elements …
Browse files Browse the repository at this point in the history
…in chrome docs. r=smaug

I don't think there's a point in making <link> elements match :visited, and it's
an issue for Chrome docs because some chrome code can run before we have a
profile.

Make the already-existent workaround for localization links work more generally.

There's no interop across browsers here anyhow:

  w3c/csswg-drafts#3817

tracks that.

Differential Revision: https://phabricator.services.mozilla.com/D26910

UltraBlame original commit: e2a285801fb26598983b83b32fa3a44f1840c441
  • Loading branch information
marco-c committed Oct 4, 2019
1 parent 7bc84dd commit 1ee6f92
Showing 1 changed file with 16 additions and 28 deletions.
44 changes: 16 additions & 28 deletions dom/html/HTMLLinkElement.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -714,29 +714,6 @@ rv
rv
)
;
bool
isLocalizationLink
=
aDocument
&
&
this
-
>
AttrValueIs
(
kNameSpaceID_None
nsGkAtoms
:
:
rel
nsGkAtoms
:
:
localization
eIgnoreCase
)
;
if
(
Document
Expand All @@ -751,10 +728,6 @@ GetComposedDoc
if
(
!
isLocalizationLink
|
|
!
doc
-
>
Expand Down Expand Up @@ -822,7 +795,22 @@ update
;
if
(
isLocalizationLink
aDocument
&
&
AttrValueIs
(
kNameSpaceID_None
nsGkAtoms
:
:
rel
nsGkAtoms
:
:
localization
eIgnoreCase
)
)
{
aDocument
Expand Down

0 comments on commit 1ee6f92

Please sign in to comment.