diff options
author | Johan Klokkhammer Helsing <johan.helsing@qt.io> | 2018-03-13 13:04:40 +0100 |
---|---|---|
committer | Johan Helsing <johan.helsing@qt.io> | 2018-03-19 08:23:54 +0000 |
commit | efa6e989125b3a9cf54a408aae8a86ef4bddf778 (patch) | |
tree | 8960e3999024da8e5d73a9439c470be6781e35b2 /src/gui/image/qpixmap_raster_p.h | |
parent | d3121e3888ae43a7dc074f5d8acc26b1973a7726 (diff) |
QPixmap don't assume QPlatformScreen::format is opaque
QRasterPlatformPixmap::systemOpaqueFormat returned QPlatformScreen::format
without checking that the format was actually opaque.
This caused several QPixmap tests to fail on Wayland because Wayland
compositors don't communicate the native format of the screen, just a list of
supported pixel formats, so we just return ARGB32_premultiplied in
QWaylandScreen::format().
Rename the method systemOpaqueFormat to systemNativeFormat since that's how
it's used most of the time. And do a conversion when we actually care whether
the format is opaque or not.
Task-number: QTBUG-51748
Change-Id: I47dc1c3f185fb802016ca361206d47d02e8d3cf1
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'src/gui/image/qpixmap_raster_p.h')
-rw-r--r-- | src/gui/image/qpixmap_raster_p.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gui/image/qpixmap_raster_p.h b/src/gui/image/qpixmap_raster_p.h index cff962181a..fe2a1e581d 100644 --- a/src/gui/image/qpixmap_raster_p.h +++ b/src/gui/image/qpixmap_raster_p.h @@ -88,7 +88,7 @@ protected: void createPixmapForImage(QImage sourceImage, Qt::ImageConversionFlags flags); void setImage(const QImage &image); QImage image; - static QImage::Format systemOpaqueFormat(); + static QImage::Format systemNativeFormat(); private: friend class QPixmap; |