Closed Bug 1286973 Opened 4 years ago Closed 3 years ago

Remove MacQuirks (Thunderbird part)

Categories

(Thunderbird :: OS Integration, defect)

Unspecified
macOS
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 50.0

People

(Reporter: Nomis101, Assigned: Nomis101)

References

Details

Attachments

(1 file)

Bug 1282980 is removing MacQuirks from Core, so we need to do the same for mail and remove it from nsMailApp.cpp
Attached patch PatchSplinter Review
Assignee: nobody → Nomis101
Comment on attachment 8771143 [details] [diff] [review]
Patch

I expect I need two reviewers for this (im, mail part)?
Attachment #8771143 - Flags: review?(mkmelin+mozilla)
Attachment #8771143 - Flags: review?(clokep)
Comment on attachment 8771143 [details] [diff] [review]
Patch

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

Thanks!
Attachment #8771143 - Flags: review?(clokep) → review+
Comment on attachment 8771143 [details] [diff] [review]
Patch

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

Thx! r=mkmelin
Attachment #8771143 - Flags: review?(mkmelin+mozilla) → review+
The Core part seems checked-in, so I mark this checkin-needed now.
Keywords: checkin-needed
OS: Unspecified → Mac OS X
https://hg.mozilla.org/comm-central/rev/10f76d5e664cabf6112631b0a6bd3db2d31d2a30
Status: NEW → RESOLVED
Closed: 3 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 50.0
You need to log in before you can comment on or make changes to this bug.