Closed Bug 570902 Opened 12 years ago Closed 12 years ago

Remove ifdefs relating to MOZILLA_1_9_2 from Thunderbird


(Thunderbird :: Build Config, defect)

Not set


(Not tracked)

Thunderbird 3.3a1


(Reporter: Callek, Assigned: Callek)




(3 files)

Mark told me on IRC we are ready for this for the rest of c-c.
The upcoming patch obsoletes storage-backport. Instead if show a patch that removes it all, I'll just attach the hg addremove output from its MQ changeset.
Attachment #450049 - Flags: superreview?(bugzilla)
Attachment #450049 - Flags: review?(bugmail)
I'm not sure what we are meant to do with: So would appreciate someone else taking over that file work.
Attached patch kill the ifdefSplinter Review
Attachment #450060 - Flags: review?(bugzilla)
Comment on attachment 450049 [details]
remove storage-backport

This seems right and the changes in the patch seem right too.

(Per I only touched 2 files outside of storage-backport, and your patch removes those bits again.)
Attachment #450049 - Flags: review?(bugmail) → review+
Attachment #450049 - Flags: superreview?(bugzilla) → superreview+
Attachment #450060 - Flags: review?(bugzilla) → review+
This seems to be the right way to get rid of the extra code in nsMessengerOSXIntegration. Requesting review from Makoto as he put in most of the ifdefs in the first place ;-)
Attachment #450086 - Flags: superreview?(bienvenu)
Attachment #450086 - Flags: review?(m_kato)
Attachment #450086 - Flags: review?(m_kato) → review+
Attachment #450086 - Flags: superreview?(bienvenu) → superreview+
Attachment #450060 - Flags: superreview+
Both patches checked in:
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.2a1
Landed the storage-backport removal though I botched the commit message on that one.
Depends on: 571644
No longer depends on: 571644
You need to log in before you can comment on or make changes to this bug.