Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1624)

Unified Diff: chrome/browser/extensions/app_notify_channel_setup.h

Issue 12086018: GTTF: Add missing virtual destructors. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: trybots 4 Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..a0388e59c7a60d5c2d93324df837b00256ac0d09 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.
@@ -64,6 +66,8 @@ class AppNotifyChannelSetup
// forcing the return of a certain result to the delegate.
class InterceptorForTests {
public:
+ virtual ~InterceptorForTests() { }
+
virtual void DoIntercept(
const AppNotifyChannelSetup* setup,
std::string* result_channel_id,
« no previous file with comments | « chrome/browser/extensions/api/page_capture/page_capture_api.h ('k') | chrome/browser/extensions/event_router.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698