internally resize image instead of scaling bitmaps outside

When we want a different size Image, we can now set that as a
parameter at construction of the Image. Previously we needed to
create an Image, forcefully take the bitmap out, resize the bitmap
and create a new Image out of that.
Doing it internally gives us the benefit to have a more control
over the scaling process, especially when dealing with HiDPI
images.

Change-Id: I104118f4d863d519cc7aad1a17ca0289c01ed9ff
Reviewed-on: https://gerrit.libreoffice.org/70617
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>

1 files changed, 18 insertions, 6 deletions

diff --git a/vcl/inc/image.h b/vcl/inc/image.h
index c9fb5393191a..c5acc06f782a 100644
--- a/vcl/inc/image.h
+++ b/vcl/inc/image.h
@@ -25,22 +25,31 @@
 #include <unordered_map>
 #include <vector>
 
-struct ImplImage
+class ImplImage
 {
+private:
     BitmapChecksum maBitmapChecksum;
     /// if non-empty: cached original size of maStockName else Size of maBitmap
-    Size     maSizePixel;
+    Size maSizePixel;
+    Size maPreferedSizePixel;
     /// If set - defines the bitmap via images.zip*
     OUString maStockName;
 
+
     /// Original bitmap - or cache of a potentially scaled bitmap
     BitmapEx maBitmapEx;
     BitmapEx maDisabledBitmapEx;
 
+    bool loadStockAtScale(double fScale, BitmapEx &rBitmapEx);
+
+public:
     ImplImage(const BitmapEx& rBitmapEx);
-    ImplImage(const OUString &aStockName);
+    ImplImage(const OUString &aStockName, Size const & rPreferedSize);
 
-    bool isStock() const { return maStockName.getLength() > 0; }
+    bool isStock() const
+    {
+        return maStockName.getLength() > 0;
+    }
 
     /// get size in co-ordinates not scaled for HiDPI
     Size getSizePixel();
@@ -48,9 +57,12 @@ struct ImplImage
     BitmapEx getBitmapEx(bool bDisabled = false);
     /// Taking account of HiDPI scaling
     BitmapEx getBitmapExForHiDPI(bool bDisabled = false);
+
     bool isEqual(const ImplImage &ref) const;
-    bool isSizeEmpty() const { return maSizePixel == Size(0, 0); }
-    bool loadStockAtScale(double fScale, BitmapEx &rBitmapEx);
+    bool isSizeEmpty() const
+    {
+        return maSizePixel == Size();
+    }
 };
 
 #endif // INCLUDED_VCL_INC_IMAGE_H