New issue
Advanced search Search tips

Issue 715820 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature

Blocking:
issue 663569



Sign in to add a comment

Move PNGImageDecoder to SkCodec

Project Member Reported by cblume@chromium.org, Apr 26 2017

Issue description

Move PNGImageDecoder to SkCodec

SkCodec is gaining advantages over Blink's image decoders.
scroggo@ has a nice summary:

- Skia already has some features that Chromium is interested in:
  - decoding scaled versions of images
  - decoding subsets (i.e. issue 468914)
  - SIMD optimizations for writing pixels
- In general, sharing code means that new features/bug fixes in Android
  benefit Chromium and vice versa
- Skia's API is designed to allow the client to handle caching, so that
  the client that knows the bigger picture can make caching decisions in
  a more informed way than ImageDecoder can.
 

Comment 1 by cblume@chromium.org, Apr 26 2017

This is happening here: https://crrev.com/2782493002

Comment 2 by brat...@opera.com, May 29 2018

Did you do any more experiments with this?

Comment 3 by cblume@chromium.org, May 29 2018

Cc: cblume@chromium.org
Owner: scroggo@chromium.org
It got put on hold for a bit. I think scroggo@ will be working on this. No new experiments.

Sign in to add a comment