[Libreoffice-commits] core.git: vcl/headless

[Tor Lillqvist (via logerrit)]

 vcl/headless/svpinst.cxx |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 837879ec7b1f3cdfee2b7af4ec44885db0d115e6
Author:     Tor Lillqvist <tml at collabora.com>
AuthorDate: Thu Jul 16 10:38:18 2020 +0300
Commit:     Tor Lillqvist <tml at collabora.com>
CommitDate: Thu Jul 16 13:14:53 2020 +0200

    tdf#134832: Don't set mbSupportsBitmap32 on iOS
    
    Our Quartz code sure doesn't handle such bitmaps, as far as I know.
    
    (Yeah, iOS itself of course handles such bitmaps with included alpha
    (both the venerable Core Graphics stuff, not to mention all kinds of
    more modern things), but not our code.)
    
    This does not fix the bug in question, but clearly setting
    mbSupportsBitmap32 for iOS was unintentional. (It is possible that it
    would be relatively easy to just fix our Quartz code to actually
    handle such bitmaps. Then mbSupportsBitmap32 could be turned on again.
    Later.)
    
    Change-Id: I278a4c1ad1c3e882a769d9d054f3f19e1a976666
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/98881
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
    Reviewed-by: Tor Lillqvist <tml at collabora.com>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/98884
    Tested-by: Jenkins

diff --git a/vcl/headless/svpinst.cxx b/vcl/headless/svpinst.cxx
index daaa4d170b3e..055b7cad9dfc 100644
--- a/vcl/headless/svpinst.cxx
+++ b/vcl/headless/svpinst.cxx
@@ -571,7 +571,9 @@ void SvpSalInstance::AddToRecentDocumentList(const OUString&, const OUString&, c
 std::shared_ptr<vcl::BackendCapabilities> SvpSalInstance::GetBackendCapabilities()
 {
     auto pBackendCapabilities = SalInstance::GetBackendCapabilities();
+#ifndef IOS
     pBackendCapabilities->mbSupportsBitmap32 = true;
+#endif
     return pBackendCapabilities;
 }