Closed Bug 1286973 Opened 4 years ago Closed 3 years ago

Remove MacQuirks (Thunderbird part)

Categories

(Thunderbird :: OS Integration, defect)

Product:
Thunderbird
Component:
OS Integration
Platform:
Unspecified
macOS
Type:
defect
Priority:
Not set

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Thunderbird 50.0

People

(Reporter: Nomis101, Assigned: Nomis101)

References

Details

Attachments

(1 file)

Patch
2.09 KB, patch
clokep
: review+
mkmelin
: review+
Details | Diff | Splinter Review 
Bug 1282980 is removing MacQuirks from Core, so we need to do the same for mail and remove it from nsMailApp.cpp
Attached patch Patch — — Splinter 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.