diff options
author | Thorsten Behrens <tbehrens@suse.com> | 2013-06-14 13:26:43 +0200 |
---|---|---|
committer | Tor Lillqvist <tml@iki.fi> | 2013-06-20 19:28:29 +0300 |
commit | 77b273ee3a2df87664262418c360d3a4a0ed48ea (patch) | |
tree | 257c765d00c2296fe7410fceb2dff17691dbb134 /vcl | |
parent | fbb7c74a2c5cd37d4ff27deb154656655f047bbc (diff) |
Fixup alpha bitmap generation for Windows.
This improves upon d6f58fd25eeca84a94528409a05b80aa5172b8b8 by
using the BitmapEx ctor instead of manual initialisation. Several
EMF+ files would otherwise show up horribly mangled on Windows.
Change-Id: I1808cb45818f3f0118653164c83c49d6134b9324
(cherry picked from commit fefab3bb38f55471d9fa9316d1249ad00adefc37)
Diffstat (limited to 'vcl')
-rw-r--r-- | vcl/source/gdi/bitmapex.cxx | 34 |
1 files changed, 5 insertions, 29 deletions
diff --git a/vcl/source/gdi/bitmapex.cxx b/vcl/source/gdi/bitmapex.cxx index 227e630f7732..e123ac215146 100644 --- a/vcl/source/gdi/bitmapex.cxx +++ b/vcl/source/gdi/bitmapex.cxx @@ -787,9 +787,6 @@ bool BitmapEx::Create( const ::com::sun::star::uno::Reference< ::com::sun::star::rendering::XBitmapCanvas > &xBitmapCanvas, const Size &rSize ) { - SetEmpty(); - Size aSize( rSize ); - uno::Reference< beans::XFastPropertySet > xFastPropertySet( xBitmapCanvas, uno::UNO_QUERY ); if( xFastPropertySet.get() ) { @@ -809,38 +806,17 @@ bool BitmapEx::Create( const ::com::sun::star::uno::Reference< pSalBmp = ImplGetSVData()->mpDefInst->CreateSalBitmap(); pSalMask = ImplGetSVData()->mpDefInst->CreateSalBitmap(); - if( pSalBmp->Create( xBitmapCanvas, aSize ) ) + Size aLocalSize(rSize); + if( pSalBmp->Create( xBitmapCanvas, aLocalSize ) ) { -#ifdef CLAMP_BITDEPTH_PARANOIA - // did we get alpha mixed up in the bitmap itself - // eg. Cairo Canvas ... yes performance of this is awful. - if( pSalBmp->GetBitCount() > 24 ) - { - // Format convert the pixels with generic code - Bitmap aSrcPixels( pSalBmp ); - aBitmap = Bitmap( rSize, 24 ); - BitmapReadAccess aSrcRead( aSrcPixels ); - BitmapWriteAccess aDestWrite( aBitmap ); - aDestWrite.CopyBuffer( aSrcRead ); - } - else -#endif - aBitmap = Bitmap( pSalBmp ); - - aBitmapSize = rSize; - if ( pSalMask->Create( xBitmapCanvas, aSize, true ) ) + if ( pSalMask->Create( xBitmapCanvas, aLocalSize, true ) ) { - aMask = Bitmap( pSalMask ); - bAlpha = sal_True; - aBitmapSize = rSize; - eTransparent = !aMask ? TRANSPARENT_NONE : TRANSPARENT_BITMAP; - + *this = BitmapEx(Bitmap(pSalBmp), Bitmap(pSalMask) ); return true; } else { - bAlpha = sal_False; - eTransparent = TRANSPARENT_NONE; + *this = BitmapEx(Bitmap(pSalBmp)); return true; } } |