1650330 - Import validator patch for locator changes

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, task)

Comment 1

[Henri Sivonen (:hsivonen)]

Attachment: Bug 1650330 - Ensure every Locator is also a Locator2 (#10). DONTBUILD NPOTB

• Replace org.xml.sax.helpers.LocatorImpl with nu.validator.htmlparser.impl.LocatorImpl

• instanceof before cast to Locator2

Comment 2

[Henri Sivonen (:hsivonen)]

Asked the validator patch author for OK for the way authorship is recorded here:
https://github.com/validator/htmlparser/pull/10#issuecomment-653454214

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:hsivonen, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 4

[Henri Sivonen (:hsivonen)]

(In reply to Henri Sivonen (:hsivonen) from comment #2)

Asked the validator patch author for OK for the way authorship is recorded here:
https://github.com/validator/htmlparser/pull/10#issuecomment-653454214

The reply was yes:
https://github.com/validator/htmlparser/pull/10#issuecomment-653496670

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e478779437b5
Ensure every Locator is also a Locator2 (#10). DONTBUILD NPOTB r=alchen

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/e478779437b5