[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