From 9cf31f40fb5df6b6b914e087394c40b8303e5e32 Mon Sep 17 00:00:00 2001 From: brianosman Date: Fri, 25 Mar 2016 12:55:52 -0700 Subject: SkMipMap::Build supports 1xN and Nx1 inputs, so remove the special case from the test harness. Update a no-longer-correct comment. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1831983003 Review URL: https://codereview.chromium.org/1831983003 --- tests/MipMapTest.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'tests/MipMapTest.cpp') diff --git a/tests/MipMapTest.cpp b/tests/MipMapTest.cpp index 4d4574698e..467fd34998 100644 --- a/tests/MipMapTest.cpp +++ b/tests/MipMapTest.cpp @@ -20,10 +20,8 @@ DEF_TEST(MipMap, reporter) { SkRandom rand; for (int i = 0; i < 500; ++i) { - // for now, Build needs a min size of 2, otherwise it will return nullptr. - // should fix that to support 1 X N, where N > 1 to return non-null. - int width = 2 + rand.nextU() % 1000; - int height = 2 + rand.nextU() % 1000; + int width = 1 + rand.nextU() % 1000; + int height = 1 + rand.nextU() % 1000; make_bitmap(&bm, width, height); SkAutoTUnref mm(SkMipMap::Build(bm, nullptr)); @@ -118,7 +116,7 @@ DEF_TEST(MipMap_ComputeLevelCount, reporter) { {-100, -100, 0}, // Test mipmaps with 0, 1, 2 as dimensions - // (SkMipMap::Build requires a min size of 2) + // (SkMipMap::Build requires a min size of 1) // // 0 {0, 100, 0}, -- cgit v1.2.3