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

Review-Url: https://codereview.chromium.org/2355833002

diff --git a/src/gpu/GrShape.cpp b/src/gpu/GrShape.cpp
index 65c982571d48bf7110bcce1a2d91883402703655..582858e1d9608b506669ecb3ee9ae7a9c5c4c55d 100644 (file)
--- 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) {