Remove unneeded png:ignore_text_chunks

This unblocks reading XMP metadata, which Adobe put into a generic text chunk for some reason. We're not going to question it. There may be other information that may be buried in text chunks of sorts. This option was previously set to ignore text chunks entirely since png<=0.17 would error on malformed chunks even when they were ancillary. But some real world images have UTF8 text in an tEXT Latin-1 chunk or similar dubious encoding issues. Those must not influence our ability to display such images though, they exist in the wild.

Author: 197g

src/codecs/png.rs

@@ -45,7 +45,6 @@ impl<R: BufRead + Seek> PngDecoder<R> {
 
  let max_bytes = usize::try_from(limits.max_alloc.unwrap_or(u64::MAX)).unwrap_or(usize::MAX);
  let mut decoder = png::Decoder::new_with_limits(r, png::Limits { bytes: max_bytes });
- decoder.set_ignore_text_chunk(true);
 
  let info = decoder.read_header_info().map_err(ImageError::from_png)?;
  limits.check_dimensions(info.width, info.height)?;