Title: [106964] trunk/Source/WebCore
Revision
106964
Author
d...@apple.com
Date
2012-02-07 11:18:47 -0800 (Tue, 07 Feb 2012)

Log Message

Apple/Safari: Enable WebGL multisampling on ATI cards
for OS X 10.7.2 and above.
https://bugs.webkit.org/show_bug.cgi?id=77922

Address review comments by Alexey Proskuryakov and Mark Rowe.

* platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:
(WebCore::systemAllowsMultisamplingOnATICards):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (106963 => 106964)


--- trunk/Source/WebCore/ChangeLog	2012-02-07 19:18:19 UTC (rev 106963)
+++ trunk/Source/WebCore/ChangeLog	2012-02-07 19:18:47 UTC (rev 106964)
@@ -21,6 +21,17 @@
         for OS X 10.7.2 and above.
         https://bugs.webkit.org/show_bug.cgi?id=77922
 
+        Address review comments by Alexey Proskuryakov and Mark Rowe.
+
+        * platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:
+        (WebCore::systemAllowsMultisamplingOnATICards):
+
+2012-02-07  Dean Jackson  <d...@apple.com>
+
+        Apple/Safari: Enable WebGL multisampling on ATI cards
+        for OS X 10.7.2 and above.
+        https://bugs.webkit.org/show_bug.cgi?id=77922
+
         Reviewed by Chris Marrin.
 
         Follow Chrome's lead to allow WebGL antialiasing

Modified: trunk/Source/WebCore/platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp (106963 => 106964)


--- trunk/Source/WebCore/platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp	2012-02-07 19:18:19 UTC (rev 106963)
+++ trunk/Source/WebCore/platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp	2012-02-07 19:18:47 UTC (rev 106964)
@@ -63,16 +63,21 @@
 static bool systemAllowsMultisamplingOnATICards()
 {
 #if PLATFORM(MAC)
+#if !defined(BUILDING_ON_SNOW_LEOPARD) && !defined(BUILDING_ON_LION)
+    return true;
+#else
+    ASSERT(isMainThread());
     static SInt32 version;
     if (!version) {
         if (Gestalt(gestaltSystemVersion, &version) != noErr)
             return false;
     }
     // See https://bugs.webkit.org/show_bug.cgi?id=77922 for more details
-    return version > 0x1072;
+    return version >= 0x1072;
+#endif // SNOW_LEOPARD and LION
 #else
     return false;
-#endif
+#endif // PLATFORM(MAC)
 }
 
 void GraphicsContext3D::validateAttributes()
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to