357431 - [PATCH] added a note to removeObserver about using the same pref branch for which addObserver was used

Status: RESOLVED FIXED

(Core :: Preferences: Backend, defect)

Description

[Andreas Wuest]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.0.7) Gecko/20060909 Firefox/1.5.0.7
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.0.7) Gecko/20060909 Firefox/1.5.0.7

removeObserver has to be called on the same pref branch instance on which addObserver was called for the observer to be removed.

Reproducible: Always




Patch follows.

Comment 1

[Andreas Wuest]

Attachment: Note about using the same pref branch instance as for addObserver().

Comment 2

[Darin Fisher]

Comment on attachment 242908 [details] [diff] [review]
Note about using the same pref branch instance as for addObserver().

ok, r+sr=darin

Comment 3

[Andreas Wuest]

Sorry for the bug spam.

Comment 4

[Robert Sayre]

checked in.

Checking in nsIPrefBranch2.idl;
/cvsroot/mozilla/modules/libpref/public/nsIPrefBranch2.idl,v  <--  nsIPrefBranch2.idl
new revision: 1.4; previous revision: 1.3
done