Closed Bug 734550 Opened 9 years ago Closed 8 years ago

test_mailGlue_distribution.js: Services.prefs.getPref() does not exist

Categories

(Thunderbird :: Build Config, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 23.0

People

(Reporter: sgautherie, Assigned: sshagarwal)

References

()

Details

Attachments

(1 file)

yeah we should probably just error out there.
Assignee: nobody → syshagarwal
Status: NEW → ASSIGNED
Attached patch PatchSplinter Review
Sir,

I have tried to make the necessary change. Please review and give feedback.
Attachment #745976 - Flags: review?(sid.bugzilla)
Attachment #745976 - Flags: review?(mconley)
Attachment #745976 - Flags: feedback?(acelists)
Comment on attachment 745976 [details] [diff] [review]
Patch

Yes, this seems fine and does what sid0 proposed.
Attachment #745976 - Flags: feedback?(acelists) → feedback+
Comment on attachment 745976 [details] [diff] [review]
Patch

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

Thanks!
Attachment #745976 - Flags: review?(mconley) → review+
Attachment #745976 - Flags: review?(sid.bugzilla)
https://hg.mozilla.org/comm-central/rev/fbe5fc95050d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Flags: in-testsuite+
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 23.0
You need to log in before you can comment on or make changes to this bug.