Avoid double pixel-ratio scaling of dbus tray icons

QIcon::pixmap does automatic scaling to DPR by default, so
use QIconEngine API to access internal pixmaps directly instead.

Pick-to: 5.15
Task-number: QTBUG-83806
Change-Id: I3ccbed8387811a87bbea3f5d4ad9963e1be28a49
Reviewed-by: Dmitry Shachnev <mitya57@gmail.com>
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>

2 files changed, 11 insertions, 10 deletions

diff --git a/src/gui/platform/unix/dbustray/qdbustrayicon.cpp b/src/gui/platform/unix/dbustray/qdbustrayicon.cpp
index e8fcb83c38..9d222ffef6 100644
--- a/src/gui/platform/unix/dbustray/qdbustrayicon.cpp
+++ b/src/gui/platform/unix/dbustray/qdbustrayicon.cpp
@@ -213,10 +213,9 @@ QTemporaryFile *QDBusTrayIcon::tempIcon(const QIcon &icon)
     }
     if (!necessary)
         return nullptr;
-    qreal dpr = qGuiApp->devicePixelRatio();
     QTemporaryFile *ret = new QTemporaryFile(tempFileTemplate(), this);
     ret->open();
-    icon.pixmap(QSize(22 * dpr, 22 * dpr)).save(ret);
+    icon.pixmap(QSize(22, 22)).save(ret);
     ret->close();
     return ret;
 }
diff --git a/src/gui/platform/unix/dbustray/qdbustraytypes.cpp b/src/gui/platform/unix/dbustray/qdbustraytypes.cpp
index 97cc8b7f36..3ebe7a1236 100644
--- a/src/gui/platform/unix/dbustray/qdbustraytypes.cpp
+++ b/src/gui/platform/unix/dbustray/qdbustraytypes.cpp
@@ -46,6 +46,7 @@
 #include <QDBusMetaType>
 #include <QImage>
 #include <QIcon>
+#include <QIconEngine>
 #include <QImage>
 #include <QPixmap>
 #include <QDebug>
@@ -54,6 +55,7 @@
 #include <QGuiApplication>
 #include <qpa/qplatformmenu.h>
 #include <private/qdbusplatformmenu_p.h>
+#include <private/qicon_p.h>
 
 QT_BEGIN_NAMESPACE
 
@@ -64,35 +66,35 @@ static const int IconNormalMediumSize = 64;
 QXdgDBusImageVector iconToQXdgDBusImageVector(const QIcon &icon)
 {
     QXdgDBusImageVector ret;
-    QList<QSize> sizes = icon.availableSizes();
+    QIconEngine *engine = const_cast<QIcon &>(icon).data_ptr()->engine;
+    QList<QSize> sizes = engine->availableSizes(QIcon::Normal, QIcon::Off);
 
     // Omit any size larger than 64 px, to save D-Bus bandwidth;
     // ensure that 22px or smaller exists, because it's a common size;
     // and ensure that something between 22px and 64px exists, for better scaling to other sizes.
     bool hasSmallIcon = false;
     bool hasMediumIcon = false;
-    qreal dpr = qGuiApp->devicePixelRatio();
     QList<QSize> toRemove;
     for (const QSize &size : qAsConst(sizes)) {
         int maxSize = qMax(size.width(), size.height());
-        if (maxSize <= IconNormalSmallSize * dpr)
+        if (maxSize <= IconNormalSmallSize)
             hasSmallIcon = true;
-        else if (maxSize <= IconNormalMediumSize * dpr)
+        else if (maxSize <= IconNormalMediumSize)
             hasMediumIcon = true;
-        else if (maxSize > IconSizeLimit * dpr)
+        else if (maxSize > IconSizeLimit)
             toRemove << size;
     }
     for (const QSize &size : qAsConst(toRemove))
         sizes.removeOne(size);
     if (!hasSmallIcon)
-        sizes.append(QSize(IconNormalSmallSize * dpr, IconNormalSmallSize * dpr));
+        sizes.append(QSize(IconNormalSmallSize, IconNormalSmallSize));
     if (!hasMediumIcon)
-        sizes.append(QSize(IconNormalMediumSize * dpr, IconNormalMediumSize * dpr));
+        sizes.append(QSize(IconNormalMediumSize, IconNormalMediumSize));
 
     ret.reserve(sizes.size());
     for (const QSize &size : qAsConst(sizes)) {
         // Protocol specifies ARGB32 format in network byte order
-        QImage im = icon.pixmap(size).toImage().convertToFormat(QImage::Format_ARGB32);
+        QImage im = engine->pixmap(size, QIcon::Normal, QIcon::Off).toImage().convertToFormat(QImage::Format_ARGB32);
         // letterbox if necessary to make it square
         if (im.height() != im.width()) {
             int maxSize = qMax(im.width(), im.height());