Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(53)

Unified Diff: gm/conicpaths.cpp

Issue 835593002: enable conics gm (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | gyp/gmslides.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gm/conicpaths.cpp
diff --git a/gm/conicpaths.cpp b/gm/conicpaths.cpp
index 96242faf5f9a5a04fd24de1c9f3af2a30cd89e37..d04cd263598f321f16cfaf86f65b2ccca1e64695 100644
--- a/gm/conicpaths.cpp
+++ b/gm/conicpaths.cpp
@@ -17,14 +17,14 @@ protected:
}
SkISize onISize() SK_OVERRIDE {
- return SkISize::Make(950, 1000);
+ return SkISize::Make(920, 960);
}
void onOnceBeforeDraw() SK_OVERRIDE {
{
const SkScalar w = SkScalarSqrt(2)/2;
SkPath* conicCirlce = &fPaths.push_back();
- conicCirlce->moveTo(0, -0);
+ conicCirlce->moveTo(0, 0);
conicCirlce->conicTo(0, 50, 50, 50, w);
conicCirlce->rConicTo(50, 0, 50, -50, w);
conicCirlce->rConicTo(0, -50, -50, -50, w);
@@ -33,43 +33,43 @@ protected:
}
{
SkPath* hyperbola = &fPaths.push_back();
- hyperbola->moveTo(0, -0);
+ hyperbola->moveTo(0, 0);
hyperbola->conicTo(0, 100, 100, 100, 2);
}
{
SkPath* thinHyperbola = &fPaths.push_back();
- thinHyperbola->moveTo(0, -0);
+ thinHyperbola->moveTo(0, 0);
thinHyperbola->conicTo(100, 100, 5, 0, 2);
}
{
SkPath* veryThinHyperbola = &fPaths.push_back();
- veryThinHyperbola->moveTo(0, -0);
+ veryThinHyperbola->moveTo(0, 0);
veryThinHyperbola->conicTo(100, 100, 1, 0, 2);
}
{
SkPath* closedHyperbola = &fPaths.push_back();
- closedHyperbola->moveTo(0, -0);
+ closedHyperbola->moveTo(0, 0);
closedHyperbola->conicTo(100, 100, 0, 0, 2);
}
{
// using 1 as weight defaults to using quadTo
SkPath* nearParabola = &fPaths.push_back();
- nearParabola->moveTo(0, -0);
+ nearParabola->moveTo(0, 0);
nearParabola->conicTo(0, 100, 100, 100, 0.999f);
}
{
SkPath* thinEllipse = &fPaths.push_back();
- thinEllipse->moveTo(0, -0);
+ thinEllipse->moveTo(0, 0);
thinEllipse->conicTo(100, 100, 5, 0, SK_ScalarHalf);
}
{
SkPath* veryThinEllipse = &fPaths.push_back();
- veryThinEllipse->moveTo(0, -0);
+ veryThinEllipse->moveTo(0, 0);
veryThinEllipse->conicTo(100, 100, 1, 0, SK_ScalarHalf);
}
{
SkPath* closedEllipse = &fPaths.push_back();
- closedEllipse->moveTo(0, -0);
+ closedEllipse->moveTo(0, 0);
closedEllipse->conicTo(100, 100, 0, 0, SK_ScalarHalf);
}
}
@@ -77,58 +77,39 @@ protected:
void onDraw(SkCanvas* canvas) SK_OVERRIDE {
const SkAlpha kAlphaValue[] = { 0xFF, 0x40 };
- SkScalar maxH = 0;
const SkScalar margin = 15;
canvas->translate(margin, margin);
- canvas->save();
- SkScalar x = margin;
- int counter = 0;
+ SkPaint paint;
for (int p = 0; p < fPaths.count(); ++p) {
+ canvas->save();
for (size_t a = 0; a < SK_ARRAY_COUNT(kAlphaValue); ++a) {
+ paint.setARGB(kAlphaValue[a], 0, 0, 0);
for (int aa = 0; aa < 2; ++aa) {
+ paint.setAntiAlias(SkToBool(aa));
for (int fh = 0; fh < 2; ++fh) {
+ paint.setStyle(fh ? SkPaint::kStroke_Style : SkPaint::kFill_Style);
const SkRect& bounds = fPaths[p].getBounds();
-
- SkPaint paint;
- paint.setARGB(kAlphaValue[a], 0, 0, 0);
- paint.setAntiAlias(SkToBool(aa));
- if (fh == 1) {
- paint.setStyle(SkPaint::kStroke_Style);
- paint.setStrokeWidth(0);
- } else if (fh == 0) {
- paint.setStyle(SkPaint::kFill_Style);
- }
canvas->save();
canvas->translate(-bounds.fLeft, -bounds.fTop);
canvas->drawPath(fPaths[p], paint);
canvas->restore();
- maxH = SkMaxScalar(maxH, bounds.height());
-
- SkScalar dx = bounds.width() + margin;
- x += dx;
- canvas->translate(dx, 0);
-
- if (++counter == 8) {
- counter = 0;
-
- canvas->restore();
- canvas->translate(0, maxH + margin);
- canvas->save();
- maxH = 0;
- x = margin;
- }
+ canvas->translate(110, 0);
}
}
}
+ canvas->restore();
+ canvas->translate(0, 110);
}
canvas->restore();
}
uint32_t onGetFlags() const SK_OVERRIDE {
- return kSkipPDF_Flag;
+ // tiling w/ non-antialias paths can cause off-by-1-pixels differences which are
+ // unavoidable (chopping in floats -vs- stepping in scan-converter).
+ return kSkipTiled_Flag;
}
private:
« no previous file with comments | « no previous file | gyp/gmslides.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698