1116231 - Move "reader.parse-on-load." pref change observer back to Reader.js

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Reader View, defect)

Description

[:Margaret Leibovic]

Looks like a mistake from the changes in bug 793920.

Comment 1

[:Margaret Leibovic]

Attachment: MozReview Request: bz://1116231/margaret

Comment 2

[:Margaret Leibovic]

/r/1769 - Bug 1116231 - Move reader.parse-on-load. pref change observer back to Reader.js

Pull down this commit:

hg pull review -r 84b2b8daaf056f544174c174904400957aa8a94e

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

How did you notice that the wrong change was made? It looks like we should add an automated test so that we don't only notice the bustage after landing.

Comment 4

[:Margaret Leibovic]

/r/1769 - Bug 1116231 - Move reader.parse-on-load. pref change observer back to Reader.js
/r/1771 - Bug 1111142 - (Part 1) Turn aboutReader.js into an AboutReader module
/r/1773 - Bug 1111142 - (Part 2) Load AboutReader in a frame script on Fennec
/r/1775 - Bug 1111142 - (Part 3) Replace fennec-specific logic in AboutReader.jsm with messages

Pull down these commits:

hg pull review -r 28d2fe13d47b04073aca4d32d7c178bb263a7080

Comment 5

[:Margaret Leibovic]

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #3)
> How did you notice that the wrong change was made? It looks like we should
> add an automated test so that we don't only notice the bustage after landing.

I noticed because I was moving this to another place in bug 1111142 :)

I agree, some automated tests would be good. We currently have a few basic reader mode tests for Android, but after the dust settles on the refactoring I'm doing, it would be good to get some shared xpcshell tests in toolkit. The fact that we're using modules should hopefully make test writing more pleasant!

Comment 6

[:Margaret Leibovic]

(In reply to :Margaret Leibovic from comment #4)
> /r/1769 - Bug 1116231 - Move reader.parse-on-load. pref change observer back
> to Reader.js
> /r/1771 - Bug 1111142 - (Part 1) Turn aboutReader.js into an AboutReader
> module
> /r/1773 - Bug 1111142 - (Part 2) Load AboutReader in a frame script on Fennec
> /r/1775 - Bug 1111142 - (Part 3) Replace fennec-specific logic in
> AboutReader.jsm with messages
> 
> Pull down these commits:
> 
> hg pull review -r 28d2fe13d47b04073aca4d32d7c178bb263a7080

Gr, review board doesn't seem to handle building patches on top of patches from other bugs... I'll try to fix this.

Comment 7

[:Margaret Leibovic]

/r/1769 - Bug 1116231 - Move reader.parse-on-load. pref change observer back to Reader.js

Pull down this commit:

hg pull review -r 84b2b8daaf056f544174c174904400957aa8a94e

Comment 8

[Brian Nicholson (:bnicholson)]

https://reviewboard.mozilla.org/r/1767/#review1241

Comment 9

[:Margaret Leibovic]

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

Comment 10

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/9198ef9c7df6

Comment 12

[:Margaret Leibovic]

Attachment: MozReview Request: Bug 1116231 - Move reader.parse-on-load. pref change observer back to Reader.js

Comment 13

[:Margaret Leibovic]

Attachment: MozReview Request: Bug 1111142 - (Part 1) Turn aboutReader.js into an AboutReader module

Comment 14

[:Margaret Leibovic]

Attachment: MozReview Request: Bug 1111142 - (Part 2) Load AboutReader in a frame script on Fennec

Comment 15

[:Margaret Leibovic]

Attachment: MozReview Request: Bug 1111142 - (Part 3) Replace fennec-specific logic in AboutReader.jsm with messages