commit 12e760e958a7eac453e7bddd66d4000701d4f71d
author Herb Derby <herb@google.com> Tue Mar 02 11:28:19 2021 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Mar 02 21:58:08 2021 +0000
tree bbb61fe627ab73815ea98803c7945db19ebe5f01
parent 587efaeb1573ed26941ce5435ddab4402cfd939f

simplify check_integer_translate's API

Change-Id: I369ff838d0467b1447237c6721d4e380933f630b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/378376
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Herb Derby <herb@google.com>

src/gpu/text/GrTextBlob.cpp

diff --git a/src/gpu/text/GrTextBlob.cpp b/src/gpu/text/GrTextBlob.cpp
index 34811f5..12917eb 100644
--- a/src/gpu/text/GrTextBlob.cpp
+++ b/src/gpu/text/GrTextBlob.cpp
@@ -149,9 +149,7 @@
 
 // Check for integer translate with the same 2x2 matrix.
 std::tuple<bool, SkVector> check_integer_translate(
-        const GrTextBlob& blob, const SkMatrix& drawMatrix) {
-    const SkMatrix& initialMatrix = blob.initialMatrix();
-
+        const SkMatrix& initialMatrix, const SkMatrix& drawMatrix) {
     if (initialMatrix.getScaleX() != drawMatrix.getScaleX() ||
         initialMatrix.getScaleY() != drawMatrix.getScaleY() ||
         initialMatrix.getSkewX()  != drawMatrix.getSkewX()  ||
@@ -285,7 +283,7 @@
         return false;
     }
 
-    auto [reuse, _] = check_integer_translate(fBlob, drawMatrix);
+    auto [reuse, _] = check_integer_translate(fBlob.initialMatrix(), drawMatrix);
     return reuse;
 }
 
@@ -585,7 +583,7 @@
         return false;
     }
 
-    auto [reuse, translation] = check_integer_translate(*fBlob, drawMatrix);
+    auto [reuse, translation] = check_integer_translate(fBlob->initialMatrix(), drawMatrix);
 
     // If glyphs were excluded because of position bounds, then this subrun can only be reused if
     // there is no change in position.