Changeset 199045 in webkit

Timestamp:

Apr 4, 2016, 11:11:22 PM (11 years ago)

Author:

zandobersek@gmail.com

Message:

[TexMap] Improve viewport array access in TextureMapperGL::bindDefaultSurface()
​https://bugs.webkit.org/show_bug.cgi?id=156159

Reviewed by Antonio Gomes.

• platform/graphics/texmap/TextureMapperGL.cpp:

(WebCore::TextureMapperGL::bindDefaultSurface): Create a reference to the
viewport array in the TextureMapperGLData object. Inline the IntSize constructor
for the object that's passed to createProjectionMatrix(), and use the reference
to access all four elements of the array as necessary.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/texmap/TextureMapperGL.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-04-04  Zan Dobersek  <zdobersek@igalia.com>
+
+        [TexMap] Improve viewport array access in TextureMapperGL::bindDefaultSurface()
+        https://bugs.webkit.org/show_bug.cgi?id=156159
+
+        Reviewed by Antonio Gomes.
+
+        * platform/graphics/texmap/TextureMapperGL.cpp:
+        (WebCore::TextureMapperGL::bindDefaultSurface): Create a reference to the
+        viewport array in the TextureMapperGLData object. Inline the IntSize constructor
+        for the object that's passed to createProjectionMatrix(), and use the reference
+        to access all four elements of the array as necessary.
+
 2016-04-04  Zan Dobersek  <zdobersek@igalia.com>
 

trunk/Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp

@@ -619 +619 @@
 {
     m_context3D->bindFramebuffer(GraphicsContext3D::FRAMEBUFFER, data().targetFrameBuffer);
-    IntSize viewportSize(data().viewport[2], data().viewport[3]);
-    data().projectionMatrix = createProjectionMatrix(viewportSize, data().PaintFlags & PaintingMirrored);
-    m_context3D->viewport(data().viewport[0], data().viewport[1], viewportSize.width(), viewportSize.height());
+    auto& viewport = data().viewport;
+    data().projectionMatrix = createProjectionMatrix(IntSize(viewport[2], viewport[3]), data().PaintFlags & PaintingMirrored);
+    m_context3D->viewport(viewport[0], viewport[1], viewport[2], viewport[3]);
     m_clipStack.apply(*m_context3D);
     data().currentSurface = nullptr;