<http://webkit.org/b/55534> Clean up macros in Extensions3DOpenGL.cpp
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Mar 2011 14:47:38 +0000 (14:47 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Mar 2011 14:47:38 +0000 (14:47 +0000)
Reviewed by Darin Adler.

Change "#if GL_APPLE_vertex_array_object" macros to check that
GL_APPLE_vertex_array_object is both defined and non-zero.

* platform/graphics/opengl/Extensions3DOpenGL.cpp:
(WebCore::Extensions3DOpenGL::createVertexArrayOES):
(WebCore::Extensions3DOpenGL::deleteVertexArrayOES): Remove
empty #else clause.
(WebCore::Extensions3DOpenGL::isVertexArrayOES):
(WebCore::Extensions3DOpenGL::bindVertexArrayOES): Add early
return check.  Remove #else clause that would never have
compiled (since array is not a WTF::String).

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/opengl/Extensions3DOpenGL.cpp

index b8233643b7cc574778f12751280e466522036aae..a49655ada693e6ffd55ee3aae50f92ce1ccc6968 100644 (file)
@@ -1,3 +1,21 @@
+2011-03-02  David Kilzer  <ddkilzer@apple.com>
+
+        <http://webkit.org/b/55534> Clean up macros in Extensions3DOpenGL.cpp
+
+        Reviewed by Darin Adler.
+
+        Change "#if GL_APPLE_vertex_array_object" macros to check that
+        GL_APPLE_vertex_array_object is both defined and non-zero.
+
+        * platform/graphics/opengl/Extensions3DOpenGL.cpp:
+        (WebCore::Extensions3DOpenGL::createVertexArrayOES):
+        (WebCore::Extensions3DOpenGL::deleteVertexArrayOES): Remove
+        empty #else clause.
+        (WebCore::Extensions3DOpenGL::isVertexArrayOES):
+        (WebCore::Extensions3DOpenGL::bindVertexArrayOES): Add early
+        return check.  Remove #else clause that would never have
+        compiled (since array is not a WTF::String).
+
 2011-03-01  Ryosuke Niwa  <rniwa@webkit.org>
 
         Reviewed by Darin Adler.
index d761eb9a9388b070290d919168d48267cf2aa120..e09534e08ef96718058acaba260d34be725f1c82 100644 (file)
@@ -134,7 +134,7 @@ void Extensions3DOpenGL::renderbufferStorageMultisample(unsigned long target, un
 Platform3DObject Extensions3DOpenGL::createVertexArrayOES()
 {
     m_context->makeContextCurrent();
-#if GL_APPLE_vertex_array_object
+#if defined GL_APPLE_vertex_array_object && GL_APPLE_vertex_array_object
     GLuint array = 0;
     glGenVertexArraysAPPLE(1, &array);
     return array;
@@ -149,9 +149,8 @@ void Extensions3DOpenGL::deleteVertexArrayOES(Platform3DObject array)
         return;
     
     m_context->makeContextCurrent();
-#if GL_APPLE_vertex_array_object
+#if defined GL_APPLE_vertex_array_object && GL_APPLE_vertex_array_object
     glDeleteVertexArraysAPPLE(1, &array);
-#else
 #endif
 }
 
@@ -161,7 +160,7 @@ GC3Dboolean Extensions3DOpenGL::isVertexArrayOES(Platform3DObject array)
         return GL_FALSE;
     
     m_context->makeContextCurrent();
-#if GL_APPLE_vertex_array_object
+#if defined GL_APPLE_vertex_array_object && GL_APPLE_vertex_array_object
     return glIsVertexArrayAPPLE(array);
 #else
     return GL_FALSE;
@@ -170,11 +169,12 @@ GC3Dboolean Extensions3DOpenGL::isVertexArrayOES(Platform3DObject array)
 
 void Extensions3DOpenGL::bindVertexArrayOES(Platform3DObject array)
 {
+    if (!array)
+        return;
+
     m_context->makeContextCurrent();
-#if GL_APPLE_vertex_array_object
+#if defined GL_APPLE_vertex_array_object && GL_APPLE_vertex_array_object
     glBindVertexArrayAPPLE(array);
-#else
-    ASSERT_UNUSED(array, supports(array));
 #endif
 }