Attachment 373221 Details for Bug 199358 – Patch

[patch] Patch

bug-199358-20190701142927.patch (text/plain), 4.29 KB, created by Miguel Gomez on 2019-07-01 05:29:29 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Miguel Gomez

Created: 2019-07-01 05:29:29 PDT

Size: 4.29 KB

patch

obsolete

>Subversion Revision: 246964
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 157cee590b84dd72be80ba82cdab254915dcd22a..74af636ecbfcb1dce45c7cadc46074bea27882c9 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,18 @@
>+2019-07-01  Miguel Gomez  <magomez@igalia.com>
>+
>+        REGRESSION(r246963) GTK's debug build is broken
>+        https://bugs.webkit.org/show_bug.cgi?id=199358
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add traits to be able to downcast AnimatedBackingStoreClient.
>+
>+        * platform/graphics/nicosia/NicosiaAnimatedBackingStoreClient.h:
>+        (Nicosia::AnimatedBackingStoreClient::AnimatedBackingStoreClient):
>+        (Nicosia::AnimatedBackingStoreClient::type const):
>+        * platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp:
>+        (WebCore::CoordinatedGraphicsLayer::updateContentBuffers):
>+
> 2019-07-01  Carlos Garcia Campos  <cgarcia@igalia.com>
> 
>         WebSockets: add support for sending blob messages when using web sockets platform APIs
>diff --git a/Source/WebCore/platform/graphics/nicosia/NicosiaAnimatedBackingStoreClient.h b/Source/WebCore/platform/graphics/nicosia/NicosiaAnimatedBackingStoreClient.h
>index 21de15ecd54151c06795537be954cfa9e8160bfe..301d292471609923d7d413cd70621af46fe87463 100644
>--- a/Source/WebCore/platform/graphics/nicosia/NicosiaAnimatedBackingStoreClient.h
>+++ b/Source/WebCore/platform/graphics/nicosia/NicosiaAnimatedBackingStoreClient.h
>@@ -40,10 +40,28 @@ namespace Nicosia {
> 
> class AnimatedBackingStoreClient : public ThreadSafeRefCounted<AnimatedBackingStoreClient> {
> public:
>+    enum ClientType {
>+        Coordinated
>+    };
>+
>+    AnimatedBackingStoreClient(ClientType type)
>+        : m_type(type)
>+    {
>+    }
>+
>+    ClientType type() const { return m_type; }
>     virtual ~AnimatedBackingStoreClient() = default;
>     virtual void requestBackingStoreUpdateIfNeeded(const WebCore::TransformationMatrix&) = 0;
>+
>+private:
>+    ClientType m_type;
> };
> 
> } // namespace Nicosia
> 
>+#define SPECIALIZE_TYPE_TRAITS_ANIMATEDBACKINGSTORECLIENT(ToValueTypeName, predicate) \
>+SPECIALIZE_TYPE_TRAITS_BEGIN(ToValueTypeName) \
>+    static bool isType(const Nicosia::AnimatedBackingStoreClient& client) { return client.predicate; } \
>+SPECIALIZE_TYPE_TRAITS_END()
>+
> #endif // USE(COORDINATED_GRAPHICS)
>diff --git a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
>index 0b051a9be58f9b01434535f691df9bf0800473e8..6f573c41a61914492fdc8824f70002a508763cdd 100644
>--- a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
>+++ b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
>@@ -699,7 +699,8 @@ public:
> 
> private:
>     CoordinatedAnimatedBackingStoreClient(RefPtr<CoordinatedGraphicsLayer::AnimatedBackingStoreHost>&& host, const FloatRect& visibleRect, const FloatRect& coverRect, const FloatSize& size, float contentsScale)
>-        : m_host(WTFMove(host))
>+        : Nicosia::AnimatedBackingStoreClient(Coordinated)
>+        , m_host(WTFMove(host))
>         , m_visibleRect(visibleRect)
>         , m_coverRect(coverRect)
>         , m_size(size)
>@@ -1004,9 +1005,9 @@ void CoordinatedGraphicsLayer::updateContentBuffers()
>         layerState.mainBackingStore->createTilesIfNeeded(transformedVisibleRect(), IntRect(0, 0, m_size.width(), m_size.height()));
>     }
> 
>-    if (m_nicosia.animatedBackingStoreClient) {
>+    if (is<CoordinatedAnimatedBackingStoreClient>(m_nicosia.animatedBackingStoreClient)) {
>         // Determine the coverRect and set it to the client.
>-        downcast<CoordinatedAnimatedBackingStoreClient>(m_nicosia.animatedBackingStoreClient.get())->setCoverRect(layerState.mainBackingStore->coverRect());
>+        downcast<CoordinatedAnimatedBackingStoreClient>(*m_nicosia.animatedBackingStoreClient).setCoverRect(layerState.mainBackingStore->coverRect());
>     }
> 
>     ASSERT(m_coordinator && m_coordinator->isFlushingLayerChanges());
>@@ -1303,4 +1304,6 @@ bool CoordinatedGraphicsLayer::usesContentsLayer() const
> 
> } // namespace WebCore
> 
>+SPECIALIZE_TYPE_TRAITS_ANIMATEDBACKINGSTORECLIENT(WebCore::CoordinatedAnimatedBackingStoreClient, type() == Nicosia::AnimatedBackingStoreClient::Coordinated)
>+
> #endif // USE(COORDINATED_GRAPHICS)

Actions: View | Formatted Diff | Diff

Attachments on bug 199358: 373220 | 373221 | 373222 | 373227