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

Unified Diff: ui/gfx/gl/gl_surface_win.cc

Issue 7021014: GLContext no longer holds a pointer to a GLSurface. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 7 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: ui/gfx/gl/gl_surface_win.cc
===================================================================
--- ui/gfx/gl/gl_surface_win.cc (revision 86168)
+++ ui/gfx/gl/gl_surface_win.cc (working copy)
@@ -82,7 +82,7 @@
NativeViewGLSurfaceOSMesa::NativeViewGLSurfaceOSMesa(
gfx::PluginWindowHandle window)
- : GLSurfaceOSMesa(gfx::Size()),
+ : GLSurfaceOSMesa(OSMESA_RGBA, gfx::Size()),
window_(window),
device_context_(NULL) {
DCHECK(window);
@@ -105,7 +105,6 @@
if (window_ && device_context_)
ReleaseDC(window_, device_context_);
- window_ = NULL;
device_context_ = NULL;
GLSurfaceOSMesa::Destroy();
@@ -205,7 +204,8 @@
GLSurface* GLSurface::CreateOffscreenGLSurface(const gfx::Size& size) {
switch (GetGLImplementation()) {
case kGLImplementationOSMesaGL: {
- scoped_ptr<GLSurfaceOSMesa> surface(new GLSurfaceOSMesa(size));
+ scoped_ptr<GLSurfaceOSMesa> surface(new GLSurfaceOSMesa(OSMESA_RGBA,
+ size));
if (!surface->Initialize())
return NULL;

Powered by Google App Engine
This is Rietveld 408576698