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

Issue 6350016: Refactor WebGraphicsContext3D to use WGC3D types which match corresponding GL... (Closed)

Created:
9 years, 11 months ago by Zhenyao Mo
Modified:
9 years, 7 months ago
CC:
chromium-reviews, darin-cc_chromium.org, apatrick_chromium, brettw-cc_chromium.org
Visibility:
Public.

Description

Refactor WebGraphicsContext3D to use WGC3D types which match corresponding GL types. BUG=none TEST=compiled fine Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72807

Patch Set 1 #

Patch Set 2 : This is the ugly but safe and correct way #

Total comments: 3

Patch Set 3 : For the records #

Patch Set 4 : For the records after another merging #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3477 lines, -21 lines) Patch
M chrome/renderer/webgraphicscontext3d_command_buffer_impl.h View 1 2 3 4 chunks +429 lines, -0 lines 0 comments Download
M chrome/renderer/webgraphicscontext3d_command_buffer_impl.cc View 1 2 3 4 chunks +1020 lines, -0 lines 0 comments Download
M webkit/gpu/webgraphicscontext3d_in_process_impl.h View 1 2 3 chunks +455 lines, -0 lines 0 comments Download
M webkit/gpu/webgraphicscontext3d_in_process_impl.cc View 1 2 4 chunks +1573 lines, -21 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Zhenyao Mo
9 years, 11 months ago (2011-01-26 02:21:42 UTC) #1
Zhenyao Mo
Forget to mention, this CL build and run WebGL fine both with and without USE_WGC3D_TYPES ...
9 years, 11 months ago (2011-01-26 18:26:44 UTC) #2
Ken Russell (switch to Gerrit)
9 years, 11 months ago (2011-01-26 23:58:02 UTC) #3
LGTM aside from these comments. Also please sync to TOT and incorporate the new
GL_CHROMIUM_framebuffer_blit entry points. Thanks.

http://codereview.chromium.org/6350016/diff/6001/chrome/renderer/webgraphicsc...
File chrome/renderer/webgraphicscontext3d_command_buffer_impl.h (right):

http://codereview.chromium.org/6350016/diff/6001/chrome/renderer/webgraphicsc...
chrome/renderer/webgraphicscontext3d_command_buffer_impl.h:43: using
WebKit::WGC3Dsizeiptr;
I think this block should go inside the #if defined(USE_WGC3D_TYPES) block for
safety. Not sure what happens if you have "using" directives for unknown types.

http://codereview.chromium.org/6350016/diff/6001/webkit/gpu/webgraphicscontex...
File webkit/gpu/webgraphicscontext3d_in_process_impl.cc (right):

http://codereview.chromium.org/6350016/diff/6001/webkit/gpu/webgraphicscontex...
webkit/gpu/webgraphicscontext3d_in_process_impl.cc:2867: break;
Looks like you need to sync to TOT.

http://codereview.chromium.org/6350016/diff/6001/webkit/gpu/webgraphicscontex...
File webkit/gpu/webgraphicscontext3d_in_process_impl.h (right):

http://codereview.chromium.org/6350016/diff/6001/webkit/gpu/webgraphicscontex...
webkit/gpu/webgraphicscontext3d_in_process_impl.h:34: using
WebKit::WGC3Dsizeiptr;
The block above should I think be guarded by #if defined(USE_WGC3D_TYPES) for
safety.

Powered by Google App Engine
This is Rietveld 408576698