signal no clipping required
The original code had the fill vertex method handling the clip, but
each glyph was clipped individually. Signal no clipping at all is need
when the sub run is entirely enclosed in the clip rect.
Change-Id: Ifc7174fe7af967c9ab7240b2c7e2c0b062d7e259
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/310556
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Herb Derby <herb@google.com>
diff --git a/src/gpu/text/GrTextBlob.cpp b/src/gpu/text/GrTextBlob.cpp
index 6c2c9c8..8efb3b5 100644
--- a/src/gpu/text/GrTextBlob.cpp
+++ b/src/gpu/text/GrTextBlob.cpp
@@ -337,6 +337,10 @@
if (result.fIsRRect && result.fRRect.isRect() && result.fAA == GrAA::kNo) {
// Clip geometrically during onPrepare using clipRect.
result.fRRect.getBounds().round(&clipRect);
+ if (clipRect.contains(subRunBounds)) {
+ // If fully within the clip, then signal no clipping using the empty rect.
+ clipRect = SkIRect::MakeEmpty();
+ }
clip = nullptr;
}
} else if (result.fEffect == GrClip::Effect::kClippedOut) {
