Make cap only affect the keys of GrShapes that are possibly-open

src/gpu/GrStyle.h

 /*
  * Copyright 2016 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrStyle_DEFINED
 #define GrStyle_DEFINED
 
@@ skipping 30 matching lines @@
     static const GrStyle& SimpleHairline() {
         static const GrStyle kHairline(SkStrokeRec::kHairline_InitStyle);
         return kHairline;
     }
 
     enum class Apply {
         kPathEffectOnly,
         kPathEffectAndStrokeRec
     };
 
     /**

[robertphillips, 2016/05/04 20:27:00]

do to -> due to ?

[bsalomon, 2016/05/04 20:29:22]

Done.

+     * Optional flags for computing keys that may remove unnecessary variation in the key do to
+     * style settings that don't affect particular classes of geometry.
+     */
+    enum KeyFlags {
+        // The shape being styled has no open contours.
+        kClosed_KeyFlag = 0x1
+    };
+
+    /**
      * Computes the key length for a GrStyle. The return will be negative if it cannot be turned
      * into a key. This occurs when there is a path effect that is not a dash. The key can
      * either reflect just the path effect (if one) or the path effect and the strokerec. Note
      * that a simple fill has a zero sized key.
      */
-    static int KeySize(const GrStyle& , Apply);
+    static int KeySize(const GrStyle& , Apply, uint32_t flags = 0);

[bsalomon, 2016/05/04 20:17:10]

KeySize doesn't actually use this but figured it was best to keep the params
between KeySize and WriteKey in sync in case it matters for new flags.

 
     /**
      * Writes a unique key for the style into the provided buffer. This function assumes the buffer
      * has room for at least KeySize() values. It assumes that KeySize() returns a non-negative
-     * value for the style and Apply param. This is written so that the key for just dash
-     * application followed by the key for the remaining SkStrokeRec is the same as the key for
-     * applying dashing and SkStrokeRec all at once.
+     * value for the combination of GrStyle, Apply and flags params. This is written so that the key
+     * for just dash application followed by the key for the remaining SkStrokeRec is the same as
+     * the key for applying dashing and SkStrokeRec all at once.
      */
-    static void WriteKey(uint32_t*, const GrStyle&, Apply);
+    static void WriteKey(uint32_t*, const GrStyle&, Apply, uint32_t flags = 0);
 
     GrStyle() : GrStyle(SkStrokeRec::kFill_InitStyle) {}
 
     explicit GrStyle(SkStrokeRec::InitStyle initStyle) : fStrokeRec(initStyle) {}
 
     GrStyle(const SkStrokeRec& strokeRec, SkPathEffect* pe) : fStrokeRec(strokeRec) {
         SkASSERT(SkStrokeRec::kStrokeAndFill_Style != strokeRec.getStyle());
         this->initPathEffect(pe);
     }
 
@@ skipping 94 matching lines @@
         SkScalar                    fPhase;
         SkAutoSTArray<4, SkScalar>  fIntervals;
     };
 
     SkStrokeRec         fStrokeRec;
     sk_sp<SkPathEffect> fPathEffect;
     DashInfo            fDashInfo;
 };
 
 #endif