Attachment 347804 Details for Bug 188850 – Patch

[patch] Patch

bug-188850-20180822181815.patch (text/plain), 8.86 KB, created by Zan Dobersek on 2018-08-22 09:18:17 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Zan Dobersek

Created: 2018-08-22 09:18:17 PDT

Size: 8.86 KB

patch

obsolete

>Subversion Revision: 235165
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 10922dc9633b88dd43b463f74641a3cbb52852c1..42820ee1b0261de2c174edc55505b81888b86bac 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,30 @@
>+2018-08-22  Zan Dobersek  <zdobersek@igalia.com>
>+
>+        [CoordGraphics] Drop old-school PlatformLayer management in CoordinatedGraphicsLayer
>+        https://bugs.webkit.org/show_bug.cgi?id=188850
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Remove the m_platformLayer member from the CoordinatedGraphicsLayer
>+        class as it's been unused since the rework in r235165.
>+
>+        The CoordinatedGraphicsLayer::syncPlatformLayer() helper method and the
>+        related member variable can both be removed as well. On the
>+        CoordinatedGraphicsLayerState class, we can remove the obsolete
>+        flags and the TextureMapperPlatformLayerProxy member variable that's
>+        unused now that such objects are handled through the Nicosia impl class.
>+
>+        * platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp:
>+        (WebCore::CoordinatedGraphicsLayer::CoordinatedGraphicsLayer):
>+        (WebCore::CoordinatedGraphicsLayer::setContentsNeedsDisplay):
>+        (WebCore::CoordinatedGraphicsLayer::setContentsToPlatformLayer):
>+        (WebCore::CoordinatedGraphicsLayer::updatePlatformLayer):
>+        (WebCore::CoordinatedGraphicsLayer::flushCompositingStateForThisLayerOnly):
>+        (WebCore::CoordinatedGraphicsLayer::syncPlatformLayer): Deleted.
>+        * platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h:
>+        * platform/graphics/texmap/coordinated/CoordinatedGraphicsState.h:
>+        (WebCore::CoordinatedGraphicsLayerState::CoordinatedGraphicsLayerState):
>+
> 2018-08-22  Zan Dobersek  <zdobersek@igalia.com>
> 
>         [CoordGraphics] Switch to Nicosia::CompositionLayer state tracking
>diff --git a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
>index 59db0f592ad8bb9496412fb9846fdb6cce3f81e6..362b7e0dfe37fb65855568b83cdd63b7ea2d3ebf 100644
>--- a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
>+++ b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
>@@ -141,12 +141,10 @@ CoordinatedGraphicsLayer::CoordinatedGraphicsLayer(Type layerType, GraphicsLayer
>     , m_pendingContentsScaleAdjustment(false)
>     , m_pendingVisibleRectAdjustment(false)
> #if USE(COORDINATED_GRAPHICS_THREADED)
>-    , m_shouldSyncPlatformLayer(false)
>     , m_shouldUpdatePlatformLayer(false)
> #endif
>     , m_coordinator(0)
>     , m_compositedNativeImagePtr(0)
>-    , m_platformLayer(0)
>     , m_animationStartedTimer(*this, &CoordinatedGraphicsLayer::animationStartedTimerFired)
> {
>     static CoordinatedLayerID nextLayerID = 1;
>@@ -426,14 +424,9 @@ bool GraphicsLayer::supportsContentsTiling()
> 
> void CoordinatedGraphicsLayer::setContentsNeedsDisplay()
> {
>-#if USE(COORDINATED_GRAPHICS_THREADED)
>-#if USE(NICOSIA)
>+#if USE(COORDINATED_GRAPHICS_THREADED) && USE(NICOSIA)
>     if (m_nicosia.contentLayer)
>         m_shouldUpdatePlatformLayer = true;
>-#else
>-    if (m_platformLayer)
>-        m_shouldUpdatePlatformLayer = true;
>-#endif
> #endif
> 
>     notifyFlushRequired();
>@@ -442,20 +435,12 @@ void CoordinatedGraphicsLayer::setContentsNeedsDisplay()
> 
> void CoordinatedGraphicsLayer::setContentsToPlatformLayer(PlatformLayer* platformLayer, ContentsLayerPurpose)
> {
>-#if USE(COORDINATED_GRAPHICS_THREADED)
>-#if USE(NICOSIA)
>+#if USE(COORDINATED_GRAPHICS_THREADED) && USE(NICOSIA)
>     auto* contentLayer = downcast<Nicosia::ContentLayer>(platformLayer);
>     if (m_nicosia.contentLayer != contentLayer) {
>-        m_shouldSyncPlatformLayer = true;
>         m_nicosia.contentLayer = contentLayer;
>         m_nicosia.delta.contentLayerChanged = true;
>     }
>-#else
>-    if (m_platformLayer != platformLayer)
>-        m_shouldSyncPlatformLayer = true;
>-
>-    m_platformLayer = platformLayer;
>-#endif
>     notifyFlushRequired();
> #else
>     UNUSED_PARAM(platformLayer);
>@@ -760,37 +745,15 @@ void CoordinatedGraphicsLayer::syncAnimations()
>     m_nicosia.delta.animationsChanged = true;
> }
> 
>-void CoordinatedGraphicsLayer::syncPlatformLayer()
>-{
>-    if (!m_shouldSyncPlatformLayer)
>-        return;
>-
>-    m_shouldSyncPlatformLayer = false;
>-#if USE(COORDINATED_GRAPHICS_THREADED)
>-#if USE(NICOSIA)
>-#else
>-    m_layerState.platformLayerChanged = true;
>-    if (m_platformLayer)
>-        m_layerState.platformLayerProxy = m_platformLayer->proxy();
>-#endif
>-#endif
>-}
>-
> void CoordinatedGraphicsLayer::updatePlatformLayer()
> {
>     if (!m_shouldUpdatePlatformLayer)
>         return;
> 
>     m_shouldUpdatePlatformLayer = false;
>-#if USE(COORDINATED_GRAPHICS_THREADED)
>-#if USE(NICOSIA)
>+#if USE(COORDINATED_GRAPHICS_THREADED) && USE(NICOSIA)
>     if (m_nicosia.contentLayer)
>         downcast<Nicosia::ContentLayerTextureMapperImpl>(m_nicosia.contentLayer->impl()).swapBuffersIfNeeded();
>-#else
>-    m_layerState.platformLayerUpdated = true;
>-    if (m_platformLayer)
>-        m_platformLayer->swapBuffersIfNeeded();
>-#endif
> #endif
> }
> 
>@@ -813,7 +776,6 @@ void CoordinatedGraphicsLayer::flushCompositingStateForThisLayerOnly()
>     computeTransformedVisibleRect();
>     syncChildren();
>     syncFilters();
>-    syncPlatformLayer();
>     updatePlatformLayer();
> 
>     // Only unset m_movingVisibleRect after we have updated the visible rect after the animation stopped.
>diff --git a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h
>index 7d6b66db3d2e19adc3e66c3da947112b8c894746..589778c39e892206776f1cceb6edaacffeba7ce4 100644
>--- a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h
>+++ b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h
>@@ -110,7 +110,7 @@ public:
>     void removeAnimation(const String&) override;
>     void suspendAnimations(MonotonicTime) override;
>     void resumeAnimations() override;
>-    bool usesContentsLayer() const override { return m_platformLayer || m_nicosia.contentLayer || m_compositedImage; }
>+    bool usesContentsLayer() const override { return m_nicosia.contentLayer || m_compositedImage; }
> 
>     void syncPendingStateChangesIncludingSubLayers();
>     void updateContentBuffersIncludingSubLayers();
>@@ -139,7 +139,6 @@ public:
> private:
>     bool isCoordinatedGraphicsLayer() const override { return true; }
> 
>-    void syncPlatformLayer();
>     void updatePlatformLayer();
> 
>     void setDebugBorder(const Color&, float width) override;
>@@ -202,7 +201,6 @@ private:
>     bool m_pendingContentsScaleAdjustment : 1;
>     bool m_pendingVisibleRectAdjustment : 1;
> #if USE(COORDINATED_GRAPHICS_THREADED)
>-    bool m_shouldSyncPlatformLayer : 1;
>     bool m_shouldUpdatePlatformLayer : 1;
> #endif
> 
>@@ -219,7 +217,6 @@ private:
>     NativeImagePtr m_compositedNativeImagePtr;
>     RefPtr<CoordinatedImageBacking> m_coordinatedImageBacking;
> 
>-    PlatformLayer* m_platformLayer;
>     Timer m_animationStartedTimer;
>     TextureMapperAnimations m_animations;
>     MonotonicTime m_lastAnimationStartTime;
>diff --git a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsState.h b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsState.h
>index 6f9624f4e5d4b101c2ee611113e19d987b6e499b..7b35950fc3fd6c37dfa00881f1882812791630a6 100644
>--- a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsState.h
>+++ b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsState.h
>@@ -43,10 +43,6 @@
> #include "TextureMapperAnimation.h"
> #include "TransformationMatrix.h"
> 
>-#if USE(COORDINATED_GRAPHICS_THREADED)
>-#include "TextureMapperPlatformLayerProxy.h"
>-#endif
>-
> namespace WebCore {
> 
> typedef uint32_t CoordinatedLayerID;
>@@ -97,9 +93,6 @@ struct CoordinatedGraphicsLayerState {
>             bool filtersChanged: 1;
>             bool childrenChanged: 1;
>             bool repaintCountChanged : 1;
>-            bool platformLayerChanged: 1;
>-            bool platformLayerUpdated: 1;
>-            bool platformLayerShouldSwapBuffers: 1;
>             bool isScrollableChanged: 1;
>             bool contentsTilingChanged: 1;
>         };
>@@ -131,9 +124,6 @@ struct CoordinatedGraphicsLayerState {
>         , replica(InvalidCoordinatedLayerID)
>         , mask(InvalidCoordinatedLayerID)
>         , imageID(InvalidCoordinatedImageBackingID)
>-#if USE(COORDINATED_GRAPHICS_THREADED)
>-        , platformLayerProxy(0)
>-#endif
>     {
>     }
> 
>@@ -160,10 +150,6 @@ struct CoordinatedGraphicsLayerState {
> 
>     Vector<TileUpdateInfo> tilesToUpdate;
> 
>-#if USE(COORDINATED_GRAPHICS_THREADED)
>-    RefPtr<TextureMapperPlatformLayerProxy> platformLayerProxy;
>-#endif
>-
>     bool hasPendingChanges() const
>     {
>         return changeMask || tilesToUpdate.size() || tilesToRemove.size() || tilesToCreate.size();

Actions: View | Formatted Diff | Diff

Attachments on bug 188850: 347804 | 347908