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

Unified Diff: gpu/command_buffer/common/gles2_cmd_utils.cc

Issue 2121043002: 16 bpp video stream capture, render and WebGL usage - Realsense R200 & SR300 support. Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Tests: cc, skcanvas_video_renderer, wrtcrecorder... Fake capture supports Y16. Created 4 years, 3 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
Index: gpu/command_buffer/common/gles2_cmd_utils.cc
diff --git a/gpu/command_buffer/common/gles2_cmd_utils.cc b/gpu/command_buffer/common/gles2_cmd_utils.cc
index b68bb0b0ca830bef7b52ce73f69e077b980c0377..dba4b4f574133f8c174b48db7beb1d155506c968 100644
--- a/gpu/command_buffer/common/gles2_cmd_utils.cc
+++ b/gpu/command_buffer/common/gles2_cmd_utils.cc
@@ -1091,7 +1091,7 @@ uint32_t GLES2Util::GetGLReadPixelsImplementationType(uint32_t internal_format,
case GL_RGB32F:
case GL_RGBA32F:
case GL_R11F_G11F_B10F:
- return GL_UNSIGNED_BYTE;
+ return GL_FLOAT;
Ken Russell (switch to Gerrit) 2016/09/30 23:01:29 I think this change is actually wrong. For these f
aleksandar.stojiljkovic 2016/10/03 10:56:52 Done. Wasn't important. GL_IMPLEMENTATION_COLOR_RE
case GL_R16F:
case GL_RG16F:
case GL_RGB16F:

Powered by Google App Engine
This is Rietveld 408576698