path is rect bug number twelve

Exposes that final close along a diagonal need not
include a close verb if the subsequent verb is move;
so we have to check for a diagonal then.

The later check for diagonal included a comment that
it may not be needed which does appear to be the case.

R=robertphillips@google.com
Bug: 824145,skia:7792
Change-Id: I17a9414e8b3e69b82c2eda28195696eae4e3d513
Reviewed-on: https://skia-review.googlesource.com/121801
Commit-Queue: Cary Clark <caryclark@skia.org>
Commit-Queue: Robert Phillips <robertphillips@google.com>
Auto-Submit: Cary Clark <caryclark@skia.org>
Reviewed-by: Robert Phillips <robertphillips@google.com>

1 files changed, 10 insertions, 0 deletions

diff --git a/gm/pathfill.cpp b/gm/pathfill.cpp
index 0fcffdc23f..db1f3f2557 100644
--- a/gm/pathfill.cpp
+++ b/gm/pathfill.cpp
@@ -558,4 +558,14 @@ DEF_SIMPLE_GM(bug7792, canvas, 600, 800) {
     path.moveTo(75, 75);
     path.close();
     canvas->drawPath(path, p);
+    // from skbug.com/7792#c36
+    canvas->translate(200, 0);
+    path.reset();
+    path.moveTo(75, 75);
+    path.lineTo(150, 75);
+    path.lineTo(150, 150);
+    path.lineTo(10, 150);
+    path.moveTo(75, 75);
+    path.lineTo(75, 75);
+    canvas->drawPath(path, p);
 }