GTTF: Add missing virtual destructors.

chrome/browser/extensions/app_notify_channel_setup.h

Index: chrome/browser/extensions/app_notify_channel_setup.h
diff --git a/chrome/browser/extensions/app_notify_channel_setup.h b/chrome/browser/extensions/app_notify_channel_setup.h
index a43e08b3cf3a0ac22c6ecded36a9c9b77f575882..a98b8401e31b57177cd5b332c74ac1781dfc604a 100644
--- a/chrome/browser/extensions/app_notify_channel_setup.h
+++ b/chrome/browser/extensions/app_notify_channel_setup.h
@@ -51,6 +51,8 @@ class AppNotifyChannelSetup
 
   class Delegate {
    public:
+    virtual ~Delegate() { }
+
     // If successful, |channel_id| will be non-empty. On failure, |channel_id|
     // will be empty and |error| will contain an error to report to the JS
     // callback.
@@ -68,6 +70,9 @@ class AppNotifyChannelSetup
         const AppNotifyChannelSetup* setup,
         std::string* result_channel_id,
         AppNotifyChannelSetup::SetupError* result_error) = 0;
+
+   protected:
+    virtual ~InterceptorForTests() { }

[jar (doing other things), 2013/02/01 03:41:07]

nit: IMO, better is public, unless you have a STRONG reason, so as to not change
existing semantics.

[Paweł Hajdan Jr., 2013/02/01 10:12:43]

Done, sorry I just forgot to change it to public.

   };
   static void SetInterceptorForTests(InterceptorForTests* interceptor);