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

Unified Diff: tests/RecordDrawTest.cpp

Issue 772533004: Change clear() to respect the clip (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: rebase Created 6 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 side-by-side diff with in-line comments
Download patch
« tests/DeferredCanvasTest.cpp ('K') | « tests/DeferredCanvasTest.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/RecordDrawTest.cpp
diff --git a/tests/RecordDrawTest.cpp b/tests/RecordDrawTest.cpp
index 48981ef0a208a1554810833f0c04ed59acc5cc4b..2b6a0f9fc8378b7fdcc8bdd1cc7efce737dd36a7 100644
--- a/tests/RecordDrawTest.cpp
+++ b/tests/RecordDrawTest.cpp
@@ -202,6 +202,7 @@ DEF_TEST(RecordDraw_PartialStartStop, r) {
REPORTER_ASSERT(r, drawRect->rect == r2);
}
+#if 0
mtklein 2014/12/02 15:27:37 Same?
reed1 2014/12/02 16:21:15 Seems like I should just remove this, since clear(
// Check that clears are converted to drawRects
DEF_TEST(RecordDraw_PartialClear, r) {
static const int kWidth = 10, kHeight = 10;
@@ -225,6 +226,7 @@ DEF_TEST(RecordDraw_PartialClear, r) {
REPORTER_ASSERT(r, drawRect->rect == rect);
REPORTER_ASSERT(r, drawRect->paint.getColor() == SK_ColorRED);
}
+#endif
// A regression test for crbug.com/415468 and skbug.com/2957.
//
« tests/DeferredCanvasTest.cpp ('K') | « tests/DeferredCanvasTest.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698