KeyHandle should have a virtual destructor

https://bugs.webkit.org/show_bug.cgi?id=274354 Reviewed by Chris Dumez. Added a virtual destructor since this class has subclasses. * Source/WebCore/platform/encryptedmedia/CDMProxy.h: (WebCore::KeyHandle::~KeyHandle): Canonical link: https://commits.webkit.org/278963@main
WebKit · May 18, 2024 · 20328d8 · 20328d8
1 parent 5713584
commit 20328d8
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/Source/WebCore/platform/encryptedmedia/CDMProxy.h b/Source/WebCore/platform/encryptedmedia/CDMProxy.h
@@ -74,6 +74,9 @@ class KeyHandle : public ThreadSafeRefCounted<KeyHandle> {
     {
         return adoptRef(*new KeyHandle(status, WTFMove(keyID), WTFMove(keyHandleValue)));
     }
+
+    virtual ~KeyHandle() { }
+
     Ref<SharedBuffer> idAsSharedBuffer() const { return SharedBuffer::create(m_id.span()); }
 
     bool takeValueIfDifferent(KeyHandleValueVariant&&);