diff options
| author | ziadh@chromium.org <ziadh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-24 02:47:40 +0000 |
|---|---|---|
| committer | ziadh@chromium.org <ziadh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-24 02:47:40 +0000 |
| commit | 46ebf0640d32b313618fe443a34e959059b2c3f9 (patch) | |
| tree | 84b616ce5385bce6c8d4b966e74daf7648fdc8c5 /chrome/browser/aeropeek_manager.h | |
| parent | 7c543fd101874d1453dfcf9727dd0d39d296ef50 (diff) | |
| download | chromium_src-46ebf0640d32b313618fe443a34e959059b2c3f9.zip chromium_src-46ebf0640d32b313618fe443a34e959059b2c3f9.tar.gz chromium_src-46ebf0640d32b313618fe443a34e959059b2c3f9.tar.bz2 | |
Add undeclared virtual destructors.
Preventative maintainance for abstract classes that do not declare virtual destructors. Base classes that do not declare their destructors as virtual could potentially lead to memory leaks.
r=jar
Review URL: http://codereview.chromium.org/2856051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53570 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/aeropeek_manager.h')
| -rw-r--r-- | chrome/browser/aeropeek_manager.h | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/chrome/browser/aeropeek_manager.h b/chrome/browser/aeropeek_manager.h index 80d60db..4495a95 100644 --- a/chrome/browser/aeropeek_manager.h +++ b/chrome/browser/aeropeek_manager.h @@ -45,6 +45,9 @@ class AeroPeekWindowDelegate { virtual void GetContentInsets(gfx::Insets* insets) = 0; virtual bool GetTabThumbnail(int tab_id, SkBitmap* thumbnail) = 0; virtual bool GetTabPreview(int tab_id, SkBitmap* preview) = 0; + + protected: + virtual ~AeroPeekWindowDelegate() {} }; // A class that implements AeroPeek of Windows 7: @@ -91,7 +94,7 @@ class AeroPeekManager : public TabStripModelObserver, public AeroPeekWindowDelegate { public: explicit AeroPeekManager(HWND application_window); - ~AeroPeekManager(); + virtual ~AeroPeekManager(); // Sets the margins of the "user-perceived content area". // (See comments of |content_insets_|). |