aboutsummaryrefslogtreecommitdiffhomepage
path: root/tests
diff options
context:
space:
mode:
authorGravatar cblume <cblume@chromium.org>2016-06-03 11:59:50 -0700
committerGravatar Commit bot <commit-bot@chromium.org>2016-06-03 11:59:50 -0700
commit44e09ec810e6d28df87caa3df61cf6fbfd806544 (patch)
tree12ac17c24b37183c0b20f559de0ae8b64ecd4622 /tests
parent87fee2380cba0ca13b8b2892ac2232fb8e04b6ab (diff)
SkMipMap::ComputeLevelSize to return SkISize
This previously returned SkSize, which had scalar components. That doesn't make sense for a mipmap size. R=bsalomon@google.com BUG=578304 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2036313002 Review-Url: https://codereview.chromium.org/2036313002
Diffstat (limited to 'tests')
-rw-r--r--tests/MipMapTest.cpp43
1 files changed, 22 insertions, 21 deletions
diff --git a/tests/MipMapTest.cpp b/tests/MipMapTest.cpp
index 5070d04df6..e8d02b878d 100644
--- a/tests/MipMapTest.cpp
+++ b/tests/MipMapTest.cpp
@@ -155,45 +155,46 @@ struct LevelSizeScenario {
int fBaseWidth;
int fBaseHeight;
int fLevel;
- SkSize fExpectedMipMapLevelSize;
+ SkISize fExpectedMipMapLevelSize;
};
DEF_TEST(MipMap_ComputeLevelSize, reporter) {
const LevelSizeScenario tests[] = {
// Test mipmaps with negative sizes
- {-100, 100, 1, SkSize::Make(0, 0)},
- {100, -100, 1, SkSize::Make(0, 0)},
- {-100, -100, 1, SkSize::Make(0, 0)},
+ {-100, 100, 1, SkISize::Make(0, 0)},
+ {100, -100, 1, SkISize::Make(0, 0)},
+ {-100, -100, 1, SkISize::Make(0, 0)},
// Test mipmaps with 0, 1, 2 as dimensions
// (SkMipMap::Build requires a min size of 1)
//
// 0
- {0, 100, 1, SkSize::Make(0, 0)},
- {100, 0, 1, SkSize::Make(0, 0)},
- {0, 0, 1, SkSize::Make(0, 0)},
+ {0, 100, 1, SkISize::Make(0, 0)},
+ {100, 0, 1, SkISize::Make(0, 0)},
+ {0, 0, 1, SkISize::Make(0, 0)},
// 1
- {1, 100, 1, SkSize::Make(1, 50)},
- {100, 1, 1, SkSize::Make(50, 1)},
- {1, 1, 1, SkSize::Make(0, 0)},
+ {1, 100, 1, SkISize::Make(1, 50)},
+ {100, 1, 1, SkISize::Make(50, 1)},
+ {1, 1, 1, SkISize::Make(0, 0)},
// 2
- {2, 100, 1, SkSize::Make(1, 50)},
- {100, 2, 2, SkSize::Make(25, 1)},
- {2, 2, 1, SkSize::Make(1, 1)},
+ {2, 100, 1, SkISize::Make(1, 50)},
+ {100, 2, 2, SkISize::Make(25, 1)},
+ {2, 2, 1, SkISize::Make(1, 1)},
// Test a handful of cases
- {63, 63, 3, SkSize::Make(7, 7)},
- {64, 64, 3, SkSize::Make(8, 8)},
- {127, 127, 3, SkSize::Make(15, 15)},
- {64, 129, 4, SkSize::Make(4, 8)},
- {255, 32, 7, SkSize::Make(1, 1)},
- {500, 1000, 2, SkSize::Make(125, 250)},
+ {63, 63, 3, SkISize::Make(7, 7)},
+ {64, 64, 3, SkISize::Make(8, 8)},
+ {127, 127, 3, SkISize::Make(15, 15)},
+ {64, 129, 4, SkISize::Make(4, 8)},
+ {255, 32, 7, SkISize::Make(1, 1)},
+ {500, 1000, 2, SkISize::Make(125, 250)},
};
for (auto& currentTest : tests) {
- SkSize levelSize = SkMipMap::ComputeLevelSize(currentTest.fBaseWidth,
- currentTest.fBaseHeight, currentTest.fLevel);
+ SkISize levelSize = SkMipMap::ComputeLevelSize(currentTest.fBaseWidth,
+ currentTest.fBaseHeight,
+ currentTest.fLevel);
REPORTER_ASSERT(reporter, currentTest.fExpectedMipMapLevelSize == levelSize);
}
}