655756 - Remove final XP_MAC remnants

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Ed Morley [:emorley]]

Bug 281889 removed most of the XP_MAC (ie Mac Classic) code, but there is some still left to do.

Bug 480745 is already covering the /security/manager/ parts, which leaves the rest of:

http://mxr.mozilla.org/mozilla-central/search?string=XP_MAC&case=1&find=&findi=&filter=XP_MAC\b&hitlimit=&tree=mozilla-central

Comment 1

[Ed Morley [:emorley]]

Attachment: Remove final XP_MAC remnants

http://dev.philringnalda.com/tbpl/?tree=Try&rev=b646f3f1b8dc

Comment 2

[Ed Morley [:emorley]]

Meant to add, I deliberately omitted the activex/src/pluginhostctrl/pluginsdk_include/ instances, since it seems to manually set XP_MAC even on XP_MACOSX, since it checks |ifdef macintosh|, which afaik is defined on OS X too:
http://mxr.mozilla.org/mozilla-central/source/embedding/browser/activex/src/pluginhostctrl/pluginsdk_include/npapi.h#20

Comment 3

[Ludovic Hirlimann [:Usul]]

NSS part is being taken care in bug 480745

Comment 4

[Ed Morley [:emorley]]

Yup, I deliberately omitted those parts for that reason :-) (Comment 0)

Comment 5

[Ed Morley [:emorley]]

Attachment: Remove final XP_MAC remnants

Only change is updated commit message to include r=josh
Carrying forwards r+

Comment 6

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

http://hg.mozilla.org/mozilla-central/rev/b68afa9fd243