[webkit-reviews] review requested: [Bug 170928] [WinCairo] 'WebCore::GraphicsLayerTextureMapper::flushCompositingStated': method with override specifier 'override' did not override any base class methods : [Attachment 307333] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Apr 17 19:27:13 PDT 2017
Fujii Hironori <Hironori.Fujii at sony.com> has asked for review:
Bug 170928: [WinCairo]
'WebCore::GraphicsLayerTextureMapper::flushCompositingStated': method with
override specifier 'override' did not override any base class methods
https://bugs.webkit.org/show_bug.cgi?id=170928
Attachment 307333: Patch
https://bugs.webkit.org/attachment.cgi?id=307333&action=review
--- Comment #3 from Fujii Hironori <Hironori.Fujii at sony.com> ---
Comment on attachment 307333
--> https://bugs.webkit.org/attachment.cgi?id=307333
Patch
>Subversion Revision: 215441
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index
c84582531b1db89a981fb6288c045152f2175d7a..3305e2ab6d3443e9600c7470b5589e8e24449
13e 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,16 @@
>+2017-04-17 Fujii Hironori <Hironori.Fujii at sony.com>
>+
>+ [WinCairo]
'WebCore::GraphicsLayerTextureMapper::flushCompositingStated': method with
override specifier 'override' did not override any base class methods
>+ https://bugs.webkit.org/show_bug.cgi?id=170928
>+
>+ Reviewed by NOBODY (OOPS!).
>+
>+ Apply the same change of CoordinatedGraphicsLayer.{cpp,h} of r215410
to GraphicsLayerTextureMapper.{cpp,h}.
>+
>+ * platform/graphics/texmap/GraphicsLayerTextureMapper.cpp:
>+ (WebCore::GraphicsLayerTextureMapper::flushCompositingState): Added
the second arguent.
>+ * platform/graphics/texmap/GraphicsLayerTextureMapper.h: Ditto.
>+
> 2017-04-17 Alex Christensen <achristensen at webkit.org>
>
> Fix CMake build.
>diff --git
a/Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.cpp
b/Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.cpp
>index
78a0246f7f550882c30d5fd06a24222eec80ac04..8f467822b4e38961d8a71c6314aa014de9ed4
982 100644
>--- a/Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.cpp
>+++ b/Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.cpp
>@@ -497,7 +497,7 @@ void GraphicsLayerTextureMapper::commitLayerChanges()
> m_changeMask = NoChanges;
> }
>
>-void GraphicsLayerTextureMapper::flushCompositingState(const FloatRect& rect)
>+void GraphicsLayerTextureMapper::flushCompositingState(const FloatRect& rect,
FlushScope)
> {
> if (!m_layer.textureMapper())
> return;
>diff --git
a/Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h
b/Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h
>index
42ed713d3e633a4c12bc874ccb5b45f5391311ad..71bfe61e0c693300af5bd3b613b10e74b4862
968 100644
>--- a/Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h
>+++ b/Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h
>@@ -83,7 +83,7 @@ public:
> void setDebugBorder(const Color&, float width) override;
> void setShowRepaintCounter(bool) override;
>
>- void flushCompositingState(const FloatRect&) override;
>+ void flushCompositingState(const FloatRect&, FlushScope) override;
> void flushCompositingStateForThisLayerOnly() override;
>
> void updateBackingStoreIncludingSubLayers();
More information about the webkit-reviews
mailing list