864417 - broken observer calls in editor.js due to bug 795158

Status: RESOLVED FIXED

(MailNews Core :: Composition, defect)

Description

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Attachment: patch, v1

Ian made me aware what two uses of the variable |pref| haven't been converted to Services.prefs, see bug 795158 comment 31. The patch should also be uplifted to aurora and beta.

Comment 1

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 740388 [details] [diff] [review]
patch, v1

Review of attachment 740388 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!

Comment 2

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Attachment: patch, v2, r=IanN r=mconley

Comment 3

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Comment on attachment 751488 [details] [diff] [review]
patch, v2, r=IanN r=mconley

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: 
Testing completed (on c-c, etc.): 
Risk to taking this patch (and alternatives if risky):

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/6fd3a5c7ab95

Comment 5

[Mark Banner (:standard8)]

https://hg.mozilla.org/releases/comm-aurora/rev/524e6222349b

Comment 6

[Mark Banner (:standard8)]

https://hg.mozilla.org/releases/comm-beta/rev/2db318511ed4