CroppedImageMinimumSourceSize loads the image twice in the bad scenario

Bug #1604025 reported by Albert Astals Cid
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
qtdeclarative-opensource-src (Ubuntu)
Triaged
Undecided
Unassigned
unity8 (Ubuntu)
Triaged
Medium
Unassigned

Bug Description

To get the best looking possible Image sometimes we end up loading the Image twice, see http://lists.qt-project.org/pipermail/development/2016-July/026562.html for an in depth explanation.

Changed in unity8 (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Albert Astals Cid (aacid) wrote :

The upstream patch has been merged

https://codereview.qt-project.org/#/c/165299/

We need to figure out how much we want this and if we want to backport it to our Qt or not.

Changed in qtdeclarative-opensource-src (Ubuntu):
assignee: nobody → Timo Jyrinki (timo-jyrinki)
Changed in qtdeclarative-opensource-src (Ubuntu):
status: New → In Progress
Revision history for this message
Albert Astals Cid (aacid) wrote :

The issue we found was fixed in https://codereview.qt-project.org/#/c/181717/

Now given how we are de-empathizing a bit the dash and it'll be reworked, not sure if we want to backport this anymore.

Revision history for this message
Albert Astals Cid (aacid) wrote :

And if we backport this, we'll also need to backport https://codereview.qt-project.org/#/c/184501/ now that the UITK uses QQuickPixmap::isCached

Changed in qtdeclarative-opensource-src (Ubuntu):
status: In Progress → Triaged
Changed in qtdeclarative-opensource-src (Ubuntu):
assignee: Timo Jyrinki (timo-jyrinki) → nobody
Changed in unity8 (Ubuntu):
assignee: Albert Astals Cid (aacid) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.