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

Unified Diff: cc/resources/video_resource_updater_unittest.cc

Issue 2763503002: Move HalfFloatMaker to media (Closed)
Patch Set: Added math.h in half_float_maker_unittest.cc for POW undefined error Created 3 years, 9 months 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
« no previous file with comments | « cc/resources/video_resource_updater.cc ('k') | media/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/video_resource_updater_unittest.cc
diff --git a/cc/resources/video_resource_updater_unittest.cc b/cc/resources/video_resource_updater_unittest.cc
index 4b54fbe759f3cfb2c014f880d73c6b21bc93418d..3c7874e498ecb6e984d5045f95dd2087a0f2904a 100644
--- a/cc/resources/video_resource_updater_unittest.cc
+++ b/cc/resources/video_resource_updater_unittest.cc
@@ -519,62 +519,5 @@ TEST_F(VideoResourceUpdaterTest, CreateForHardwarePlanes_StreamTexture) {
EXPECT_FALSE(context3d_->WasImmutableTextureCreated());
}
-namespace {
-
-// Convert an IEEE 754 half-float to a double value
-// that we can do math on.
-double FromHalfFloat(uint16_t half_float) {
- if (!half_float)
- return 0.0;
- int sign = (half_float & 0x8000) ? -1 : 1;
- int exponent = (half_float >> 10) & 0x1F;
- int fraction = half_float & 0x3FF;
- if (exponent == 0) {
- return pow(2.0, -24.0) * fraction;
- } else if (exponent == 0x1F) {
- return sign * 1000000000000.0;
- } else {
- return pow(2.0, exponent - 25) * (0x400 + fraction);
- }
-}
-
-} // namespace
-
-TEST_F(VideoResourceUpdaterTest, MakeHalfFloatTest) {
- unsigned short integers[1 << 16];
- unsigned short half_floats[1 << 16];
- for (int bits = 9; bits <= 16; bits++) {
- std::unique_ptr<VideoResourceUpdater::HalfFloatMaker> half_float_maker;
- half_float_maker = VideoResourceUpdater::NewHalfFloatMaker(bits);
- int num_values = 1 << bits;
- for (int i = 0; i < num_values; i++)
- integers[i] = i;
-
- half_float_maker->MakeHalfFloats(integers, num_values, half_floats);
- // Multiplier to converting integers to 0.0..1.0 range.
- double multiplier = 1.0 / (num_values - 1);
-
- for (int i = 0; i < num_values; i++) {
- // This value is in range 0..1
- float value = integers[i] * multiplier;
- // Reverse the effect of offset and multiplier to get the expected
- // output value from the half-float converter.
- float expected_value =
- value / half_float_maker->Multiplier() + half_float_maker->Offset();
- EXPECT_EQ(integers[i], i);
-
- // We expect the result to be within +/- one least-significant bit.
- // Within the range we care about, half-floats values and
- // their representation both sort in the same order, so we
- // can just add one to get the next bigger half-float.
- float expected_precision =
- FromHalfFloat(half_floats[i] + 1) - FromHalfFloat(half_floats[i]);
- EXPECT_NEAR(FromHalfFloat(half_floats[i]), expected_value,
- expected_precision)
- << "i = " << i << " bits = " << bits;
- }
- }
-}
-
} // namespace
} // namespace cc
« no previous file with comments | « cc/resources/video_resource_updater.cc ('k') | media/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698