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

Side by Side Diff: gpu/command_buffer/service/feature_info.cc

Issue 301243005: gpu: Add disable_arb_sync workaround and blacklist some MacOSX drivers. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | gpu/config/gpu_driver_bug_list_json.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 "gpu/command_buffer/service/feature_info.h" 5 #include "gpu/command_buffer/service/feature_info.h"
6 6
7 #include <set> 7 #include <set>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 784 matching lines...) Expand 10 before | Expand all | Expand 10 after
795 feature_flags_.ext_shader_texture_lod = true; 795 feature_flags_.ext_shader_texture_lod = true;
796 } 796 }
797 797
798 bool egl_khr_fence_sync = false; 798 bool egl_khr_fence_sync = false;
799 #if !defined(OS_MACOSX) 799 #if !defined(OS_MACOSX)
800 if (workarounds_.disable_egl_khr_fence_sync) { 800 if (workarounds_.disable_egl_khr_fence_sync) {
801 gfx::g_driver_egl.ext.b_EGL_KHR_fence_sync = false; 801 gfx::g_driver_egl.ext.b_EGL_KHR_fence_sync = false;
802 } 802 }
803 egl_khr_fence_sync = gfx::g_driver_egl.ext.b_EGL_KHR_fence_sync; 803 egl_khr_fence_sync = gfx::g_driver_egl.ext.b_EGL_KHR_fence_sync;
804 #endif 804 #endif
805 bool arb_sync = false;
806 if (workarounds_.disable_arb_sync) {
807 gfx::g_driver_gl.ext.b_GL_ARB_sync = false;
808 }
809 arb_sync = gfx::g_driver_gl.ext.b_GL_ARB_sync;
no sievers 2014/05/30 18:05:20 ultra-nit: I guess you could remove line 805 and d
reveman 2014/05/30 19:16:17 Cleaned this up a bit in latest patch by not using
805 bool ui_gl_fence_works = is_es3 || extensions.Contains("GL_NV_fence") || 810 bool ui_gl_fence_works = is_es3 || extensions.Contains("GL_NV_fence") ||
806 extensions.Contains("GL_ARB_sync") || 811 arb_sync ||
807 egl_khr_fence_sync; 812 egl_khr_fence_sync;
808 UMA_HISTOGRAM_BOOLEAN("GPU.FenceSupport", ui_gl_fence_works); 813 UMA_HISTOGRAM_BOOLEAN("GPU.FenceSupport", ui_gl_fence_works);
809 814
810 feature_flags_.map_buffer_range = 815 feature_flags_.map_buffer_range =
811 is_es3 || extensions.Contains("GL_ARB_map_buffer_range"); 816 is_es3 || extensions.Contains("GL_ARB_map_buffer_range");
812 817
813 // Really it's part of core OpenGL 2.1 and up, but let's assume the 818 // Really it's part of core OpenGL 2.1 and up, but let's assume the
814 // extension is still advertised. 819 // extension is still advertised.
815 bool has_pixel_buffers = 820 bool has_pixel_buffers =
816 is_es3 || extensions.Contains("GL_ARB_pixel_buffer_object"); 821 is_es3 || extensions.Contains("GL_ARB_pixel_buffer_object");
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
851 if (pos == std::string::npos) { 856 if (pos == std::string::npos) {
852 extensions_ += (extensions_.empty() ? "" : " ") + str; 857 extensions_ += (extensions_.empty() ? "" : " ") + str;
853 } 858 }
854 } 859 }
855 860
856 FeatureInfo::~FeatureInfo() { 861 FeatureInfo::~FeatureInfo() {
857 } 862 }
858 863
859 } // namespace gles2 864 } // namespace gles2
860 } // namespace gpu 865 } // namespace gpu
OLDNEW
« no previous file with comments | « no previous file | gpu/config/gpu_driver_bug_list_json.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698