Skip to content
  • thakis@chromium.org's avatar
    2011-06-07 Nico Weber <thakis@chromium.org> · d0e4d419
    thakis@chromium.org authored
            Reviewed by Dimitri Glazkov.
    
            [chromium] -Wdelete-non-virtual-dtor pass for DumpRenderTree
            https://bugs.webkit.org/show_bug.cgi?id=62210
    
            The change to TestEventPrinter fixes a latent bug, because
            objects are deleted through the TestEventPrinter type, but none
            of the subclasses have destructors or non-POD members.
    
            The changes to NotificationPresenter and WebViewHost do _not_ to fix a
            real bug, they just 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.
    
            * DumpRenderTree/chromium/NotificationPresenter.cpp:
            (NotificationPresenter::~NotificationPresenter):
            * DumpRenderTree/chromium/NotificationPresenter.h:
            * DumpRenderTree/chromium/TestEventPrinter.cpp:
            (TestEventPrinter::~TestEventPrinter):
            * DumpRenderTree/chromium/TestEventPrinter.h:
            * DumpRenderTree/chromium/WebViewHost.h:
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@88246 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    d0e4d419