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

Issue 2856022: Added support to GLEW for dynamically looking up the core WGL entry... (Closed)

Created:
10 years, 5 months ago by Ken Russell (switch to Gerrit)
Modified:
9 years, 7 months ago
Reviewers:
apatrick_chromium
CC:
chromium-reviews
Visibility:
Public.

Description

Added support to GLEW for dynamically looking up the core WGL entry points to avoid the need to link against opengl32.lib. Added _ATL_NO_OPENGL #define in build/common.gypi to prevent link directive from forcibly pulling in opengl32.lib for any Chrome code using ATL. Once upstream code in WebKit is changed to use the new wglew entry points and remove opengl32.lib from the link line, chrome.dll will no longer depend on opengl32.dll. BUG=45586 TEST=ran WebGL with --in-process-webgl Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=51277

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+105 lines, -1 line) Patch
M build/common.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M third_party/glew/README.chromium View 1 chunk +3 lines, -0 lines 0 comments Download
M third_party/glew/include/GL/wglew.h View 3 chunks +48 lines, -0 lines 0 comments Download
M third_party/glew/src/glew.c View 5 chunks +53 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
Ken Russell (switch to Gerrit)
Please review. Thanks.
10 years, 5 months ago (2010-06-30 02:04:38 UTC) #1
apatrick_chromium
10 years, 5 months ago (2010-06-30 18:18:24 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698