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

Side by Side Diff: ui/gl/gl_fence.cc

Issue 2629633003: Refactor GL bindings so there is no global GLApi or DriverGL. (Closed)
Patch Set: rebase Created 3 years, 10 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
« no previous file with comments | « ui/gl/gl_egl_api_implementation.cc ('k') | ui/gl/gl_fence_arb.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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "ui/gl/gl_fence.h" 5 #include "ui/gl/gl_fence.h"
6 6
7 #include "base/compiler_specific.h" 7 #include "base/compiler_specific.h"
8 #include "build/build_config.h" 8 #include "build/build_config.h"
9 #include "ui/gl/gl_bindings.h" 9 #include "ui/gl/gl_bindings.h"
10 #include "ui/gl/gl_context.h" 10 #include "ui/gl/gl_context.h"
11 #include "ui/gl/gl_fence_arb.h" 11 #include "ui/gl/gl_fence_arb.h"
12 #include "ui/gl/gl_fence_egl.h" 12 #include "ui/gl/gl_fence_egl.h"
13 #include "ui/gl/gl_fence_nv.h" 13 #include "ui/gl/gl_fence_nv.h"
14 #include "ui/gl/gl_gl_api_implementation.h"
15 #include "ui/gl/gl_implementation.h" 14 #include "ui/gl/gl_implementation.h"
16 #include "ui/gl/gl_version_info.h" 15 #include "ui/gl/gl_version_info.h"
17 16
18 #if defined(OS_MACOSX) 17 #if defined(OS_MACOSX)
19 #include "ui/gl/gl_fence_apple.h" 18 #include "ui/gl/gl_fence_apple.h"
20 #endif 19 #endif
21 20
22 namespace gl { 21 namespace gl {
23 22
24 GLFence::GLFence() { 23 GLFence::GLFence() {
25 } 24 }
26 25
27 GLFence::~GLFence() { 26 GLFence::~GLFence() {
28 } 27 }
29 28
30 bool GLFence::IsSupported() { 29 bool GLFence::IsSupported() {
31 DCHECK(GetGLVersionInfo()); 30 DCHECK(g_current_gl_version && g_current_gl_driver);
32 return g_driver_gl.ext.b_GL_ARB_sync || GetGLVersionInfo()->is_es3 || 31 return g_current_gl_driver->ext.b_GL_ARB_sync ||
33 GetGLVersionInfo()->is_desktop_core_profile || 32 g_current_gl_version->is_es3 ||
33 g_current_gl_version->is_desktop_core_profile ||
34 #if defined(OS_MACOSX) 34 #if defined(OS_MACOSX)
35 g_driver_gl.ext.b_GL_APPLE_fence || 35 g_current_gl_driver->ext.b_GL_APPLE_fence ||
36 #else 36 #else
37 g_driver_egl.ext.b_EGL_KHR_fence_sync || 37 g_driver_egl.ext.b_EGL_KHR_fence_sync ||
38 #endif 38 #endif
39 g_driver_gl.ext.b_GL_NV_fence; 39 g_current_gl_driver->ext.b_GL_NV_fence;
40 } 40 }
41 41
42 GLFence* GLFence::Create() { 42 GLFence* GLFence::Create() {
43 DCHECK(GLContext::GetCurrent()) 43 DCHECK(GLContext::GetCurrent())
44 << "Trying to create fence with no context"; 44 << "Trying to create fence with no context";
45 45
46 std::unique_ptr<GLFence> fence; 46 std::unique_ptr<GLFence> fence;
47 #if !defined(OS_MACOSX) 47 #if !defined(OS_MACOSX)
48 if (g_driver_egl.ext.b_EGL_KHR_fence_sync && 48 if (g_driver_egl.ext.b_EGL_KHR_fence_sync &&
49 g_driver_egl.ext.b_EGL_KHR_wait_sync) { 49 g_driver_egl.ext.b_EGL_KHR_wait_sync) {
50 // Prefer GLFenceEGL which doesn't require GL context switching. 50 // Prefer GLFenceEGL which doesn't require GL context switching.
51 fence.reset(new GLFenceEGL); 51 fence.reset(new GLFenceEGL);
52 } else 52 } else
53 #endif 53 #endif
54 if (g_driver_gl.ext.b_GL_ARB_sync || GetGLVersionInfo()->is_es3 || 54 if (g_current_gl_driver->ext.b_GL_ARB_sync ||
55 GetGLVersionInfo()->is_desktop_core_profile) { 55 g_current_gl_version->is_es3 ||
56 g_current_gl_version->is_desktop_core_profile) {
56 // Prefer ARB_sync which supports server-side wait. 57 // Prefer ARB_sync which supports server-side wait.
57 fence.reset(new GLFenceARB); 58 fence.reset(new GLFenceARB);
58 #if defined(OS_MACOSX) 59 #if defined(OS_MACOSX)
59 } else if (g_driver_gl.ext.b_GL_APPLE_fence) { 60 } else if (g_current_gl_driver->ext.b_GL_APPLE_fence) {
60 fence.reset(new GLFenceAPPLE); 61 fence.reset(new GLFenceAPPLE);
61 #else 62 #else
62 } else if (g_driver_egl.ext.b_EGL_KHR_fence_sync) { 63 } else if (g_driver_egl.ext.b_EGL_KHR_fence_sync) {
63 fence.reset(new GLFenceEGL); 64 fence.reset(new GLFenceEGL);
64 #endif 65 #endif
65 } else if (g_driver_gl.ext.b_GL_NV_fence) { 66 } else if (g_current_gl_driver->ext.b_GL_NV_fence) {
66 fence.reset(new GLFenceNV); 67 fence.reset(new GLFenceNV);
67 } 68 }
68 69
69 DCHECK_EQ(!!fence.get(), GLFence::IsSupported()); 70 DCHECK_EQ(!!fence.get(), GLFence::IsSupported());
70 return fence.release(); 71 return fence.release();
71 } 72 }
72 73
73 bool GLFence::ResetSupported() { 74 bool GLFence::ResetSupported() {
74 // Resetting a fence to its original state isn't supported by default. 75 // Resetting a fence to its original state isn't supported by default.
75 return false; 76 return false;
76 } 77 }
77 78
78 void GLFence::ResetState() { 79 void GLFence::ResetState() {
79 NOTIMPLEMENTED(); 80 NOTIMPLEMENTED();
80 } 81 }
81 82
82 void GLFence::Invalidate() { 83 void GLFence::Invalidate() {
83 NOTIMPLEMENTED(); 84 NOTIMPLEMENTED();
84 } 85 }
85 86
86 } // namespace gl 87 } // namespace gl
OLDNEW
« no previous file with comments | « ui/gl/gl_egl_api_implementation.cc ('k') | ui/gl/gl_fence_arb.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698