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

Side by Side Diff: tests/PathTest.cpp

Issue 1533873002: Use the unswapped end point y for early out case in winding_line (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years 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 unified diff | Download patch
« no previous file with comments | « src/core/SkPath.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2011 Google Inc. 2 * Copyright 2011 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "SkCanvas.h" 8 #include "SkCanvas.h"
9 #include "SkGeometry.h" 9 #include "SkGeometry.h"
10 #include "SkPaint.h" 10 #include "SkPaint.h"
(...skipping 3524 matching lines...) Expand 10 before | Expand all | Expand 10 after
3535 // test on edge 3535 // test on edge
3536 REPORTER_ASSERT(reporter, p.contains(4, 6)); 3536 REPORTER_ASSERT(reporter, p.contains(4, 6));
3537 REPORTER_ASSERT(reporter, p.contains(6, 5)); 3537 REPORTER_ASSERT(reporter, p.contains(6, 5));
3538 REPORTER_ASSERT(reporter, p.contains(6, 7)); 3538 REPORTER_ASSERT(reporter, p.contains(6, 7));
3539 // test various crossings in y 3539 // test various crossings in y
3540 REPORTER_ASSERT(reporter, !p.contains(7, 5)); 3540 REPORTER_ASSERT(reporter, !p.contains(7, 5));
3541 REPORTER_ASSERT(reporter, p.contains(7, 6)); 3541 REPORTER_ASSERT(reporter, p.contains(7, 6));
3542 REPORTER_ASSERT(reporter, !p.contains(7, 7)); 3542 REPORTER_ASSERT(reporter, !p.contains(7, 7));
3543 p.reset(); 3543 p.reset();
3544 p.moveTo(4, 4); 3544 p.moveTo(4, 4);
3545 p.lineTo(8, 4);
3546 p.lineTo(8, 8);
3547 p.lineTo(4, 8);
3548 // test on vertices
3549 REPORTER_ASSERT(reporter, p.contains(4, 4));
3550 REPORTER_ASSERT(reporter, p.contains(8, 4));
3551 REPORTER_ASSERT(reporter, p.contains(8, 8));
3552 REPORTER_ASSERT(reporter, p.contains(4, 8));
3553 p.reset();
3554 p.moveTo(4, 4);
3545 p.lineTo(6, 8); 3555 p.lineTo(6, 8);
3546 p.lineTo(2, 8); 3556 p.lineTo(2, 8);
3547 // test on edge 3557 // test on edge
3548 REPORTER_ASSERT(reporter, p.contains(5, 6)); 3558 REPORTER_ASSERT(reporter, p.contains(5, 6));
3549 REPORTER_ASSERT(reporter, p.contains(4, 8)); 3559 REPORTER_ASSERT(reporter, p.contains(4, 8));
3550 REPORTER_ASSERT(reporter, p.contains(3, 6)); 3560 REPORTER_ASSERT(reporter, p.contains(3, 6));
3551 p.reset(); 3561 p.reset();
3552 p.moveTo(4, 4); 3562 p.moveTo(4, 4);
3553 p.lineTo(0, 6); 3563 p.lineTo(0, 6);
3554 p.lineTo(4, 8); 3564 p.lineTo(4, 8);
(...skipping 449 matching lines...) Expand 10 before | Expand all | Expand 10 after
4004 PathTest_Private::TestPathTo(reporter); 4014 PathTest_Private::TestPathTo(reporter);
4005 PathRefTest_Private::TestPathRef(reporter); 4015 PathRefTest_Private::TestPathRef(reporter);
4006 PathTest_Private::TestPathrefListeners(reporter); 4016 PathTest_Private::TestPathrefListeners(reporter);
4007 test_dump(reporter); 4017 test_dump(reporter);
4008 test_path_crbug389050(reporter); 4018 test_path_crbug389050(reporter);
4009 test_path_crbugskia2820(reporter); 4019 test_path_crbugskia2820(reporter);
4010 test_skbug_3469(reporter); 4020 test_skbug_3469(reporter);
4011 test_skbug_3239(reporter); 4021 test_skbug_3239(reporter);
4012 test_bounds_crbug_513799(reporter); 4022 test_bounds_crbug_513799(reporter);
4013 } 4023 }
OLDNEW
« no previous file with comments | « src/core/SkPath.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698