Fix for GrAALinearizingConvexPathRenderer incorrectly drawing non-convex paths

src/core/SkPathPriv.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkPathPriv_DEFINED
 #define SkPathPriv_DEFINED
 
@@ skipping 37 matching lines @@
      *  cheapComputDirection() and if that computed direction matches the
      *  specified direction. If dir is kUnknown, returns true if the direction
      *  cannot be computed.
      */
     static bool CheapIsFirstDirection(const SkPath& path, FirstDirection dir) {
         FirstDirection computedDir = kUnknown_FirstDirection;
         (void)CheapComputeFirstDirection(path, &computedDir);
         return computedDir == dir;
     }
 
-    static bool LastVerbIsClose(const SkPath& path) {
-        int count = path.countVerbs();
-        return count >= 1 && path.fPathRef->verbs()[~(count - 1)] == SkPath::Verb::kClose_Verb;
+    static bool IsClosedSingleContour(const SkPath& path) {
+        int verbCount = path.countVerbs();
+        if (verbCount == 0)
+            return false;
+        int moveCount = 0;
+        auto verbs = path.fPathRef->verbs();
+        for (int i = 0; i < verbCount; i++) {

[reed1, 2015/12/14 18:52:09]

Is this loop traversing forwards or backwards through the verbs (hard to know
since we store them funnily)?

+            auto verb = verbs[~i];
+            switch (verb) {
+                case SkPath::Verb::kMove_Verb:
+                    moveCount += 1;
+                    if (moveCount > 1) {
+                        return false;
+                    }
+                    break;
+                case SkPath::Verb::kClose_Verb:
+                    if (i == verbCount - 1) {
+                        return true;
+                    }
+                    return false;
+                default: break;
+            }
+        }
+        return false;
     }
 
     static void AddGenIDChangeListener(const SkPath& path, SkPathRef::GenIDChangeListener* listener) {
         path.fPathRef->addGenIDChangeListener(listener);
     }
 };
 
 #endif