Closed Bug 657067 Opened 13 years ago Closed 13 years ago

XPIProvider.jsm refers to nsIPrefBranch.getComplexPref... which doesn't exist

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla6

People

(Reporter: WeirdAl, Assigned: mfinkle)

References

()

Details

Attachments

(1 file)

The non-existent method results in exceptions being thrown (and caught in Venkman).
Attached patch patchSplinter Review
This patch uses the right method name. I also changed the helper method name to match, to avoid any confusion. I am sending to Try server. I am also trying to figure out why this embarrassing error didn't cause any visible problems.
Assignee: nobody → mark.finkle
Attachment #532409 - Flags: review?(dtownsend)
Try server results are completely green, except for 2 known Android failures:

http://hg.mozilla.org/try/rev/28b7a30635bb
http://tbpl.mozilla.org/?tree=Try&pusher=mfinkle@mozilla.com
Attachment #532409 - Flags: review?(dtownsend) → review+
pushed:
http://hg.mozilla.org/mozilla-central/rev/f717485edc51
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Verified by check-in (merge) on mozilla-aurora.
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
Flags: in-litmus-
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: