path_stroke_with_zero_length: turn off drawFat() by default.

The replay modes cannot readback, and so cannot draw the fat parts, which
means we've had to exempt them from checking that they draw identically.

I don't mind if we leave it alone, but I thought I'd check to see your opinion.

BUG=skia:4095
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1862113004

Review URL: https://codereview.chromium.org/1862113004
This commit is contained in:
mtklein 2016-04-06 14:25:35 -07:00 committed by Commit bot
parent 758586c7f1
commit 48156ed412
3 changed files with 2 additions and 244 deletions

View File

@ -91,7 +91,7 @@ protected:
if (indexMask & (1 << index)) { if (indexMask & (1 << index)) {
canvas->drawPath(path, paint); canvas->drawPath(path, paint);
} }
if (paint.getStrokeWidth() < 2) { if (this->getMode() == skiagm::GM::kSample_Mode && paint.getStrokeWidth() < 2) {
drawFat(canvas, path, paint, index); drawFat(canvas, path, paint, index);
} }
++index; ++index;

View File

@ -137,10 +137,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -161,18 +157,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -317,10 +301,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -341,18 +321,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -594,10 +562,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -618,18 +582,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -869,10 +821,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -893,18 +841,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -1147,10 +1083,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -1171,18 +1103,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -1418,10 +1338,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -1442,18 +1358,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -1691,10 +1595,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -1715,18 +1615,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -1960,10 +1848,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -1984,18 +1868,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -2176,10 +2048,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -2200,18 +2068,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -2485,10 +2341,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -2509,18 +2361,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -2670,10 +2510,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -2694,18 +2530,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -2858,10 +2682,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -2882,18 +2702,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -3063,10 +2871,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -3087,18 +2891,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -3267,10 +3059,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -3291,18 +3079,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",
@ -3597,10 +3373,6 @@
"serialize-8888", "serialize-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"serialize-8888",
"gm",
"_",
"verylargebitmap", "verylargebitmap",
"serialize-8888", "serialize-8888",
"gm", "gm",
@ -3621,18 +3393,6 @@
"sp-8888", "sp-8888",
"gm", "gm",
"_", "_",
"path_stroke_with_zero_length",
"pic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"2ndpic-8888",
"gm",
"_",
"path_stroke_with_zero_length",
"sp-8888",
"gm",
"_",
"patch_primitive", "patch_primitive",
"sp-8888", "sp-8888",
"gm", "gm",

View File

@ -168,13 +168,11 @@ def get_args(bot):
'fontmgr_match', 'fontmgr_match',
'fontmgr_iter', 'fontmgr_iter',
'lightingshader', 'lightingshader',
'path_stroke_with_zero_length',
'verylargebitmap', # Windows only. 'verylargebitmap', # Windows only.
'verylarge_picture_image']: # Windows only. 'verylarge_picture_image']: # Windows only.
blacklist.extend(['serialize-8888', 'gm', '_', test]) blacklist.extend(['serialize-8888', 'gm', '_', test])
# skia:4769 # skia:4769
for test in ['drawfilter', for test in ['drawfilter']:
'path_stroke_with_zero_length']:
blacklist.extend([ 'sp-8888', 'gm', '_', test]) blacklist.extend([ 'sp-8888', 'gm', '_', test])
blacklist.extend([ 'pic-8888', 'gm', '_', test]) blacklist.extend([ 'pic-8888', 'gm', '_', test])
blacklist.extend(['2ndpic-8888', 'gm', '_', test]) blacklist.extend(['2ndpic-8888', 'gm', '_', test])