Merge lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk-fix-icon-svg-image-loading into lp:ubuntu-ui-toolkit/staging

[Michał Sawicz]

Adjust the UnityThemeIconProvider to duplicate Qt's own image loading behaviour, fixes some image scaling issues

The Pixmap->Image thing might be unnecessary, but
1) I wanted to duplicate exactly Qt's behaviour and
2) Qt converts the QPixmap to a QImage internally before uploading to texture

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:1282
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/501/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2794/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/501/rebuild

review: Needs Fixing (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:1282
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/720/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2795/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/720/rebuild

review: Needs Fixing (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1282
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/687/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2796/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/687/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:1282
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/475/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2797/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/475/rebuild

review: Needs Fixing (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:1282
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/316/
Executed test runs:
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/687
        deb: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/687/artifact/artifacts/*zip*/artifacts.zip
    FAILURE: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/720/console
    FAILURE: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/501/console
    FAILURE: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/475/console
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2798/console
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2796
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2795
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2794
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2797

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/316/rebuild

review: Needs Fixing (continuous-integration)

[lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk-fix-icon-svg-image-loading]

1283. By Michał Sawicz on 2016-04-21

Make tst_IconProvider talk in QImage

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:1283
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/505/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2814/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/505/rebuild

review: Needs Fixing (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:1283
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/724/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2815/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/724/rebuild

review: Needs Fixing (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:1283
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/479/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2816/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/479/rebuild

review: Needs Fixing (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1283
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/691/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2817/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/691/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:1283
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/320/
Executed test runs:
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/691
        deb: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/691/artifact/artifacts/*zip*/artifacts.zip
    FAILURE: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/724/console
    FAILURE: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/505/console
    FAILURE: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/479/console
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2818/console
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2817
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2815
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2814
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2816

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/320/rebuild

review: Needs Fixing (continuous-integration)

[lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk-fix-icon-svg-image-loading]

1284. By Lukáš Tinkl on 2016-04-22

fix loading & scaling SVGs

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1284
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/737/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2877/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/737/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1284
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/518/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2878/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/518/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1284
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/492/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2879/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/492/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1284
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/704/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2880/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/704/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1284
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/331/
Executed test runs:
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/704
        deb: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/704/artifact/artifacts/*zip*/artifacts.zip
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/737
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/518
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/492
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2881/console
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2880
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2877
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2878
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/2879

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/331/rebuild

review: Approve (continuous-integration)

[lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk-fix-icon-svg-image-loading]

1285. By Lukáš Tinkl on 2016-04-25

add a test with wrong aspect ratio that returns a correct one

1286. By Lukáš Tinkl on 2016-04-27

add a test that passes here and fails with trunk

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1286
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/549/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/3025/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/549/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1286
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/768/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/3026/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/768/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1286
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/523/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/3027/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/523/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1286
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/737/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/3028/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/737/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:1286
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/353/
Executed test runs:
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/737
        deb: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-armhf-stable/737/artifact/artifacts/*zip*/artifacts.zip
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/768
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/549
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-i386-gles-stable/523
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/3029/console
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/3028
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/3026
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/3025
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/3027

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/353/rebuild

review: Approve (continuous-integration)

[Michał Sawicz (saviq)]

Replaced with https://code.launchpad.net/~lukas-kde/ubuntu-ui-toolkit/staging-fix-icon-svg-image-loading/+merge/293281

review: Disapprove

Unmerged revisions

=== modified file 'src/Ubuntu/Components/plugin/unitythemeiconprovider.cpp'
--- src/Ubuntu/Components/plugin/unitythemeiconprovider.cpp	2016-03-16 14:24:15 +0000
+++ src/Ubuntu/Components/plugin/unitythemeiconprovider.cpp	2016-04-27 16:15:41 +0000
@@ -21,8 +21,7 @@
 #include <QDir>
 #include <QFileInfo>
 #include <QSettings>
-#include <QSvgRenderer>
-#include <QPainter>
+#include <QImageReader>
 #include <QStandardPaths>
 #include <QtDebug>
 
@@ -48,27 +47,27 @@
     // Does a breadth-first search for an icon with any name in @names. Parent
     // themes are only looked at if the current theme doesn't contain any icon
     // in @names.
-    QPixmap findBestIcon(const QStringList &names, const QSize &size, QSet<QString> *alreadySearchedThemes)
+    QImage findBestIcon(const QStringList &names, QSize *impsize, const QSize &size, QSet<QString> *alreadySearchedThemes)
     {
         if (alreadySearchedThemes) {
             if (alreadySearchedThemes->contains(name))
-                return QPixmap();
+                return QImage();
             alreadySearchedThemes->insert(name);
         }
 
         Q_FOREACH(const QString &name, names) {
-            QPixmap pixmap = lookupIcon(name, size);
-            if (!pixmap.isNull())
-                return pixmap;
+            QImage image = lookupIcon(name, impsize, size);
+            if (!image.isNull())
+                return image;
         }
 
         Q_FOREACH(IconThemePointer theme, parents) {
-            QPixmap pixmap = theme->findBestIcon(names, size, alreadySearchedThemes);
-            if (!pixmap.isNull())
-                return pixmap;
+            QImage image = theme->findBestIcon(names, impsize, size, alreadySearchedThemes);
+            if (!image.isNull())
+                return image;
         }
 
-        return QPixmap();
+        return QImage();
     }
 
 private:
@@ -82,7 +81,7 @@
 
     IconTheme(const QString &name): name(name)
     {
-        QStringList paths = QStandardPaths::standardLocations(QStandardPaths::GenericDataLocation);
+        const QStringList paths = QStandardPaths::standardLocations(QStandardPaths::GenericDataLocation);
 
         Q_FOREACH(const QString &path, paths) {
             QDir dir(path + "/icons/" + name);
@@ -130,30 +129,42 @@
         return Fixed;
     }
 
-    static QPixmap loadIcon(const QString &filename, const QSize &size)
+    static QImage loadIcon(const QString &filename, QSize *impsize, const QSize &requestSize)
     {
-        QPixmap pixmap;
-
-        const bool anyZero = size.width() <= 0 || size.height() <= 0;
-        const Qt::AspectRatioMode scaleMode = anyZero ? Qt::KeepAspectRatioByExpanding : Qt::KeepAspectRatio;
-
-        if (filename.endsWith(QLatin1String(".png"))) {
-            pixmap = QPixmap(filename);
-            if (!pixmap.isNull() && !size.isNull() && (pixmap.width() != size.width() || pixmap.height() != size.height())) {
-                const QSize newSize = pixmap.size().scaled(size.width(), size.height(), scaleMode);
-                pixmap = pixmap.scaled(newSize);
-            }
-        }
-        else if (filename.endsWith(QLatin1String(".svg"))) {
-            QSvgRenderer renderer(filename);
-            pixmap = QPixmap(renderer.defaultSize().scaled(size.width(), size.height(), scaleMode));
-            pixmap.fill(Qt::transparent);
-            QPainter painter(&pixmap);
-            renderer.render(&painter);
-            painter.end();
-        }
-
-        return pixmap;
+        QImage image;
+        QImageReader imgio(filename);
+
+        const bool force_scale = imgio.format() == QStringLiteral("svg")
+            || imgio.format() == QStringLiteral("svgz");
+
+        if (requestSize.width() > 0 || requestSize.height() > 0) {
+            QSize s = imgio.size();
+            qreal ratio = 0.0;
+            if (requestSize.width() > 0 && (force_scale || requestSize.width() < s.width())) {
+                ratio = qreal(requestSize.width())/s.width();
+            }
+            if (requestSize.height() > 0 && (force_scale || requestSize.height() < s.height())) {
+                qreal hr = qreal(requestSize.height())/s.height();
+                if (ratio == 0.0 || hr < ratio)
+                    ratio = hr;
+            }
+            if (ratio > 0.0) {
+                s.setHeight(qRound(s.height() * ratio));
+                s.setWidth(qRound(s.width() * ratio));
+                imgio.setScaledSize(s);
+            }
+        }
+
+        if (impsize)
+            *impsize = imgio.scaledSize();
+
+        if (imgio.read(&image)) {
+            if (impsize)
+                *impsize = image.size();
+            return image;
+        } else {
+            return QImage();
+        }
     }
 
     QString lookupIconFile(const QString &dir, const QString &name)
@@ -174,16 +185,16 @@
         return QString();
     }
 
-    QPixmap lookupIcon(const QString &iconName, const QSize &size)
+    QImage lookupIcon(const QString &iconName, QSize *impsize, const QSize &size)
     {
         const int iconSize = qMax(size.width(), size.height());
         if (iconSize > 0)
-            return lookupBestMatchingIcon(iconName, size);
+            return lookupBestMatchingIcon(iconName, impsize, size);
         else
-            return lookupLargestIcon(iconName);
+            return lookupLargestIcon(iconName, impsize);
     }
 
-    QPixmap lookupBestMatchingIcon(const QString &iconName, const QSize &size)
+    QImage lookupBestMatchingIcon(const QString &iconName, QSize *impsize, const QSize &size)
     {
         int minDistance = 10000;
         QString bestFilename;
@@ -205,12 +216,12 @@
         }
 
         if (!bestFilename.isNull())
-            return loadIcon(bestFilename, size);
+            return loadIcon(bestFilename, impsize, size);
 
-        return QPixmap();
+        return QImage();
     }
 
-    QPixmap lookupLargestIcon(const QString &iconName)
+    QImage lookupLargestIcon(const QString &iconName, QSize *impsize)
     {
         int maxSize = 0;
         QString bestFilename;
@@ -228,9 +239,9 @@
         }
 
         if (!bestFilename.isNull())
-            return loadIcon(bestFilename, QSize(maxSize, maxSize));
+            return loadIcon(bestFilename, impsize, QSize(maxSize, maxSize));
 
-        return QPixmap();
+        return QImage();
     }
 
     int directorySizeDistance(const Directory &dir, const QSize &iconSize)
@@ -258,24 +269,23 @@
 };
 
 UnityThemeIconProvider::UnityThemeIconProvider(const QString &themeName):
-  QQuickImageProvider(QQuickImageProvider::Pixmap)
+  QQuickImageProvider(QQuickImageProvider::Image)
 {
     theme = IconTheme::get(themeName);
 }
 
-QPixmap UnityThemeIconProvider::requestPixmap(const QString &id, QSize *size, const QSize &requestedSize)
+QImage UnityThemeIconProvider::requestImage(const QString &id, QSize *size, const QSize &requestedSize)
 {
     // The hicolor theme will be searched last as per
     // https://specifications.freedesktop.org/icon-theme-spec/icon-theme-spec-latest.html
     QSet<QString> alreadySearchedThemes;
     const QStringList names = id.split(QLatin1Char(','), QString::SkipEmptyParts);
-    QPixmap pixmap = theme->findBestIcon(names, requestedSize, &alreadySearchedThemes);
+    QImage image = theme->findBestIcon(names, size, requestedSize, &alreadySearchedThemes);
 
-    if (pixmap.isNull()) {
+    if (image.isNull()) {
         IconTheme::IconThemePointer theme = IconTheme::get(QStringLiteral("hicolor"));
-        return theme->findBestIcon(names, requestedSize, nullptr);
+        return theme->findBestIcon(names, size, requestedSize, nullptr);
     }
 
-    *size = pixmap.size();
-    return pixmap;
+    return image;
 }

=== modified file 'src/Ubuntu/Components/plugin/unitythemeiconprovider.h'
--- src/Ubuntu/Components/plugin/unitythemeiconprovider.h	2016-03-16 14:24:15 +0000
+++ src/Ubuntu/Components/plugin/unitythemeiconprovider.h	2016-04-27 16:15:41 +0000
@@ -25,7 +25,7 @@
 {
 public:
     UnityThemeIconProvider(const QString &themeName = QStringLiteral("suru"));
-    QPixmap requestPixmap(const QString &id, QSize *size, const QSize &requestedSize);
+    QImage requestImage(const QString &id, QSize *size, const QSize &requestedSize);
 
 private:
     QSharedPointer<class IconTheme> theme;

=== modified file 'tests/unit_x11/tst_iconprovider/tst_iconprovider.cpp'
--- tests/unit_x11/tst_iconprovider/tst_iconprovider.cpp	2016-03-16 15:02:35 +0000
+++ tests/unit_x11/tst_iconprovider/tst_iconprovider.cpp	2016-04-27 16:15:41 +0000
@@ -39,14 +39,15 @@
         QTest::addColumn<QSize>("requestSize");
         QTest::addColumn<QSize>("resultSize");
 
-        QTest::newRow("battery0") << "battery-100-charging" << QSize(-1, -1) << QSize(256, 165);
-        QTest::newRow("battery1") << "battery-100-charging" << QSize(-1, 16) << QSize(24, 16);
+        QTest::newRow("battery0") << "battery-100-charging" << QSize(-1, -1) << QSize(256, 166);
+        QTest::newRow("battery1") << "battery-100-charging" << QSize(-1, 16) << QSize(25, 16);
         QTest::newRow("battery2") << "battery-100-charging" << QSize(16, -1) << QSize(16, 10);
-        QTest::newRow("battery3") << "battery-100-charging" << QSize(0, 16) << QSize(24, 16);
+        QTest::newRow("battery3") << "battery-100-charging" << QSize(0, 16) << QSize(25, 16);
         QTest::newRow("battery4") << "battery-100-charging" << QSize(16, 0) << QSize(16, 10);
         QTest::newRow("battery5") << "battery-100-charging" << QSize(24, 16) << QSize(24, 16);
-        QTest::newRow("battery6") << "battery-100-charging" << QSize(24, 24) << QSize(24, 15);
+        QTest::newRow("battery6") << "battery-100-charging" << QSize(24, 24) << QSize(24, 16);
         QTest::newRow("battery7") << "battery-100-charging" << QSize(37, 24) << QSize(37, 24);
+        QTest::newRow("battery8") << "battery-100-charging" << QSize(15, 512) << QSize(15, 10);
 
         QTest::newRow("gallery0") << "gallery-app" << QSize(-1, -1) << QSize(512, 512);
         QTest::newRow("gallery1") << "gallery-app" << QSize(-1, 16) << QSize(16, 16);
@@ -65,13 +66,13 @@
 
         UnityThemeIconProvider provider("mockTheme");
         QSize returnedSize;
-        QPixmap p = provider.requestPixmap(icon, &returnedSize, requestSize);
-        QCOMPARE(p.size(), resultSize);
+        QImage i = provider.requestImage(icon, &returnedSize, requestSize);
+        QCOMPARE(i.size(), resultSize);
         QCOMPARE(returnedSize, resultSize);
 
         // Search again to make sure subsequent searches still work
-        p = provider.requestPixmap(icon, &returnedSize, requestSize);
-        QCOMPARE(p.size(), resultSize);
+        i = provider.requestImage(icon, &returnedSize, requestSize);
+        QCOMPARE(i.size(), resultSize);
         QCOMPARE(returnedSize, resultSize);
     }
 
@@ -82,14 +83,14 @@
 
         // myapp is in MockTheme3 (white) and hicolor (black)
         // MockTheme3 one is returned since hicolor is last per spec
-        QPixmap p = provider.requestPixmap("myapp", &returnedSize, QSize(-1, -1));
-        QVERIFY(!p.isNull());
-        QCOMPARE(QColor(p.toImage().pixel(0,0)), QColor(Qt::white));
+        QImage i = provider.requestImage("myapp", &returnedSize, QSize(-1, -1));
+        QVERIFY(!i.isNull());
+        QCOMPARE(QColor(i.pixel(0,0)), QColor(Qt::white));
 
         // myapp2 is only in hicolor (black) so that's the one returned
-        p = provider.requestPixmap("myapp2", &returnedSize, QSize(-1, -1));
-        QVERIFY(!p.isNull());
-        QCOMPARE(QColor(p.toImage().pixel(0,0)), QColor(Qt::black));
+        i = provider.requestImage("myapp2", &returnedSize, QSize(-1, -1));
+        QVERIFY(!i.isNull());
+        QCOMPARE(QColor(i.pixel(0,0)), QColor(Qt::black));
     }
 };