Changeset 88112 in webkit

Timestamp:

Jun 4, 2011 5:06:49 AM (13 years ago)

Author:

thakis@chromium.org

Message:

2011-06-04 Nico Weber <​thakis@chromium.org>

Reviewed by James Robinson.

[chromium] Make ResourceHandleInternal's destructor virtual
​https://bugs.webkit.org/show_bug.cgi?id=62073

This is _not_ to fix a real bug, just to make clang's
-Wdelete-non-virtual-dtor happy. As discussed at
​http://codereview.chromium.org/7094005/, we prefer making leaf class
destructors virtual over making the leaf classes final.

• src/ResourceHandle.cpp: (WebCore::ResourceHandleInternal::~ResourceHandleInternal):

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• src/ResourceHandle.cpp (modified) (1 diff)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2011-06-04  Nico Weber  <thakis@chromium.org>
+
+        Reviewed by James Robinson.
+
+        [chromium] Make ResourceHandleInternal's destructor virtual
+        https://bugs.webkit.org/show_bug.cgi?id=62073
+
+        This is _not_ to fix a real bug, just to make clang's
+        -Wdelete-non-virtual-dtor happy. As discussed at
+        http://codereview.chromium.org/7094005/, we prefer making leaf class
+        destructors virtual over making the leaf classes final.
+
+        * src/ResourceHandle.cpp:
+        (WebCore::ResourceHandleInternal::~ResourceHandleInternal):
+
 2011-06-04  Nico Weber  <thakis@chromium.org>
 

trunk/Source/WebKit/chromium/src/ResourceHandle.cpp

@@ -63 +63 @@
     }
 
+    virtual ~ResourceHandleInternal() { }
+
     void start();
     void cancel();