Changeset 112075 in webkit

Timestamp:

Mar 26, 2012 4:17:40 AM (12 years ago)

Author:

zeno.albisser@nokia.com

Message:

[Qt]WK2] TextureMapperShaderManager should query the program map before creating new shader program instances.
​https://bugs.webkit.org/show_bug.cgi?id=82178

If TextureMapperShaderManager already has a reference to an instance of the
requested shader program type, this instance should be reused instead of
creating a new one.

Reviewed by Noam Rosenthal.

• platform/graphics/texmap/TextureMapperShaderManager.h:

(WebCore::TextureMapperShaderManager::getShaderProgram):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/texmap/TextureMapperShaderManager.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-03-26  Zeno Albisser  <zeno@webkit.org>
+
+        [Qt]WK2] TextureMapperShaderManager should query the program map before creating new shader program instances.
+        https://bugs.webkit.org/show_bug.cgi?id=82178
+
+        If TextureMapperShaderManager already has a reference to an instance of the
+        requested shader program type, this instance should be reused instead of
+        creating a new one.
+
+        Reviewed by Noam Rosenthal.
+
+        * platform/graphics/texmap/TextureMapperShaderManager.h:
+        (WebCore::TextureMapperShaderManager::getShaderProgram):
+
 2012-03-26  Yury Semikhatsky  <yurys@chromium.org>
 

trunk/Source/WebCore/platform/graphics/texmap/TextureMapperShaderManager.h

@@ -108 +108 @@
             return program;
 
+        TextureMapperShaderProgramMap::iterator it = m_textureMapperShaderProgramMap.find(shaderType);
+        if (it != m_textureMapperShaderProgramMap.end())
+            return it->second;
+
         switch (shaderType) {
         case Simple: