1284007 - Remove obsolete OS X version checks in dom/media

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect, P3)

Description

[Stefan [:stefanh]]

Attachment: Remove obsolete code

We don't support 10.6-10.8 anymore and I want to remove the OS X (now obsolete) version checks from nsCocoaFeatures.h (bug 1282251).

Comment 1

[Jean-Yves Avenard [:jya]]

To me there's a difference between no longer supporting a particular OS version and explicitly killing it. 

Removing this will kill video playback on 10.6

Comment 2

[Stefan [:stefanh]]

I don't understand the reason to keep video playback on 10.6.

Firefox doesn't even start on < 10.9 anymore due to this: https://dxr.mozilla.org/mozilla-central/source/browser/app/macbuild/Contents/Info.plist.in#209 (same for all comm-central apps). And I doubt that any mozilla app will run on 10.6 even if you reverted that change, because a lot of code specific to 10.6-10.8 have already been removed in widget/cocoa.

Comment 3

[Stefan [:stefanh]]

For code that's already been removed, see for example https://hg.mozilla.org/mozilla-central/rev/4ffe615e05a8 (sandboxing) and https://hg.mozilla.org/mozilla-central/rev/22f34da35017 (widget/cocoa).

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8767417 [details] [diff] [review]
Remove obsolete code

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

jesup can you look at the changes to dom/media/MediaManager.cpp please?

Anthony Jones assures me that we aren't updating existing Firefox installs on now-unsupported versions of MacOSX. So I'm fine with removing this code from Media Playback code.

This patch would have been significantly easier to review in mozreview. Please use that for review requests.

::: dom/media/platforms/apple/AppleVDADecoder.cpp
@@ +71,2 @@
>  #endif
> +  , mIs106(false)

Please remove mIs106 from AppleVDADecoder.h, and remove the "if (mIs106)" block in AppleVDADecoder::DoDecode(); we should never go down that path now.

Comment 5

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Mass change P2 -> P3

Comment 6

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 8767417 [details] [diff] [review]
Remove obsolete code

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

::: dom/media/MediaManager.cpp
@@ +1876,5 @@
>        case MediaSourceEnum::Application:
>        case MediaSourceEnum::Window:
>          // Deny screensharing request if support is disabled, or
>          // the requesting document is not from a host on the whitelist, or
> +        // we're on WinXP until proved that it work

work -> works

Comment 7

[Stefan [:stefanh]]

Thanks for the reviews. I haven't gotten to mozreview yet, but I guess I should...

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a41833214621
Remove obsolete OS X version checks in dom/media. r=cpearce, jesup.

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/a41833214621