diff options
| author | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-22 23:45:01 +0000 |
|---|---|---|
| committer | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-22 23:45:01 +0000 |
| commit | 5c3dfee55042ad313919568d8302ea0e9d644c7e (patch) | |
| tree | 7481da39b2a1808f76c15629f284eb105bde6091 /gpu | |
| parent | fe674603a5c8787a5e39993617b685a2ec7a66aa (diff) | |
| download | chromium_src-5c3dfee55042ad313919568d8302ea0e9d644c7e.zip chromium_src-5c3dfee55042ad313919568d8302ea0e9d644c7e.tar.gz chromium_src-5c3dfee55042ad313919568d8302ea0e9d644c7e.tar.bz2 | |
Fix WebGL getParameter bug in command buffer.
GetHelper was turning all getParameter return values into integers, even
if they should have been floats. By returning false in GetHelper, it
lets the individual DoGet(Float,Boolean,Integer)v functions skip this
behavior and return the real value directly.
BUG=51331
TEST=LayoutTests/fast/canvas/webgl/gl-get-calls.html
R=apatrick,gman
Patch by Adrienne Walker (enne@chromium.org)
Originally reviewed at http://codereview.chromium.org/3023042
Review URL: http://codereview.chromium.org/3402026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60236 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'gpu')
| -rw-r--r-- | gpu/command_buffer/service/gles2_cmd_decoder.cc | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/gpu/command_buffer/service/gles2_cmd_decoder.cc b/gpu/command_buffer/service/gles2_cmd_decoder.cc index 4a38f62..68fa2f3 100644 --- a/gpu/command_buffer/service/gles2_cmd_decoder.cc +++ b/gpu/command_buffer/service/gles2_cmd_decoder.cc @@ -2616,10 +2616,7 @@ bool GLES2DecoderImpl::GetHelper( return true; default: *num_written = util_.GLGetNumValuesReturned(pname); - if (params) { - glGetIntegerv(pname, params); - } - return true; + return false; } } |