commit 9cf31f40fb5df6b6b914e087394c40b8303e5e32
author brianosman <brianosman@google.com> Fri Mar 25 12:55:52 2016 -0700
committer Commit bot <commit-bot@chromium.org> Fri Mar 25 12:55:52 2016 -0700
tree 482dda75bde1107885d6142e4aab0631c70e70b3
parent 397536cabe12a9936659870dd220c869789424ba

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

diff --git a/tests/MipMapTest.cpp b/tests/MipMapTest.cpp
index 4d45746..467fd34 100644
--- a/tests/MipMapTest.cpp
+++ b/tests/MipMapTest.cpp
@@ -20,10 +20,8 @@
     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<SkMipMap> mm(SkMipMap::Build(bm, nullptr));
 
@@ -118,7 +116,7 @@
         {-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},