Support ping, rel, referrerPolicy, relList, hreflang, type and text properties on SVG <a> elements

RESOLVED FIXED in Firefox 61

Status

()

enhancement
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: longsonr, Assigned: longsonr)

Tracking

({dev-doc-complete})

Trunk
mozilla61
Points:
---

Firefox Tracking Flags

(firefox61 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

a year ago
No description provided.
Assignee

Comment 1

a year ago
Posted patch patch (obsolete) — Splinter Review
Assignee: nobody → longsonr
Assignee

Updated

a year ago
Attachment #8965403 - Attachment is patch: true
Assignee

Comment 3

a year ago
Attachment #8965403 - Attachment is obsolete: true
Assignee

Comment 5

a year ago
Comment on attachment 8965458 [details] [diff] [review]
implement and fix test

Passes the tests in bug 1450824 except for 1 expected failure and 1 test bug (https://github.com/w3c/web-platform-tests/pull/10275)

The attributes are DOMString based on the resolution in https://github.com/w3c/svgwg/issues/315 and the tests from https://github.com/w3c/web-platform-tests/pull/10275 expect that too.

Checkin comment:

Bug 1451823 - support ping, rel, referrerPolicy, relList, hreflang, type and text on SVG a elements r=mystor
Attachment #8965458 - Flags: review?(nika)
Comment on attachment 8965458 [details] [diff] [review]
implement and fix test

Review of attachment 8965458 [details] [diff] [review]:
-----------------------------------------------------------------

Please make sure you send a Request to Implement & Ship to dev-platform before landing this :-)

::: dom/svg/SVGAElement.cpp
@@ +34,5 @@
>    { &nsGkAtoms::target, kNameSpaceID_None, true }
>  };
>  
> +// static
> +const DOMTokenListSupportedToken SVGAElement::sSupportedRelValues[] = {

It'd be nice to merge this with the HTMLAnchorElement::sSupportedRelValues, but that may not be worth doing in this patch.
Attachment #8965458 - Flags: review?(nika) → review+

Comment 9

a year ago
Pushed by longsonr@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a1d4a3e6c77a
support ping, rel, referrerPolicy, relList, hreflang, type and text on SVG a elements r=mystor
Created web-platform-tests PR https://github.com/w3c/web-platform-tests/pull/10473 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 12

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a1d4a3e6c77a
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Upstream PR merged
You need to log in before you can comment on or make changes to this bug.