Unreviewed, fix the Windows 10 debug build after r269525

whsieh · whsieh · commit 9b98de980eeb · 2020-11-06T20:22:59.000Z
Speculative fix: add `NO_RETURN_DUE_TO_ASSERT` to method declarations in DisplayListItems.h as well. * platform/graphics/displaylists/DisplayListItems.h: Canonical link: https://commits.webkit.org/231325@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@269530 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
@@ -1,3 +1,11 @@
+2020-11-06  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        Unreviewed, fix the Windows 10 debug build after r269525
+
+        Speculative fix: add `NO_RETURN_DUE_TO_ASSERT` to method declarations in DisplayListItems.h as well.
+
+        * platform/graphics/displaylists/DisplayListItems.h:
+
 2020-11-06  Joey Arhar  <jarhar@chromium.org>
 
         Text gets clobbered when assigning to input.defaultValue
diff --git a/Source/WebCore/platform/graphics/displaylists/DisplayListItems.h b/Source/WebCore/platform/graphics/displaylists/DisplayListItems.h
@@ -1328,7 +1328,7 @@ class DrawImageBuffer {
     template<class Encoder> void encode(Encoder&) const;
     template<class Decoder> static Optional<Ref<DrawImageBuffer>> decode(Decoder&);
 
-    void apply(GraphicsContext&) const;
+    NO_RETURN_DUE_TO_ASSERT void apply(GraphicsContext&) const;
 
     Optional<FloatRect> globalBounds() const { return WTF::nullopt; }
     Optional<FloatRect> localBounds(const GraphicsContext&) const { return m_destinationRect; }
@@ -2312,7 +2312,7 @@ class PutImageData {
     IntPoint destPoint() const { return m_destPoint; }
     AlphaPremultiplication destFormat() const { return m_destFormat; }
 
-    void apply(GraphicsContext&) const;
+    NO_RETURN_DUE_TO_ASSERT void apply(GraphicsContext&) const;
 
     Optional<FloatRect> localBounds(const GraphicsContext&) const { return WTF::nullopt; }
     Optional<FloatRect> globalBounds() const { return {{ m_destPoint, m_srcRect.size() }}; }
@@ -2381,7 +2381,7 @@ class PaintFrameForMedia {
     const FloatRect& destination() const { return m_destination; }
     MediaPlayerIdentifier identifier() const { return m_identifier; }
 
-    void apply(GraphicsContext&) const;
+    NO_RETURN_DUE_TO_ASSERT void apply(GraphicsContext&) const;
 
     Optional<FloatRect> localBounds(const GraphicsContext&) const { return WTF::nullopt; }
     Optional<FloatRect> globalBounds() const { return { m_destination }; }
