commit d1b9f5c33a12ff15a99e9831427a8d80d4c7c8c2
author Hui Yingst <nigi@chromium.org> Tue Jun 11 16:22:32 2019 +0000
committer Chromium commit bot <commit-bot@chromium.org> Tue Jun 11 16:22:32 2019 +0000
tree 02d0bd93b0cd9295f4f5d6e6e75bd1ad39f79bef
parent bfc6f146c3434be0116fcdfb573b1f174da3c288

[SkiaPath] Fix issue with missing line.

In bug_1296.pdf, the black line was drawn, but not visible. This was
because it was still using the matrix from the previous operation that
drew the green line. To fix this, reset |SkiaState::m_drawMatrix| to
the identity matrix in FlushDraw() and FlushText().

BUG=pdfium:1296
Change-Id: Ie93f8e1055d10210dfe08f53bc20f48d837360d6
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/54690
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

core/fxge/skia/fx_skia_device.cpp

diff --git a/core/fxge/skia/fx_skia_device.cpp b/core/fxge/skia/fx_skia_device.cpp
index 75fb55e..21f3b31 100644
--- a/core/fxge/skia/fx_skia_device.cpp
+++ b/core/fxge/skia/fx_skia_device.cpp
@@ -792,6 +792,7 @@
     }
     m_drawIndex = INT_MAX;
     m_type = Accumulator::kNone;
+    m_drawMatrix = CFX_Matrix();
   }
 
   bool HasRSX(int nChars,
@@ -955,6 +956,7 @@
 
     m_drawIndex = INT_MAX;
     m_type = Accumulator::kNone;
+    m_drawMatrix = CFX_Matrix();
   }
 
   bool IsEmpty() const { return !m_commands.count(); }