Closed Bug 1444082 Opened 6 years ago Closed 6 years ago

Sync reader mode to github tip ( 8525c6af36d3badbe27c4672a6f2dd99ddb4097f )


(Toolkit :: Reader Mode, enhancement)

Not set



Tracking Status
firefox60 --- fixed


(Reporter: Gijs, Assigned: Gijs)




(1 file)

Current tip:

Last sync was bug 1425270, which was fa9d8bda48ee574bcffbc19d68b4ca39e1f9036a. Git log:

commit 8525c6af36d3badbe27c4672a6f2dd99ddb4097f (HEAD -> master, upstream/master, upstream/HEAD, origin/master)
Author: Brad Philips
Date:   Fri Mar 2 11:37:16 2018 +0000

    Fix relative URIs given <base> tags (#422)

commit d598baf02b5a7d803074915095891ae17c70feea
Author: Gijs Kruitbosch
Date:   Tue Feb 27 17:36:02 2018 +0000

    Improve URL handling in JSDOMParser and Readability.js
    This change ups the required node version to 7.0 because it relies on the builtin url module.
    We now pass a url when constructing a jsdom document or JSDOMParser document.
    Because this is an API change, I'm increasing the package version.
    Ultimately, I would like to remove the  argument from the readability constructor. It should
    use the documentURI from the document it is passed.

commit 834672ef864191f780a411030bb7c73e86b0aec0
Author: Andres Rey
Date:   Tue Feb 27 14:26:54 2018 +0000

    Return longest text after failing to detect text longer than the configured value (#423)
    Save extracted text across attempts and return the longest one when all attempts fail, and add a test case from hukumusume

commit 264b8e8968dc0299d9b18d261a13d00d15baa040
Author: Tom Zöhner
Date:   Tue Jan 30 15:40:49 2018 +0100

    Remove link elements when preparing article for display

Corresponding pull requests:

Essentially, this fixes:
- inclusion of random stylesheets (though I expect in Firefox's case, the sanitizer would take care of this anyway)
- giving up when there's too little text on the page (this fixes hopefully all of - in the sense that they will now get content. It may still not be the *best* content, but that's going to be the next step, and it's still better than the useless error we currently show)
- relative URI handling when there's a <base> tag that doesn't get included in the reader mode output (e.g. because it's in <head> ).

This should be fairly straightforward to merge.
Comment on attachment 8957156 [details]
Bug 1444082 - sync reader mode to github tip ( 8525c6af36d3badbe27c4672a6f2dd99ddb4097f ),

Yup, this seems good. Thanks!
Attachment #8957156 - Flags: review?(jhofmann) → review+
Pushed by
sync reader mode to github tip ( 8525c6af36d3badbe27c4672a6f2dd99ddb4097f ), r=johannh
Backout by
Backed out changeset 609e4952a46e for perma-failing android-4.3-arm7-api-16/opt-robocop-3 (RC3) CLOSED TREE
Flags: needinfo?(gijskruitbosch+bugs)
Sigh, similarly-trivial change to a 'this is not an article' testcase as was already in the patch for the toolkit tests, green try (can't easily test locally, hi android robocop...):

Relanding in a second.
Flags: needinfo?(gijskruitbosch+bugs)
Pushed by
sync reader mode to github tip ( 8525c6af36d3badbe27c4672a6f2dd99ddb4097f ), r=johannh
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Blocks: 1427002
You need to log in before you can comment on or make changes to this bug.