Virtual destructors should have virtual keyword.

chrome/browser/extensions/extension_test_message_listener.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_TEST_MESSAGE_LISTENER_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_TEST_MESSAGE_LISTENER_H_
 #pragma once
 
 #include <string>
 
@@ skipping 29 matching lines @@
 // Note that when using it in browser tests, you need to make sure it gets
 // destructed *before* the browser gets torn down. Two common patterns are to
 // either make it a local variable inside your test body, or if it's a member
 // variable of a ExtensionBrowserTest subclass, override the
 // InProcessBrowserTest::CleanUpOnMainThread() method and clean it up there.
 class ExtensionTestMessageListener : public NotificationObserver {
  public:
   // We immediately start listening for |expected_message|.
   ExtensionTestMessageListener(const std::string& expected_message,
                                bool will_reply);
-  ~ExtensionTestMessageListener();
+  virtual ~ExtensionTestMessageListener();
 
   // This returns true immediately if we've already gotten the expected
   // message, or waits until it arrives. Returns false if the wait is
   // interrupted and we still haven't gotten the message.
   bool WaitUntilSatisfied();
 
   // Send the given message as a reply. It is only valid to call this after
   // WaitUntilSatisfied has returned true, and if will_reply is true.
   void Reply(const std::string& message);
 
@@ skipping 19 matching lines @@
 
   // If true, we expect the calling code to manually send a reply. Otherwise,
   // we send an automatic empty reply to the extension.
   bool will_reply_;
 
   // The function we need to reply to.
   ExtensionTestSendMessageFunction* function_;
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_TEST_MESSAGE_LISTENER_H_