Make WordPress Core

Opened 23 months ago

Last modified 6 months 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.


23 months ago

#2 @desrosj
23 months ago

  • Keywords needs-patch added

#3 @desrosj
23 months 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.


23 months ago

#5 follow-up: @pbearne
23 months 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
21 months 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
21 months 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
20 months 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
19 months 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
6 months 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.