Changeset 88107 in webkit


Ignore:
Timestamp:
Jun 4, 2011 3:58:47 AM (13 years ago)
Author:
thakis@chromium.org
Message:

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

Reviewed by James Robinson.

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

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/PlatformMessagePortChannel.h:
Location:
trunk/Source/WebKit/chromium
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebKit/chromium/ChangeLog

    r88106 r88107  
     12011-06-04  Nico Weber  <thakis@chromium.org>
     2
     3        Reviewed by James Robinson.
     4
     5        [chromium] Make PlatformMessagePortChannel's destructor virtual
     6        https://bugs.webkit.org/show_bug.cgi?id=62072
     7
     8        This is _not_ to fix a real bug, just to make clang's
     9        -Wdelete-non-virtual-dtor happy. As discussed at
     10        http://codereview.chromium.org/7094005/, we prefer making leaf class
     11        destructors virtual over making the leaf classes final.
     12
     13        * src/PlatformMessagePortChannel.h:
     14
    1152011-06-04  Nico Weber  <thakis@chromium.org>
    216

  • trunk/Source/WebKit/chromium/src/PlatformMessagePortChannel.h

    r81567 r88107  
    6666    WebKit::WebMessagePortChannel* webChannelRelease();
    6767
    68     ~PlatformMessagePortChannel();
     68    virtual ~PlatformMessagePortChannel();
    6969
    7070private:
Note: See TracChangeset for help on using the changeset viewer.