Changeset 121884 in webkit

Timestamp:

Jul 4, 2012 11:53:59 PM (12 years ago)

Author:

commit-queue@webkit.org

Message:

Add virtual keyword to ~BitmapImage and ~StillImage because ~Image is virtual.
​https://bugs.webkit.org/show_bug.cgi?id=90447

Patch by Huang Dongsung <​luxtella@company100.net> on 2012-07-04
Reviewed by Alexey Proskuryakov.

No new tests. This patch doesn't change behavior.

• platform/graphics/BitmapImage.h:

(BitmapImage):

• platform/graphics/qt/StillImageQt.h:

(StillImage):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/BitmapImage.h (modified) (1 diff)
• platform/graphics/qt/StillImageQt.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-07-04  Huang Dongsung  <luxtella@company100.net>
+
+        Add virtual keyword to ~BitmapImage and ~StillImage because ~Image is virtual.
+        https://bugs.webkit.org/show_bug.cgi?id=90447
+
+        Reviewed by Alexey Proskuryakov.
+
+        No new tests. This patch doesn't change behavior.
+
+        * platform/graphics/BitmapImage.h:
+        (BitmapImage):
+        * platform/graphics/qt/StillImageQt.h:
+        (StillImage):
+
 2012-07-04  Mike West  <mkwst@chromium.org>
 

trunk/Source/WebCore/platform/graphics/BitmapImage.h

@@ -113 +113 @@
         return adoptRef(new BitmapImage(observer));
     }
-    ~BitmapImage();
+    virtual ~BitmapImage();
     
     virtual bool isBitmapImage() const;

trunk/Source/WebCore/platform/graphics/qt/StillImageQt.h

@@ -59 +59 @@
         StillImage(const QPixmap& pixmap);
         StillImage(const QPixmap* pixmap);
-        ~StillImage();
+        virtual ~StillImage();
         
         const QPixmap* m_pixmap;