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

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

Issue 2449993005: Remove GLImage::Destroy(). (Closed)
Patch Set: One more Destroy() call on Mac. Created 4 years, 1 month 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
« no previous file with comments | « ui/gl/gl_image.h ('k') | ui/gl/gl_image_egl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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_EGL_H_ 5 #ifndef UI_GL_GL_IMAGE_EGL_H_
6 #define UI_GL_GL_IMAGE_EGL_H_ 6 #define UI_GL_GL_IMAGE_EGL_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/threading/thread_checker.h" 9 #include "base/threading/thread_checker.h"
10 #include "ui/gl/gl_bindings.h" 10 #include "ui/gl/gl_bindings.h"
11 #include "ui/gl/gl_export.h" 11 #include "ui/gl/gl_export.h"
12 #include "ui/gl/gl_image.h" 12 #include "ui/gl/gl_image.h"
13 13
14 namespace gl { 14 namespace gl {
15 15
16 class GL_EXPORT GLImageEGL : public GLImage { 16 class GL_EXPORT GLImageEGL : public GLImage {
17 public: 17 public:
18 explicit GLImageEGL(const gfx::Size& size); 18 explicit GLImageEGL(const gfx::Size& size);
19 19
20 bool Initialize(EGLenum target, EGLClientBuffer buffer, const EGLint* attrs); 20 bool Initialize(EGLenum target, EGLClientBuffer buffer, const EGLint* attrs);
21 21
22 // Overridden from GLImage: 22 // Overridden from GLImage:
23 void Destroy(bool have_context) override;
24 gfx::Size GetSize() override; 23 gfx::Size GetSize() override;
25 unsigned GetInternalFormat() override; 24 unsigned GetInternalFormat() override;
26 bool BindTexImage(unsigned target) override; 25 bool BindTexImage(unsigned target) override;
27 void ReleaseTexImage(unsigned target) override {} 26 void ReleaseTexImage(unsigned target) override {}
28 bool CopyTexImage(unsigned target) override; 27 bool CopyTexImage(unsigned target) override;
29 bool CopyTexSubImage(unsigned target, 28 bool CopyTexSubImage(unsigned target,
30 const gfx::Point& offset, 29 const gfx::Point& offset,
31 const gfx::Rect& rect) override; 30 const gfx::Rect& rect) override;
32 bool ScheduleOverlayPlane(gfx::AcceleratedWidget widget, 31 bool ScheduleOverlayPlane(gfx::AcceleratedWidget widget,
33 int z_order, 32 int z_order,
34 gfx::OverlayTransform transform, 33 gfx::OverlayTransform transform,
35 const gfx::Rect& bounds_rect, 34 const gfx::Rect& bounds_rect,
36 const gfx::RectF& crop_rect) override; 35 const gfx::RectF& crop_rect) override;
37 void Flush() override {} 36 void Flush() override {}
38 37
39 protected: 38 protected:
40 ~GLImageEGL() override; 39 ~GLImageEGL() override;
41 40
42 EGLImageKHR egl_image_; 41 EGLImageKHR egl_image_;
43 const gfx::Size size_; 42 const gfx::Size size_;
44 base::ThreadChecker thread_checker_; 43 base::ThreadChecker thread_checker_;
45 44
46 private: 45 private:
47 DISALLOW_COPY_AND_ASSIGN(GLImageEGL); 46 DISALLOW_COPY_AND_ASSIGN(GLImageEGL);
48 }; 47 };
49 48
50 } // namespace gl 49 } // namespace gl
51 50
52 #endif // UI_GL_GL_IMAGE_EGL_H_ 51 #endif // UI_GL_GL_IMAGE_EGL_H_
OLDNEW
« no previous file with comments | « ui/gl/gl_image.h ('k') | ui/gl/gl_image_egl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698