Title: [148680] trunk/Source/WebCore
- Revision
- 148680
- Author
- commit-qu...@webkit.org
- Date
- 2013-04-18 09:14:58 -0700 (Thu, 18 Apr 2013)
Log Message
[BlackBerry] Improper initialization of ANGLEResources (resubmission)
https://bugs.webkit.org/show_bug.cgi?id=114814
Patch by Jonathan Feldstein <jfeldst...@blackberry.com> on 2013-04-18
Reviewed by Yong Li, Rob Buis
Internally reviewed by Maxim Mogilnitsky
MaxDrawBuffers, OES_standard_derivatives, OES_EGL_image_external and ARB_texture_rectangle
are initialized through the ShBuiltInResources function so these fields do not need to be
set again in GraphicsContext3DBlackBerry.cpp. In addition, the extension flags should not
be set to true without getExtension being called (Khronos WebGL specs, section 5.14.14.).
In fact, as a direct result of these extensions being enabled prior to calling
getExtension, a WebGL conformance suite test for GL_OES_standard_derivatives was failing.
* platform/graphics/blackberry/GraphicsContext3DBlackBerry.cpp:
(WebCore::GraphicsContext3D::GraphicsContext3D):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (148679 => 148680)
--- trunk/Source/WebCore/ChangeLog 2013-04-18 16:12:12 UTC (rev 148679)
+++ trunk/Source/WebCore/ChangeLog 2013-04-18 16:14:58 UTC (rev 148680)
@@ -1,3 +1,21 @@
+2013-04-18 Jonathan Feldstein <jfeldst...@blackberry.com>
+
+ [BlackBerry] Improper initialization of ANGLEResources (resubmission)
+ https://bugs.webkit.org/show_bug.cgi?id=114814
+
+ Reviewed by Yong Li, Rob Buis
+ Internally reviewed by Maxim Mogilnitsky
+
+ MaxDrawBuffers, OES_standard_derivatives, OES_EGL_image_external and ARB_texture_rectangle
+ are initialized through the ShBuiltInResources function so these fields do not need to be
+ set again in GraphicsContext3DBlackBerry.cpp. In addition, the extension flags should not
+ be set to true without getExtension being called (Khronos WebGL specs, section 5.14.14.).
+ In fact, as a direct result of these extensions being enabled prior to calling
+ getExtension, a WebGL conformance suite test for GL_OES_standard_derivatives was failing.
+
+ * platform/graphics/blackberry/GraphicsContext3DBlackBerry.cpp:
+ (WebCore::GraphicsContext3D::GraphicsContext3D):
+
2013-04-18 Xuefei Ren <x...@blackberry.com>
remove build warning(unused parameter)
Modified: trunk/Source/WebCore/platform/graphics/blackberry/GraphicsContext3DBlackBerry.cpp (148679 => 148680)
--- trunk/Source/WebCore/platform/graphics/blackberry/GraphicsContext3DBlackBerry.cpp 2013-04-18 16:12:12 UTC (rev 148679)
+++ trunk/Source/WebCore/platform/graphics/blackberry/GraphicsContext3DBlackBerry.cpp 2013-04-18 16:14:58 UTC (rev 148680)
@@ -108,12 +108,6 @@
getIntegerv(GraphicsContext3D::MAX_TEXTURE_IMAGE_UNITS, &ANGLEResources.MaxTextureImageUnits);
getIntegerv(GraphicsContext3D::MAX_FRAGMENT_UNIFORM_VECTORS, &ANGLEResources.MaxFragmentUniformVectors);
- Extensions3D* extensions = getExtensions();
- ANGLEResources.MaxDrawBuffers = 1; // Always set to 1 for OpenGL ES.
- ANGLEResources.OES_standard_derivatives = extensions->supports("GL_OES_standard_derivatives");
- ANGLEResources.OES_EGL_image_external = extensions->supports("GL_EGL_image_external");
- ANGLEResources.ARB_texture_rectangle = extensions->supports("GL_ARB_texture_rectangle");
-
GC3Dint range[2], precision;
getShaderPrecisionFormat(GraphicsContext3D::FRAGMENT_SHADER, GraphicsContext3D::HIGH_FLOAT, range, &precision);
ANGLEResources.FragmentPrecisionHigh = (range[0] || range[1] || precision);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
