sd/qa/unit/export-tests.cxx | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
New commits: commit 1c19af307c60ed82d9d56e7b4a0f74b04be53a5a Author: Luboš Luňák <l.lu...@collabora.com> AuthorDate: Wed Nov 27 11:10:35 2019 +0100 Commit: Luboš Luňák <l.lu...@collabora.com> CommitDate: Fri Nov 29 10:54:07 2019 +0100 do not completely disable a test for !OpenGL if OpenGL may be broken The code comes from f062b89b277a0830f038cafdd13bd074b5188455 and is more like patching things up. A proper fix should be elsewhere, but at least patch this up more to be less wrong. Change-Id: Ic9b24680688d104028b1e1bf6fccc16e6f786aa8 Reviewed-on: https://gerrit.libreoffice.org/83880 Tested-by: Jenkins Reviewed-by: Luboš Luňák <l.lu...@collabora.com> diff --git a/sd/qa/unit/export-tests.cxx b/sd/qa/unit/export-tests.cxx index 9f15a20e47c4..b9a32f453452 100644 --- a/sd/qa/unit/export-tests.cxx +++ b/sd/qa/unit/export-tests.cxx @@ -21,6 +21,7 @@ #include <rtl/ustring.hxx> #include <vcl/opengl/OpenGLWrapper.hxx> +#include <vcl/skia/SkiaHelper.hxx> #include <com/sun/star/drawing/XDrawPage.hpp> #include <com/sun/star/drawing/XDrawPagesSupplier.hpp> @@ -280,8 +281,12 @@ void SdExportTest::testTransparentBackground() void SdExportTest::testMediaEmbedding() { - if (!OpenGLWrapper::isVCLOpenGLEnabled()) +#ifdef _WIN32 + // This seems broken. This test should not be disabled for all cases except when OpenGL + // is found to be working, just because in some OpenGL setups this breaks (per the commit log message). + if (!OpenGLWrapper::isVCLOpenGLEnabled() && !SkiaHelper::isVCLSkiaEnabled()) return; +#endif ::sd::DrawDocShellRef xDocShRef = loadURL(m_directories.getURLFromSrc("/sd/qa/unit/data/media_embedding.odp"), ODP); _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits