New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 697596 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Bug



Sign in to add a comment

Image decoder on ios does not support choosing image frame size

Project Member Reported by tschumann@chromium.org, Mar 1 2017

Issue description

In https://codesearch.chromium.org/chromium/src/ios/chrome/browser/suggestions/ios_image_decoder_impl.mm,

DecodeImage() does take a desired_image_frame_size parameter but does not make use of it.
How can we support this?
It had an impact for loading ico files on clank (see crbug/690029)


 
Labels: OS-iOS
Status: Available (was: Untriaged)
Components: -UI>Browser>NewTabPage UI>Browser>ContentSuggestions
Project Member

Comment 3 by sheriffbot@chromium.org, Sep 27

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Owner: gambard@chromium.org
Status: Assigned (was: Untriaged)
The iOS image decoder doesn't explicitly decode images other than WebP. The image conversion is done by the system. The only parameter is the scale of the image. Is this still relevant?
not sure. On other platforms we had the problem with ico files containing multiple resolutions. If the preferred size was not specified, the smallest frame was returned. If that's no problem for iOS (i know little about the formats), then you can close this bug.
Cc: gambard@chromium.org
Owner: ----
Status: Available (was: Assigned)
I think the ico -> UIImage might use the smallest ico image available. However, I am not sure we are hitting this issue a lot.
I don't think it is worth fixing now, but I will keep it open.

Sign in to add a comment