| commit | 4be817dd606efdba1614d32608ddd6e16bfa687d | |
| author | magjed <magjed@chromium.org> | |
| Fri, 13 Mar 2015 20:48:39 +0000 (13 13:48 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Fri, 13 Mar 2015 20:49:27 +0000 (13 20:49 +0000) | ||
| tree | a46f86287ea996b3886a2cf6a219f448d4fbcf54 | treesnapshot (tar.gz zip) |
| parent | c4db547701d45f287feb86bda19d2914f0864314 | commitdiff |
Increase YUV video clamping
The current clamping is not enough to avoid bleeding issues. This is
exposed when changing the background color in CreateEdgeBleedPass from
black to gray. The reason it works now is because interpolating U/V
textures from black (YUV = 0, 0, 0) to green (YUV = 149, 43, 21) will
still result in saturated green pixels (RGB = 0, 255, 0). All YUV values
in the range (149, 0-43, 0-21) will end up as RGB = (0, 255, 0).
The current clamping is not enough, because the calculation
'0.5f / quad->tex_size.width()' is using the Y texture size, the highest
resolution plane, but the U/V texture size is only half of that. This CL
changes tex_size to represent the lowest resolution texture instead.
BUG=429640
Review URL: https://codereview.chromium.org/1008493002
Cr-Commit-Position: refs/heads/master@{#320567}
The current clamping is not enough to avoid bleeding issues. This is
exposed when changing the background color in CreateEdgeBleedPass from
black to gray. The reason it works now is because interpolating U/V
textures from black (YUV = 0, 0, 0) to green (YUV = 149, 43, 21) will
still result in saturated green pixels (RGB = 0, 255, 0). All YUV values
in the range (149, 0-43, 0-21) will end up as RGB = (0, 255, 0).
The current clamping is not enough, because the calculation
'0.5f / quad->tex_size.width()' is using the Y texture size, the highest
resolution plane, but the U/V texture size is only half of that. This CL
changes tex_size to represent the lowest resolution texture instead.
BUG=429640
Review URL: https://codereview.chromium.org/1008493002
Cr-Commit-Position: refs/heads/master@{#320567}
