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

Side by Side Diff: ui/gl/gl_image_surface_texture.h

Issue 301793003: During image destroy, delete textures only if we have a GL context. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: change destroy flow for all GLImage types. Created 6 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef UI_GL_GL_IMAGE_SURFACE_TEXTURE_H_ 5 #ifndef UI_GL_GL_IMAGE_SURFACE_TEXTURE_H_
6 #define UI_GL_GL_IMAGE_SURFACE_TEXTURE_H_ 6 #define UI_GL_GL_IMAGE_SURFACE_TEXTURE_H_
7 7
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "ui/gl/gl_bindings.h" 9 #include "ui/gl/gl_bindings.h"
10 #include "ui/gl/gl_image.h" 10 #include "ui/gl/gl_image.h"
11 11
12 namespace gfx { 12 namespace gfx {
13 13
14 class SurfaceTexture; 14 class SurfaceTexture;
15 15
16 class GL_EXPORT GLImageSurfaceTexture : public GLImage { 16 class GL_EXPORT GLImageSurfaceTexture : public GLImage {
17 public: 17 public:
18 explicit GLImageSurfaceTexture(gfx::Size size); 18 explicit GLImageSurfaceTexture(gfx::Size size);
19 19
20 bool Initialize(gfx::GpuMemoryBufferHandle buffer); 20 bool Initialize(gfx::GpuMemoryBufferHandle buffer);
21 21
22 // Overridden from GLImage: 22 // Overridden from GLImage:
23 virtual void Destroy() OVERRIDE; 23 virtual void Destroy(bool have_context) OVERRIDE;
24 virtual gfx::Size GetSize() OVERRIDE; 24 virtual gfx::Size GetSize() OVERRIDE;
25 virtual bool BindTexImage(unsigned target) OVERRIDE; 25 virtual bool BindTexImage(unsigned target) OVERRIDE;
26 virtual void ReleaseTexImage(unsigned target) OVERRIDE {} 26 virtual void ReleaseTexImage(unsigned target) OVERRIDE {}
27 virtual void WillUseTexImage() OVERRIDE {} 27 virtual void WillUseTexImage() OVERRIDE {}
28 virtual void DidUseTexImage() OVERRIDE {} 28 virtual void DidUseTexImage() OVERRIDE {}
29 virtual void WillModifyTexImage() OVERRIDE {} 29 virtual void WillModifyTexImage() OVERRIDE {}
30 virtual void DidModifyTexImage() OVERRIDE {} 30 virtual void DidModifyTexImage() OVERRIDE {}
31 31
32 protected: 32 protected:
33 virtual ~GLImageSurfaceTexture(); 33 virtual ~GLImageSurfaceTexture();
34 34
35 private: 35 private:
36 scoped_refptr<SurfaceTexture> surface_texture_; 36 scoped_refptr<SurfaceTexture> surface_texture_;
37 gfx::Size size_; 37 gfx::Size size_;
38 GLint texture_id_; 38 GLint texture_id_;
39 39
40 DISALLOW_COPY_AND_ASSIGN(GLImageSurfaceTexture); 40 DISALLOW_COPY_AND_ASSIGN(GLImageSurfaceTexture);
41 }; 41 };
42 42
43 } // namespace gfx 43 } // namespace gfx
44 44
45 #endif // UI_GL_GL_IMAGE_SURFACE_TEXTURE_H_ 45 #endif // UI_GL_GL_IMAGE_SURFACE_TEXTURE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698