commit d4228473a552dc2e683182c47a957977f53df581
author Cary Clark <caryclark@skia.org> Fri Apr 13 07:07:04 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Apr 13 14:01:32 2018 +0000
tree d87c15c4b2888c410e9d0ed74fe05a71de4ed78d
parent d1999cacd8a1b0c29288f08cea9c0c812c8de9ad

path is rect bug number nine

This variation exploits a sequence which uses a zero
length line to note that lines have been recorded, but
no rectangle edge has been encountered.

R=robertphillips@google.com

Docs-Preview: https://skia.org/?cl=121282
Bug: 824145,skia:7792
Change-Id: I652e9482b2867c3d7da30d5f5df2aecbfd0d716d
Reviewed-on: https://skia-review.googlesource.com/121282
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Cary Clark <caryclark@skia.org>

gm/pathfill.cpp

diff --git a/gm/pathfill.cpp b/gm/pathfill.cpp
index 74cafcb..1bf396c 100644
--- a/gm/pathfill.cpp
+++ b/gm/pathfill.cpp
@@ -443,7 +443,7 @@
     path.lineTo(150, 150);
     path.lineTo(75, 150);
     canvas->drawPath(path, p);
-    // from skbug.com/7792 comment 3
+    // from skbug.com/7792#c3
     canvas->translate(200, 0);
     path.reset();
     path.moveTo(75, 50);
@@ -454,7 +454,7 @@
     path.lineTo(75, 50);
     path.close();
     canvas->drawPath(path, p);
-    // from skbug.com/7792 comment 9
+    // from skbug.com/7792#c9
     canvas->translate(200, 0);
     path.reset();
     path.moveTo(10, 10);
@@ -464,7 +464,7 @@
     path.lineTo(75, 150);
     path.close();
     canvas->drawPath(path, p);
-    // from skbug.com/7792 comment 11
+    // from skbug.com/7792#c11
     canvas->translate(-200 * 2, 200);
     path.reset();
     path.moveTo(75, 150);
@@ -474,7 +474,7 @@
     path.lineTo(75, 150);
     path.moveTo(75, 150);
     canvas->drawPath(path, p);
-    // from skbug.com/7792 comment 14
+    // from skbug.com/7792#c14
     canvas->translate(200, 0);
     path.reset();
     path.moveTo(250, 75);
@@ -489,7 +489,7 @@
     path.lineTo(0, 0);
     path.close();
     canvas->drawPath(path, p);
-    // from skbug.com/7792 comment 15
+    // from skbug.com/7792#c15
     canvas->translate(200, 0);
     path.reset();
     path.moveTo(75, 75);
@@ -498,7 +498,7 @@
     path.lineTo(75, 150);
     path.moveTo(250, 75);
     canvas->drawPath(path, p);
-    // from skbug.com/7792 comment 17
+    // from skbug.com/7792#c17
     canvas->translate(-200 * 2, 200);
     path.reset();
     path.moveTo(75, 10);
@@ -509,7 +509,7 @@
     path.lineTo(75, 10);
     path.close();
     canvas->drawPath(path, p);
-    // from skbug.com/7792 comment 19
+    // from skbug.com/7792#c19
     canvas->translate(200, 0);
     path.reset();
     path.moveTo(75, 75);
@@ -524,4 +524,16 @@
     path.lineTo(30, 10);
     path.lineTo(10, 30);
     canvas->drawPath(path, p);
+    // from skbug.com/7792#c23
+    canvas->translate(200, 0);
+    path.reset();
+    path.moveTo(75, 75);
+    path.lineTo(75, 75);
+    path.moveTo(75, 75);
+    path.lineTo(75, 75);
+    path.lineTo(150, 75);
+    path.lineTo(150, 150);
+    path.lineTo(75, 150);
+    path.close();
+    canvas->drawPath(path, p);
 }