vcl/Library_vclplug_win.mk | 2 ++ vcl/inc/win/DWriteTextRenderer.hxx | 11 +++++++---- vcl/inc/win/winlayout.hxx | 2 +- vcl/win/gdi/DWriteTextRenderer.cxx | 25 ++++++++++++++++++------- vcl/win/gdi/winlayout.cxx | 16 +++++++++++----- 5 files changed, 39 insertions(+), 17 deletions(-)
New commits: commit 3a3032f560269118dcf6705ea8cc127a4418b348 Author: Miklos Vajna <vmik...@collabora.com> AuthorDate: Tue Apr 30 17:38:14 2019 +0200 Commit: Luboš Luňák <l.lu...@collabora.com> CommitDate: Wed May 8 10:04:14 2019 +0200 tdf#114209 vcl win DirectWrite: handle rotated text Commit a51b7a1c3a7e7cf7b0c733e1dec40288278c1884 (tdf#103831, tdf#100986: Force using GDI when needed, 2017-03-03) noted that the DirectWrite text renderer doesn't support vertical text, add initial support for this now by extending the DirectWrite transform matrix to do rotation as well. This is initial support, as it can be improved in two ways: - vertical text is not cached - only vertical Latin text is handled, which wants rotated glyphs (vs e.g. Japanese text that would not rotate the glyphs) With this, the "unreadable" text in the bugdoc's chart is on par with the the GDI rendering. Change-Id: I07af4de6cb437f83cc40546396ec8c8aac456bb3 Reviewed-on: https://gerrit.libreoffice.org/71592 Reviewed-by: Miklos Vajna <vmik...@collabora.com> Tested-by: Jenkins (cherry picked from commit f7453b956bcf83ec13c805d243f20cb209289179) Reviewed-on: https://gerrit.libreoffice.org/71894 Reviewed-by: Luboš Luňák <l.lu...@collabora.com> diff --git a/vcl/Library_vclplug_win.mk b/vcl/Library_vclplug_win.mk index 24f2d2491342..f64cc0392d13 100644 --- a/vcl/Library_vclplug_win.mk +++ b/vcl/Library_vclplug_win.mk @@ -82,6 +82,8 @@ $(eval $(call gb_Library_add_exception_objects,vclplug_win,\ )) $(eval $(call gb_Library_use_system_win32_libs,vclplug_win,\ + d2d1 \ + dwrite \ gdi32 \ gdiplus \ imm32 \ diff --git a/vcl/inc/win/DWriteTextRenderer.hxx b/vcl/inc/win/DWriteTextRenderer.hxx index 270c4772716f..845d2fe7b656 100644 --- a/vcl/inc/win/DWriteTextRenderer.hxx +++ b/vcl/inc/win/DWriteTextRenderer.hxx @@ -95,12 +95,15 @@ private: D2DTextAntiAliasMode meTextAntiAliasMode; }; -/// Sets and unsets the needed DirectWrite transform to support the font's horizontal scaling. -class WinFontStretchGuard +/** + * Sets and unsets the needed DirectWrite transform to support the font's horizontal scaling and + * rotation. + */ +class WinFontTransformGuard { public: - WinFontStretchGuard(ID2D1RenderTarget* pRenderTarget, float fHScale); - ~WinFontStretchGuard(); + WinFontTransformGuard(ID2D1RenderTarget* pRenderTarget, float fHScale, const GenericSalLayout& rLayout, const D2D1_POINT_2F& rBaseline); + ~WinFontTransformGuard(); private: ID2D1RenderTarget* mpRenderTarget; diff --git a/vcl/inc/win/winlayout.hxx b/vcl/inc/win/winlayout.hxx index 2bbd87a12206..1eac2a3c28e8 100644 --- a/vcl/inc/win/winlayout.hxx +++ b/vcl/inc/win/winlayout.hxx @@ -165,7 +165,7 @@ public: const WinFontFace * GetFontFace() const { return static_cast<const WinFontFace *>(LogicalFontInstance::GetFontFace()); } - bool CacheGlyphToAtlas(HDC hDC, HFONT hFont, int nGlyphIndex, SalGraphics& rGraphics); + bool CacheGlyphToAtlas(HDC hDC, HFONT hFont, int nGlyphIndex, SalGraphics& rGraphics, const GenericSalLayout& rLayout); OpenGLGlyphCache& GetOpenGLGlyphCache() { return maOpenGLGlyphCache; } bool GetGlyphOutline(sal_GlyphId, basegfx::B2DPolyPolygon&, bool) const override; diff --git a/vcl/win/gdi/DWriteTextRenderer.cxx b/vcl/win/gdi/DWriteTextRenderer.cxx index 50ec50897c80..c068d15676a5 100644 --- a/vcl/win/gdi/DWriteTextRenderer.cxx +++ b/vcl/win/gdi/DWriteTextRenderer.cxx @@ -273,7 +273,6 @@ bool D2DWriteTextOutRenderer::performRender(GenericSalLayout const & rLayout, Sa const WinFontInstance& rWinFont = static_cast<const WinFontInstance&>(rLayout.GetFont()); float fHScale = rWinFont.getHScale(); - WinFontStretchGuard aStretchGuard(mpRT, fHScale); tools::Rectangle bounds; bool succeeded = rLayout.GetBoundRect(bounds); @@ -305,6 +304,7 @@ bool D2DWriteTextOutRenderer::performRender(GenericSalLayout const & rLayout, Sa DWRITE_GLYPH_OFFSET glyphOffsets[] = { { 0.0f, 0.0f }, }; D2D1_POINT_2F baseline = { static_cast<FLOAT>(aPos.X() - bounds.Left()) / fHScale, static_cast<FLOAT>(aPos.Y() - bounds.Top()) }; + WinFontTransformGuard aTransformGuard(mpRT, fHScale, rLayout, baseline); DWRITE_GLYPH_RUN glyphs = { mpFontFace, mlfEmHeight, @@ -423,18 +423,29 @@ bool D2DWriteTextOutRenderer::GetDWriteFaceFromHDC(HDC hDC, IDWriteFontFace ** p return succeeded; } -WinFontStretchGuard::WinFontStretchGuard(ID2D1RenderTarget* pRenderTarget, float fHScale) +WinFontTransformGuard::WinFontTransformGuard(ID2D1RenderTarget* pRenderTarget, float fHScale, + const GenericSalLayout& rLayout, + const D2D1_POINT_2F& rBaseline) : mpRenderTarget(pRenderTarget) { pRenderTarget->GetTransform(&maTransform); - if (fHScale == 1.0f) - return; + D2D1::Matrix3x2F aTransform = maTransform; + if (fHScale != 1.0f) + { + aTransform + = aTransform * D2D1::Matrix3x2F::Scale(D2D1::Size(fHScale, 1.0f), D2D1::Point2F(0, 0)); + } - D2D1::Matrix3x2F aTransform - = maTransform * D2D1::Matrix3x2F::Scale(D2D1::Size(fHScale, 1.0f), D2D1::Point2F(0, 0)); + if (rLayout.GetOrientation() != 0) + { + // DWrite angle is in clockwise degrees, our orientation is in counter-clockwise 10th + // degrees. + aTransform + = aTransform * D2D1::Matrix3x2F::Rotation(-rLayout.GetOrientation() / 10, rBaseline); + } mpRenderTarget->SetTransform(aTransform); } -WinFontStretchGuard::~WinFontStretchGuard() { mpRenderTarget->SetTransform(maTransform); } +WinFontTransformGuard::~WinFontTransformGuard() { mpRenderTarget->SetTransform(maTransform); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/vcl/win/gdi/winlayout.cxx b/vcl/win/gdi/winlayout.cxx index b9453f7b96d1..7e61e40f0baf 100644 --- a/vcl/win/gdi/winlayout.cxx +++ b/vcl/win/gdi/winlayout.cxx @@ -58,7 +58,8 @@ GlobalOpenGLGlyphCache * GlobalOpenGLGlyphCache::get() return data->m_pGlobalOpenGLGlyphCache.get(); } -bool WinFontInstance::CacheGlyphToAtlas(HDC hDC, HFONT hFont, int nGlyphIndex, SalGraphics& rGraphics) +bool WinFontInstance::CacheGlyphToAtlas(HDC hDC, HFONT hFont, int nGlyphIndex, + SalGraphics& rGraphics, const GenericSalLayout& rLayout) { OpenGLGlyphDrawElement aElement; @@ -172,7 +173,7 @@ bool WinFontInstance::CacheGlyphToAtlas(HDC hDC, HFONT hFont, int nGlyphIndex, S 0 }; - WinFontStretchGuard aStretchGuard(pRT, fHScale); + WinFontTransformGuard aTransformGuard(pRT, fHScale, rLayout, baseline); pRT->BeginDraw(); pRT->DrawGlyphRun(baseline, &glyphs, pBrush); HRESULT hResult = pRT->EndDraw(); @@ -407,6 +408,10 @@ bool WinSalGraphics::CacheGlyphs(const GenericSalLayout& rLayout) if (!bDoGlyphCaching) return false; + if (rLayout.GetOrientation()) + // Our caching is incomplete, skip it for non-horizontal text. + return false; + HDC hDC = getHDC(); WinFontInstance& rFont = *static_cast<WinFontInstance*>(&rLayout.GetFont()); HFONT hFONT = rFont.GetHFONT(); @@ -418,7 +423,7 @@ bool WinSalGraphics::CacheGlyphs(const GenericSalLayout& rLayout) { if (!rFont.GetOpenGLGlyphCache().IsGlyphCached(pGlyph->m_aGlyphId)) { - if (!rFont.CacheGlyphToAtlas(hDC, hFONT, pGlyph->m_aGlyphId, *this)) + if (!rFont.CacheGlyphToAtlas(hDC, hFONT, pGlyph->m_aGlyphId, *this, rLayout)) return false; } } @@ -479,8 +484,9 @@ void WinSalGraphics::DrawTextLayout(const GenericSalLayout& rLayout) const HFONT hLayoutFont = pWinFont->GetHFONT(); bool bUseOpenGL = OpenGLHelper::isVCLOpenGLEnabled() && !mbPrinter; - // Our DirectWrite renderer is incomplete, skip it for non-horizontal text. - bool bForceGDI = rLayout.GetOrientation(); + // Our DirectWrite renderer is incomplete, skip it for vertical text where glyphs are not + // rotated. + bool bForceGDI = rLayout.GetFont().GetFontSelectPattern().mbVertical; if (!bUseOpenGL) { _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits