298430 - drop support for Mac OS X 10.1 ("Puma")

Status: RESOLVED FIXED

(Core Graveyard :: Tracking, defect)

Description

[Josh Aas]

All Mozilla.org products should drop support for Mac OS X 10.1.x (simply 10.1
from here on) ASAP for the following reasons:

1) we do not have adequate QA for Mac OS X 10.1 and cannot ensure a quality
product - this includes Firefox 1.1
2) 10.1 has a very immature API - we can use more complete and advanced APIs
that are not available on 10.1 once support for it is dropped
3) Mac developers are stretched thin at the moment, and expending effort on 10.1
compatibility would be silly at this point
4) the number of users on 10.1 is minimal, and most people who would be using a
web browser a lot would have upgraded to another OS version or would not be
using Firefox
5) things are going to get much worse concerning 10.1 when we try to use cocoa
widgets and quartz
6) once support for 10.1 is dropped, our code can be optimized further by
removing 10.1 workarounds

This bug should remain open until 10.1 support is dropped, as I will be making
notes for things we can do once that happens (code we can remove, optimizations,
new APIs that get freed up, bugs we can close).

Comment 1

[Josh Aas]

We can remove a bunch of runtime checks that use OnJaguarOrLater() since that
will always be true...

"MacOSX 10.2 supports CGPattern, which does image/pattern tiling for us" - you
can already turn on an almost-working implementation of this in nsImageMac.cpp
by defining USE_CGPATTERN_TILING.

Comment 2

[Asaf Romano (gone)]

Some fixed bugs which need some cleanup once we drop 10.1 support:
bug 266551 - Mac BiDi caret support
bug 161332 - Need to use ATSUGetUnjustifiedBounds instead of ATSUMeasureText
bug 223545 - expose feature reveals "hidden" window

Comment 3

[Asaf Romano (gone)]

I should also note that the Launch Service patches on bug 274374 are likely to
requier this change.

Comment 4

[Asaf Romano (gone)]

*** Bug 276367 has been marked as a duplicate of this bug. ***

Comment 5

[Bill McGonigle (not currently reading bugmail; please contact directly)]

For some reason some of the dependencies on my bug were deleted when being duped into this one.

[e-mail] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  BugsThisDependsOn|163830, 197453, 266551      |
             Status|NEW                         |RESOLVED
         Resolution|                            |DUPLICATE

fixing.  Bill Mason also suggested this is core/tracking.  I'll let the current owner deal with that.

Comment 6

[Felix Miata]

Seems like you'll lose some cross-platform QA people, those who have one primary
OS, but others specifically for cross platform work here. Dropping any OS X
version like this means anyone who wants to begin on the cheap with a used Mac,
or someone who already has but can't justify or afford paying for an OS upgrade,
would likely have to choose less QA do to lack of funds or difficulty finding
used with the minimum required OS X version. Mac people don't seem to ever get
rid of a machine before it becomes impossible to upgrade to a newer OS version.

Comment 7

[Josh Aas]

Support for Mac OS X 10.1 has officially been dropped in discussions with
cbeard, chofmann, and ben goodger.

Lets get stuff cleanup up ASAP for Firefox 1.1.

Comment 8

[Asaf Romano (gone)]

Attachment: [checked in] code cleanup

Comment 9

[Asaf Romano (gone)]

Attachment: diff -w

Comment 10

[Asaf Romano (gone)]

Attachment: [parttly landed] untested configure changes

Those should do the not-allowing-to-run under 10.1 magic

Comment 11

[Asaf Romano (gone)]

Comment on attachment 187079 [details] [diff] [review]
[checked in] code cleanup

r=josh sr=darin a=shaver on irc.

Comment 12

[Asaf Romano (gone)]

Comment on attachment 187079 [details] [diff] [review]
[checked in] code cleanup

checked in.

Comment 13

[Asaf Romano (gone)]

Comment on attachment 187083 [details] [diff] [review]
[parttly landed] untested configure changes

landed only the main configure.in change. r=josh a=shaver.

Comment 14

[Asaf Romano (gone)]

decided not to requier 10.2+ on nspr, for now.

Comment 15

[Malcolm Rowe]

There's still a few references to MAC_OS_X_VERSION_10_2_HEX that look like they
could probably be removed.

http://lxr.mozilla.org/mozilla/ident?i=MAC_OS_X_VERSION_10_2_HEX