Make WordPress Core

Opened 4 years ago

Last modified 2 years ago

#53663 new defect (bug)

Media: fix and improve WebP features detection

Reported by: azaozz's profile azaozz Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Media Keywords: needs-patch
Focuses: Cc:

Description

Follow up from #53653.

Looking at https://developers.google.com/speed/webp/docs/riff_container, the "extended" WebP file format VP8X may include a lossy or lossless "bitstream" chunk (VP8 or VP8L) and few optional chunks: ALPH, EXIF, ICCP, ANIM, etc.

Ideally the wp_get_webp_info() function should be extended to detect these features.

Change History (10)

This ticket was mentioned in Slack in #core-media by desrosj. View the logs.


4 years ago

#2 @desrosj
4 years ago

  • Keywords needs-patch added

#3 @desrosj
4 years ago

  • Milestone changed from Future Release to 5.8.1

Moving to 5.8.1 to see if there's any improvement that can be made in the next minor release.

This ticket was mentioned in Slack in #core-media by pbearne. View the logs.


4 years ago

#5 follow-up: @pbearne
4 years ago

Can anybody provide a set of images that we can use to test this with?
or point to where I can get/create them

#6 @desrosj
3 years ago

  • Milestone changed from 5.8.1 to 5.8.2

With 5.8.1 RC in less than 24 hours and no patch, I'm going to punt to 5.8.2.

#7 in reply to: ↑ 5 @sourovroy
3 years ago

Replying to pbearne:

Can anybody provide a set of images that we can use to test this with?
or point to where I can get/create them

I have collected a few images you may analyze with these https://drive.google.com/drive/folders/1humlxtfXOzaBGJ66Y6Iz8nsSObj4DZ1c?usp=sharing

#8 @pbearne
3 years ago

Thankyou @sourovroy

A couple of questions
Are these royalty free images?

and what is the extra info that each image has?

We are looking for images with extra chunks of image data

Paul

#9 @desrosj
3 years ago

  • Milestone changed from 5.8.2 to Future Release

This one still needs testing and a patch, and 5.8.2 RC is due out tomorrow. Moving to Future Release.

#10 @abitofmind
2 years ago

I wanted to bring this to your attention:

  • Will this feature here ensure all 3 criteria as proposed in #57238 ?
  • Feel free to copy/merge the sample files and comments from there to here if appropriate.
Note: See TracTickets for help on using tickets.