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

Issue 1492013002: Reland of Upgrade PixelStorei to ES3/WebGL2. (Closed)

Created:
5 years ago by Zhenyao Mo
Modified:
5 years ago
CC:
chromium-reviews, blink-reviews, piman+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reland of Upgrade PixelStorei to ES3/WebGL2. (patchset #1 id:1 of https://codereview.chromium.org/1494553002/ ) Reason for revert: revert the revert (i.e., reland) the WebGL 2 CTS failure is due to the bug in the test. I am going to suppress it for now and fix it in khronos and roll in. Original issue's description: > Revert of Upgrade PixelStorei to ES3/WebGL2. (patchset #7 id:160001 of https://codereview.chromium.org/1474513003/ ) > > Reason for revert: > Failing the WebGL 2 CTS: > > WebglConformance.conformance2_state_gl_get_calls: > [3560:1299:1202/051124:INFO:CONSOLE(11)] "context.getParameter(context.UNPACK_SKIP_IMAGES) should be false (of type boolean). Was 0 (of type number).", source: (11) > [3560:1299:1202/051124:INFO:CONSOLE(11)] "FAIL context.getParameter(context.UNPACK_SKIP_IMAGES) should be false (of type boolean). Was 0 (of type number).", source: (11) > [3560:1299:1202/051124:INFO:CONSOLE(11)] "context.getParameter(context.UNPACK_SKIP_PIXELS) should be false (of type boolean). Was 0 (of type number).", source: (11) > [3560:1299:1202/051124:INFO:CONSOLE(11)] "FAIL context.getParameter(context.UNPACK_SKIP_PIXELS) should be false (of type boolean). Was 0 (of type number).", source: (11) > [3560:1299:1202/051124:INFO:CONSOLE(11)] "context.getParameter(context.UNPACK_SKIP_ROWS) should be false (of type boolean). Was 0 (of type number).", source: (11) > [3560:1299:1202/051124:INFO:CONSOLE(11)] "FAIL context.getParameter(context.UNPACK_SKIP_ROWS) should be false (of type boolean). Was 0 (of type number).", source: (11) > > http://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%2010.10%20Release%20%28Intel%29/builds/7016 > http://build.chromium.org/p/chromium.gpu.fyi/builders/Win8%20Debug%20%28NVIDIA%29/builds/10350 > http://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28NVIDIA%29/builds/33475 > > Original issue's description: > > Upgrade PixelStorei to ES3/WebGL2. > > > > BUG=295792, 429053 > > TEST=gpu_unittests, webgl2_conformance > > R=kbr@chromium.org,bajones@chromium.org,piman@chromium.org > > > > Committed: https://crrev.com/c6c114178c562feeddfc4d41a33b9999698a4144 > > Cr-Commit-Position: refs/heads/master@{#362585} > > TBR=bajones@chromium.org,kbr@chromium.org,piman@chromium.org,zmo@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=295792, 429053 > > Committed: https://crrev.com/72ea7571631a1922f7c15e7ef5447b8098c1ab24 > Cr-Commit-Position: refs/heads/master@{#362696} TBR=bajones@chromium.org,kbr@chromium.org,piman@chromium.org,jmadill@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=295792, 429053 Committed: https://crrev.com/cdfe65d79d2f9c02553392e0f38bb5a2279d31b8 Cr-Commit-Position: refs/heads/master@{#362741}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+711 lines, -98 lines) Patch
M gpu/command_buffer/build_gles2_cmd_buffer.py View 9 chunks +77 lines, -6 lines 0 comments Download
M gpu/command_buffer/client/gles2_implementation.h View 1 chunk +9 lines, -0 lines 0 comments Download
M gpu/command_buffer/client/gles2_implementation.cc View 6 chunks +61 lines, -20 lines 0 comments Download
M gpu/command_buffer/service/context_state.h View 1 chunk +7 lines, -0 lines 0 comments Download
M gpu/command_buffer/service/context_state.cc View 4 chunks +38 lines, -0 lines 0 comments Download
M gpu/command_buffer/service/context_state_autogen.h View 1 chunk +8 lines, -0 lines 0 comments Download
M gpu/command_buffer/service/context_state_impl_autogen.h View 5 chunks +168 lines, -0 lines 0 comments Download
M gpu/command_buffer/service/gles2_cmd_decoder.cc View 7 chunks +100 lines, -23 lines 0 comments Download
M gpu/command_buffer/service/gles2_cmd_decoder_unittest_0_autogen.h View 2 chunks +41 lines, -1 line 0 comments Download
M gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.h View 1 chunk +1 line, -1 line 0 comments Download
M gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.cc View 1 chunk +1 line, -1 line 0 comments Download
M gpu/command_buffer/service/gles2_cmd_decoder_unittest_context_state.cc View 2 chunks +64 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/webgl/WebGL2RenderingContextBase.h View 3 chunks +13 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/webgl/WebGL2RenderingContextBase.cpp View 7 chunks +93 lines, -25 lines 0 comments Download
M third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.h View 2 chunks +6 lines, -1 line 0 comments Download
M third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp View 7 chunks +24 lines, -20 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Zhenyao Mo
Created Reland of Upgrade PixelStorei to ES3/WebGL2.
5 years ago (2015-12-02 17:32:51 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1492013002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1492013002/1
5 years ago (2015-12-02 17:34:34 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years ago (2015-12-02 17:36:03 UTC) #4
commit-bot: I haz the power
5 years ago (2015-12-02 17:37:02 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/cdfe65d79d2f9c02553392e0f38bb5a2279d31b8
Cr-Commit-Position: refs/heads/master@{#362741}

Powered by Google App Engine
This is Rietveld 408576698