Title: [220521] trunk/Source/WebCore
Revision
220521
Author
zandober...@gmail.com
Date
2017-08-10 00:39:39 -0700 (Thu, 10 Aug 2017)

Log Message

[TexMap] Remove GraphicsContext3D usage from BitmapTexturePool
https://bugs.webkit.org/show_bug.cgi?id=175319

Reviewed by Carlos Garcia Campos.

Drop the GraphicsContext3D reference in BitmapTexturePool now
that the BitmapTextureGL doesn't require it anymore.

No new tests -- no change in behavior.

* platform/graphics/texmap/BitmapTexturePool.cpp:
(WebCore::BitmapTexturePool::BitmapTexturePool):
* platform/graphics/texmap/BitmapTexturePool.h:
* platform/graphics/texmap/TextureMapperGL.cpp:
(WebCore::TextureMapperGL::TextureMapperGL):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (220520 => 220521)


--- trunk/Source/WebCore/ChangeLog	2017-08-10 07:30:04 UTC (rev 220520)
+++ trunk/Source/WebCore/ChangeLog	2017-08-10 07:39:39 UTC (rev 220521)
@@ -1,5 +1,23 @@
 2017-08-10  Zan Dobersek  <zdober...@igalia.com>
 
+        [TexMap] Remove GraphicsContext3D usage from BitmapTexturePool
+        https://bugs.webkit.org/show_bug.cgi?id=175319
+
+        Reviewed by Carlos Garcia Campos.
+
+        Drop the GraphicsContext3D reference in BitmapTexturePool now
+        that the BitmapTextureGL doesn't require it anymore.
+
+        No new tests -- no change in behavior.
+
+        * platform/graphics/texmap/BitmapTexturePool.cpp:
+        (WebCore::BitmapTexturePool::BitmapTexturePool):
+        * platform/graphics/texmap/BitmapTexturePool.h:
+        * platform/graphics/texmap/TextureMapperGL.cpp:
+        (WebCore::TextureMapperGL::TextureMapperGL):
+
+2017-08-10  Zan Dobersek  <zdober...@igalia.com>
+
         [TexMap] Drop GraphicsContext3D usage from BitmapTextureGL
         https://bugs.webkit.org/show_bug.cgi?id=175317
 

Modified: trunk/Source/WebCore/platform/graphics/texmap/BitmapTexturePool.cpp (220520 => 220521)


--- trunk/Source/WebCore/platform/graphics/texmap/BitmapTexturePool.cpp	2017-08-10 07:30:04 UTC (rev 220520)
+++ trunk/Source/WebCore/platform/graphics/texmap/BitmapTexturePool.cpp	2017-08-10 07:39:39 UTC (rev 220521)
@@ -37,9 +37,8 @@
 static const Seconds releaseUnusedTexturesTimerInterval { 500_ms };
 
 #if USE(TEXTURE_MAPPER_GL)
-BitmapTexturePool::BitmapTexturePool(const TextureMapperContextAttributes& contextAttributes, RefPtr<GraphicsContext3D>&& context3D)
+BitmapTexturePool::BitmapTexturePool(const TextureMapperContextAttributes& contextAttributes)
     : m_contextAttributes(contextAttributes)
-    , m_context3D(WTFMove(context3D))
     , m_releaseUnusedTexturesTimer(RunLoop::current(), this, &BitmapTexturePool::releaseUnusedTexturesTimerFired)
 {
 }

Modified: trunk/Source/WebCore/platform/graphics/texmap/BitmapTexturePool.h (220520 => 220521)


--- trunk/Source/WebCore/platform/graphics/texmap/BitmapTexturePool.h	2017-08-10 07:30:04 UTC (rev 220520)
+++ trunk/Source/WebCore/platform/graphics/texmap/BitmapTexturePool.h	2017-08-10 07:39:39 UTC (rev 220521)
@@ -32,13 +32,8 @@
 #include <wtf/CurrentTime.h>
 #include <wtf/RunLoop.h>
 
-#if USE(TEXTURE_MAPPER_GL)
-#include "GraphicsContext3D.h"
-#endif
-
 namespace WebCore {
 
-class GraphicsContext3D;
 class IntSize;
 
 class BitmapTexturePool {
@@ -46,7 +41,7 @@
     WTF_MAKE_FAST_ALLOCATED;
 public:
 #if USE(TEXTURE_MAPPER_GL)
-    explicit BitmapTexturePool(const TextureMapperContextAttributes&, RefPtr<GraphicsContext3D>&&);
+    explicit BitmapTexturePool(const TextureMapperContextAttributes&);
 #endif
 
     RefPtr<BitmapTexture> acquireTexture(const IntSize&, const BitmapTexture::Flags);
@@ -70,7 +65,6 @@
 
 #if USE(TEXTURE_MAPPER_GL)
     TextureMapperContextAttributes m_contextAttributes;
-    RefPtr<GraphicsContext3D> m_context3D;
 #endif
 
     Vector<Entry> m_textures;

Modified: trunk/Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp (220520 => 220521)


--- trunk/Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp	2017-08-10 07:30:04 UTC (rev 220520)
+++ trunk/Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp	2017-08-10 07:39:39 UTC (rev 220521)
@@ -172,7 +172,7 @@
 
     m_data = new TextureMapperGLData(*m_context3D);
 #if USE(TEXTURE_MAPPER_GL)
-    m_texturePool = std::make_unique<BitmapTexturePool>(m_contextAttributes, m_context3D.copyRef());
+    m_texturePool = std::make_unique<BitmapTexturePool>(m_contextAttributes);
 #endif
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to