Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

iPhone 15 Pro (Max) supports AV1 #6837

Merged
merged 2 commits into from
Sep 27, 2023
Merged

iPhone 15 Pro (Max) supports AV1 #6837

merged 2 commits into from
Sep 27, 2023

Conversation

tats-u
Copy link
Contributor

@tats-u tats-u commented Sep 23, 2023

https://webkit.org/blog/14445/webkit-features-in-safari-17-0/

AV1

Safari 17.0 adds support for AV1 video on devices with hardware decoding support, like iPhone 15 Pro and iPhone 15 Pro Max.

I don't know whether iOS Safari supports AV1 in MP4 container too.

https://www.reddit.com/r/AV1/comments/141oqx1/new_apple_m2_promax_chips_dont_support_av1_decode/

M2 chip (Desktop/Notebook/iPad Pro) doesn't have AV1 support, so macOS shouldn't be "partially support" as of now.

The CPU in final Intel Mac is 8th gen. Intel CPU started AV1 hardware decoding in 11th gen.


Request to Viewers:

If you have iPhone 15 Pro (Max), I'd appreciate if you access to https://tests.caniuse.com/av1 by your iPhone, try playing video there, and report whether you could play it or not.

@Schweinepriester
Copy link
Contributor

Thank you for your research and the call to action in the end, but currently arguably duplicate of #6836 0:-)

@tats-u
Copy link
Contributor Author

tats-u commented Sep 23, 2023

@Schweinepriester I know. I'll close this if you adopt my change (footnote) in your PR and add Co-authored-by Tatsunori Uchino <[email protected]>.
Or you can remove changes on av1.json from your PR.

@tats-u
Copy link
Contributor Author

tats-u commented Sep 23, 2023

We must avoid misleading users of iPhone 15 (Plus) or Mac or iPad with the M1/M2 chip that they can watch AV1 videos by Safari first.

@tats-u
Copy link
Contributor Author

tats-u commented Sep 23, 2023

@Schweinepriester I have to tell you I noticed your PR after submitting this.
I believe I posted this via the VS Code extension.

@Fyrd
Copy link
Owner

Fyrd commented Sep 25, 2023

@tats-u Just saw this PR after merging @Schweinepriester 's one, happy to include your updated note if you don't mind updating your PR.

@tats-u
Copy link
Contributor Author

tats-u commented Sep 25, 2023

@Fyrd I'm willing to rebase my PR

@@ -584,7 +584,7 @@
"1":"Can be enabled in Edge on Windows 10 or later by installing the [AV1 Video Extension from the Microsoft Store](https://www.microsoft.com/store/productId/9MVZQVXJBQ9V)",
"2":"Firefox 65 only supported AV1 on Windows 64-bit",
"3":"Firefox 66 only supported AV1 on Windows and macOS",
"4":"Partial support refers to supporting only devices with hardware decoding support"
"4":"Supported only for devices with hardware decoder i.e. A17 Pro (in e.g. iPhone 15 _Pro_ (Max)) or later"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Fyrd Does this expression have to be improved?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nah it's good, thanks!

@jensimmons
Copy link
Contributor

What does this mean?
_Pro_ (Max))

Why the underscores and parentheses?

@tats-u
Copy link
Contributor Author

tats-u commented Sep 25, 2023

_Pro_ means Pro, and "Pro (Max)" means Pro or Pro Max
Markdown is available there, isn't it?

Do you have a proposed change?

Copy link
Owner

@Fyrd Fyrd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Fyrd Fyrd merged commit fa02eec into Fyrd:main Sep 27, 2023
1 check passed
@tats-u tats-u deleted the av1-iphone branch September 27, 2023 05:56
@tats-u tats-u mentioned this pull request Nov 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants