diff options
| author |  bsalomon <bsalomon@google.com> | 2016-09-21 07:47:34 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-09-21 07:47:34 -0700 |
| commit | 8eb43e5f63306a22570b977bbb69e4ec78432d24 (patch) | |
| tree | dec674c723a42e56f3e4ec4d69088509b78e4c8e /src/gpu/GrShape.cpp | |
| parent | e4a9bd79c676346ada8f4c35fa3a7fd99dfc5544 (diff) | |
Stop closing filled paths in GrShape
This can force a copy of the path data and isn't really being taken advantage of. It complicates a forthcoming change to key small paths based on their data rather than gen id.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2355833002
Committed: https://skia.googlesource.com/skia/+/ad001fdc77b2dfe32c1723cb58636728dfb776d8
Review-Url: https://codereview.chromium.org/2355833002
Diffstat (limited to 'src/gpu/GrShape.cpp')
| -rw-r--r-- | src/gpu/GrShape.cpp | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/gpu/GrShape.cpp b/src/gpu/GrShape.cpp index 65c982571d..582858e1d9 100644 --- a/src/gpu/GrShape.cpp +++ b/src/gpu/GrShape.cpp @@ -361,10 +361,6 @@ void GrShape::attemptToSimplifyPath() { } else { fPathData.fGenID = this->path().getGenerationID(); } - if (this->style().isSimpleFill()) { - this->path().close(); - this->path().setIsVolatile(true); - } if (!this->style().hasNonDashPathEffect()) { if (this->style().strokeRec().getStyle() == SkStrokeRec::kStroke_Style || this->style().strokeRec().getStyle() == SkStrokeRec::kHairline_Style) { |