[SCM] WebKit Debian packaging branch, debian/experimental, updated. debian/1.3.8-1-1049-g2e11a8e

aroben at apple.com aroben at apple.com
Fri Jan 21 15:03:53 UTC 2011


The following commit has been merged in the debian/experimental branch:
commit bf05a27621bb3e61721708766cb174ab89d010d3
Author: aroben at apple.com <aroben at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Jan 6 20:46:49 2011 +0000

    Remove some dead code from WKCACFLayerRenderer
    
    Fixes <http://webkit.org/b/51954> WKCACFLayerRenderer has dead and
    redundant code for reacting to changes to the layer tree
    
    Reviewed by Sam Weinig.
    
    * platform/graphics/win/WKCACFLayerRenderer.cpp:
    * platform/graphics/win/WKCACFLayerRenderer.h:
    Removed didFlushContext and the HashMap it depended on. We react to
    changes to the layer tree in layerTreeDidChange instead.
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@75182 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 1aa76a6..1874f55 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,17 @@
+2011-01-05  Adam Roben  <aroben at apple.com>
+
+        Remove some dead code from WKCACFLayerRenderer
+
+        Fixes <http://webkit.org/b/51954> WKCACFLayerRenderer has dead and
+        redundant code for reacting to changes to the layer tree
+
+        Reviewed by Sam Weinig.
+
+        * platform/graphics/win/WKCACFLayerRenderer.cpp:
+        * platform/graphics/win/WKCACFLayerRenderer.h:
+        Removed didFlushContext and the HashMap it depended on. We react to
+        changes to the layer tree in layerTreeDidChange instead.
+
 2011-01-06  Chris Fleizach  <cfleizach at apple.com>
 
         Reviewed by Beth Dakin.
diff --git a/WebCore/platform/graphics/win/WKCACFLayerRenderer.cpp b/WebCore/platform/graphics/win/WKCACFLayerRenderer.cpp
index 73cb794..870c53a 100644
--- a/WebCore/platform/graphics/win/WKCACFLayerRenderer.cpp
+++ b/WebCore/platform/graphics/win/WKCACFLayerRenderer.cpp
@@ -106,14 +106,6 @@ private:
     WKCACFLayerRenderer* m_renderer;
 };
 
-typedef HashMap<WKCACFContext*, WKCACFLayerRenderer*> ContextToWindowMap;
-
-static ContextToWindowMap& windowsForContexts()
-{
-    DEFINE_STATIC_LOCAL(ContextToWindowMap, map, ());
-    return map;
-}
-
 static D3DPRESENT_PARAMETERS initialPresentationParameters()
 {
     D3DPRESENT_PARAMETERS parameters = {0};
@@ -204,15 +196,6 @@ bool WKCACFLayerRenderer::acceleratedCompositingAvailable()
     return available;
 }
 
-void WKCACFLayerRenderer::didFlushContext(WKCACFContext* context)
-{
-    WKCACFLayerRenderer* window = windowsForContexts().get(context);
-    if (!window)
-        return;
-
-    window->renderSoon();
-}
-
 PassOwnPtr<WKCACFLayerRenderer> WKCACFLayerRenderer::create(WKCACFLayerRendererClient* client)
 {
     if (!acceleratedCompositingAvailable())
@@ -230,8 +213,6 @@ WKCACFLayerRenderer::WKCACFLayerRenderer(WKCACFLayerRendererClient* client)
     , m_backingStoreDirty(false)
     , m_mustResetLostDeviceBeforeRendering(false)
 {
-    windowsForContexts().set(m_context, this);
-
     // Under the root layer, we have a clipping layer to clip the content,
     // that contains a scroll layer that we use for scrolling the content.
     // The root layer is the size of the client area of the window.
@@ -380,10 +361,8 @@ bool WKCACFLayerRenderer::createRenderer()
 
 void WKCACFLayerRenderer::destroyRenderer()
 {
-    if (m_context) {
-        windowsForContexts().remove(m_context);
+    if (m_context)
         WKCACFContextFlusher::shared().removeContext(m_context);
-    }
 
     m_d3dDevice = 0;
     if (s_d3d)
diff --git a/WebCore/platform/graphics/win/WKCACFLayerRenderer.h b/WebCore/platform/graphics/win/WKCACFLayerRenderer.h
index 763fffa..fae853c 100644
--- a/WebCore/platform/graphics/win/WKCACFLayerRenderer.h
+++ b/WebCore/platform/graphics/win/WKCACFLayerRenderer.h
@@ -62,7 +62,6 @@ public:
     ~WKCACFLayerRenderer();
 
     static bool acceleratedCompositingAvailable();
-    static void didFlushContext(WKCACFContext*);
 
     void setRootContents(CGImageRef);
     void setRootContentsAndDisplay(CGImageRef);

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list