We don't use the original document in reader. No need to care about stylesheets in the original document.
Created attachment 653399 [details] [diff] [review] No need to remove stylesheets and style tags in Readability.js
Attachment #653399 - Flags: review?(bnicholson)
Attachment #653399 - Flags: review?(bnicholson) → review+
I should have only removed the part that removes stylesheets from the document. Some pages add style tags inside the body and is causing problems to the candidate score calculation. I'll put this code back.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
Comment on attachment 653399 [details] [diff] [review] No need to remove stylesheets and style tags in Readability.js [Approval Request Comment] User impact if declined: Unnecessary overhead when checking readability of pages. Testing completed (on m-c, etc.): Landed in m-c, no regressions. Risk to taking this patch (and alternatives if risky): Very low, just removes unnecessary code. String or UUID changes made by this patch: None.
Attachment #653399 - Flags: approval-mozilla-aurora?
Attachment #653399 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Pushed to mozilla-aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/c80a6ec01250
The code was cleaned up on the latest Nightly, Aurora and Beta builds. Closing bug as verified fixed. -- Device: Galaxy Note OS: Android 4.0.4
Status: RESOLVED → VERIFIED
status-firefox16: fixed → verified
status-firefox17: --- → verified
status-firefox18: --- → verified
You need to log in before you can comment on or make changes to this bug.