1155692 - Week 16 / April 17 - -Uplift github version of Readability/JSDOMParser into mozilla-central and aurora/beta

Status: RESOLVED FIXED

(Toolkit :: Reader Mode, defect)

Description

[:Gijs (he/him)]

+++ This bug was initially created as a clone of Bug #1152022 +++

Per request from the sheriffs, one bug per uplift, so here's this week's edition of "let's update Readability.js"!

Comment 1

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/bde44a7f140b

Comment 2

[:Gijs (he/him)]

Attachment: include latest Readability/JSDOMParser changes into m-c, rs=me

Comment 3

[Ryan VanderMeulen [:RyanVM]]

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

Comment 4

[:Gijs (he/him)]

Comment on attachment 8593993 [details] [diff] [review]
include latest Readability/JSDOMParser changes into m-c, rs=me

Approval Request Comment
[Feature/regressing bug #]: reader view
[User impact if declined]: no reader view bugfixes
[Describe test coverage new/current, TreeHerder]: has a bunch of test coverage on github
[Risks and why]: pretty low, the changes here are small and really just improve the handling of links - and it's all limited in impact to reader view
[String/UUID change made/needed]: nope

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8593993 [details] [diff] [review]
include latest Readability/JSDOMParser changes into m-c, rs=me

Should be in 38 beta 6

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/38b0d3f462f5

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/eb5e2063637b