Remove unused multisampling check
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 17 Aug 2015 20:19:55 +0000 (20:19 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 17 Aug 2015 20:19:55 +0000 (20:19 +0000)
https://bugs.webkit.org/show_bug.cgi?id=148030

Patch by Alex Christensen <achristensen@webkit.org> on 2015-08-17
Reviewed by Dean Jackson.

No change in behavior.
Extensions3DOpenGLCommon.m_maySupportMultisampling is always true.

* platform/graphics/Extensions3D.h:
* platform/graphics/opengl/Extensions3DOpenGLCommon.cpp:
(WebCore::Extensions3DOpenGLCommon::Extensions3DOpenGLCommon):
* platform/graphics/opengl/Extensions3DOpenGLCommon.h:
(WebCore::Extensions3DOpenGLCommon::isImagination):
(WebCore::Extensions3DOpenGLCommon::vendor):
(WebCore::Extensions3DOpenGLCommon::requiresBuiltInFunctionEmulation):
(WebCore::Extensions3DOpenGLCommon::requiresRestrictedMaximumTextureSize):
(WebCore::Extensions3DOpenGLCommon::maySupportMultisampling): Deleted.
* platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:
(WebCore::GraphicsContext3D::validateDepthStencil):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@188536 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/Extensions3D.h
Source/WebCore/platform/graphics/opengl/Extensions3DOpenGLCommon.cpp
Source/WebCore/platform/graphics/opengl/Extensions3DOpenGLCommon.h
Source/WebCore/platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp

index 9597d1f..09753b5 100644 (file)
@@ -1,3 +1,25 @@
+2015-08-17  Alex Christensen  <achristensen@webkit.org>
+
+        Remove unused multisampling check
+        https://bugs.webkit.org/show_bug.cgi?id=148030
+
+        Reviewed by Dean Jackson.
+
+        No change in behavior.
+        Extensions3DOpenGLCommon.m_maySupportMultisampling is always true.
+
+        * platform/graphics/Extensions3D.h:
+        * platform/graphics/opengl/Extensions3DOpenGLCommon.cpp:
+        (WebCore::Extensions3DOpenGLCommon::Extensions3DOpenGLCommon):
+        * platform/graphics/opengl/Extensions3DOpenGLCommon.h:
+        (WebCore::Extensions3DOpenGLCommon::isImagination):
+        (WebCore::Extensions3DOpenGLCommon::vendor):
+        (WebCore::Extensions3DOpenGLCommon::requiresBuiltInFunctionEmulation):
+        (WebCore::Extensions3DOpenGLCommon::requiresRestrictedMaximumTextureSize):
+        (WebCore::Extensions3DOpenGLCommon::maySupportMultisampling): Deleted.
+        * platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:
+        (WebCore::GraphicsContext3D::validateDepthStencil):
+
 2015-08-17  Andy Estes  <aestes@apple.com>
 
         REGRESSION (r188486): Crash in SubresourceLoader::didReceiveResponse() when TemporaryChange goes out of scope
index f65dd05..87ee55d 100644 (file)
@@ -255,13 +255,6 @@ public:
     virtual bool isImagination() = 0;
     virtual String vendor() = 0;
 
-    // If this method returns false then the system *definitely* does not support multisampling.
-    // It does not necessarily say the system does support it - callers must attempt to construct
-    // multisampled renderbuffers and check framebuffer completeness.
-    // Ports should implement this to return false on configurations where it is known
-    // that multisampling is not available.
-    virtual bool maySupportMultisampling() = 0;
-
     // Some configurations have bugs regarding built-in functions in their OpenGL drivers
     // that must be avoided. Ports should implement these flags on such configurations.
     virtual bool requiresBuiltInFunctionEmulation() = 0;
index cef2e1c..8176c50 100644 (file)
@@ -58,7 +58,6 @@ Extensions3DOpenGLCommon::Extensions3DOpenGLCommon(GraphicsContext3D* context)
     , m_isAMD(false)
     , m_isIntel(false)
     , m_isImagination(false)
-    , m_maySupportMultisampling(true)
     , m_requiresBuiltInFunctionEmulation(false)
     , m_requiresRestrictedMaximumTextureSize(false)
 {
@@ -80,17 +79,8 @@ Extensions3DOpenGLCommon::Extensions3DOpenGLCommon(GraphicsContext3D* context)
     if (m_isAMD || m_isIntel)
         m_requiresBuiltInFunctionEmulation = true;
 
-    // Currently in Mac we only allow multisampling if the vendor is NVIDIA,
-    // or if the vendor is AMD/ATI and the system is 10.7.2 and above.
-
-    bool systemSupportsMultisampling = true;
-
-    if (m_isAMD && !systemSupportsMultisampling)
-        m_maySupportMultisampling = false;
-
     // Intel HD 3000 devices have problems with large textures. <rdar://problem/16649140>
-    if (m_isIntel)
-        m_requiresRestrictedMaximumTextureSize = m_renderer.startsWith("Intel HD Graphics 3000");
+    m_requiresRestrictedMaximumTextureSize = m_renderer.startsWith("Intel HD Graphics 3000");
 #endif
 }
 
index 5047a80..f21d1b4 100644 (file)
@@ -65,7 +65,6 @@ public:
     virtual bool isImagination() { return m_isImagination; }
     virtual String vendor() { return m_vendor; }
 
-    virtual bool maySupportMultisampling() { return m_maySupportMultisampling; }
     virtual bool requiresBuiltInFunctionEmulation() { return m_requiresBuiltInFunctionEmulation; }
     virtual bool requiresRestrictedMaximumTextureSize() { return m_requiresRestrictedMaximumTextureSize; }
 
@@ -87,7 +86,6 @@ protected:
     bool m_isAMD;
     bool m_isIntel;
     bool m_isImagination;
-    bool m_maySupportMultisampling;
     bool m_requiresBuiltInFunctionEmulation;
     bool m_requiresRestrictedMaximumTextureSize;
 
index 7edd411..f342ad9 100644 (file)
@@ -133,7 +133,7 @@ void GraphicsContext3D::validateDepthStencil(const char* packedDepthStencilExten
             m_attrs.stencil = false;
     }
     if (m_attrs.antialias) {
-        if (!extensions->maySupportMultisampling() || !extensions->supports("GL_ANGLE_framebuffer_multisample") || isGLES2Compliant())
+        if (!extensions->supports("GL_ANGLE_framebuffer_multisample") || isGLES2Compliant())
             m_attrs.antialias = false;
         else
             extensions->ensureEnabled("GL_ANGLE_framebuffer_multisample");