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

Side by Side Diff: chrome/browser/ui/gtk/notifications/balloon_view_host_gtk.h

Issue 7064033: Virtual destructors should have virtual keyword. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 years, 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_GTK_NOTIFICATIONS_BALLOON_VIEW_HOST_GTK_H_ 5 #ifndef CHROME_BROWSER_UI_GTK_NOTIFICATIONS_BALLOON_VIEW_HOST_GTK_H_
6 #define CHROME_BROWSER_UI_GTK_NOTIFICATIONS_BALLOON_VIEW_HOST_GTK_H_ 6 #define CHROME_BROWSER_UI_GTK_NOTIFICATIONS_BALLOON_VIEW_HOST_GTK_H_
7 #pragma once 7 #pragma once
8 8
9 #include "chrome/browser/notifications/balloon_host.h" 9 #include "chrome/browser/notifications/balloon_host.h"
10 #include "ui/gfx/native_widget_types.h" 10 #include "ui/gfx/native_widget_types.h"
11 11
12 class RenderWidgetHostViewGtk; 12 class RenderWidgetHostViewGtk;
13 13
14 // BalloonViewHost class is a delegate to the renderer host for the HTML 14 // BalloonViewHost class is a delegate to the renderer host for the HTML
15 // notification. When initialized it creates a new RenderViewHost and loads 15 // notification. When initialized it creates a new RenderViewHost and loads
16 // the contents of the toast into it. It also handles links within the toast, 16 // the contents of the toast into it. It also handles links within the toast,
17 // loading them into a new tab. 17 // loading them into a new tab.
18 class BalloonViewHost : public BalloonHost { 18 class BalloonViewHost : public BalloonHost {
19 public: 19 public:
20 explicit BalloonViewHost(Balloon* balloon); 20 explicit BalloonViewHost(Balloon* balloon);
21 21
22 ~BalloonViewHost() { 22 virtual ~BalloonViewHost();
23 Shutdown();
24 }
25 23
26 // Changes the size of the balloon. 24 // Changes the size of the balloon.
27 void UpdateActualSize(const gfx::Size& new_size); 25 void UpdateActualSize(const gfx::Size& new_size);
28 26
29 // Accessors. 27 // Accessors.
30 gfx::NativeView native_view() const; 28 gfx::NativeView native_view() const;
31 29
32 protected: 30 protected:
33 virtual void InitRenderWidgetHostView(); 31 virtual void InitRenderWidgetHostView();
34 virtual RenderWidgetHostView* render_widget_host_view() const; 32 virtual RenderWidgetHostView* render_widget_host_view() const;
35 33
36 private: 34 private:
37 // The GTK-specific widget host view. Owned by its native view. 35 // The GTK-specific widget host view. Owned by its native view.
38 RenderWidgetHostViewGtk* render_widget_host_view_; 36 RenderWidgetHostViewGtk* render_widget_host_view_;
39 37
40 DISALLOW_COPY_AND_ASSIGN(BalloonViewHost); 38 DISALLOW_COPY_AND_ASSIGN(BalloonViewHost);
41 }; 39 };
42 40
43 #endif // CHROME_BROWSER_UI_GTK_NOTIFICATIONS_BALLOON_VIEW_HOST_GTK_H_ 41 #endif // CHROME_BROWSER_UI_GTK_NOTIFICATIONS_BALLOON_VIEW_HOST_GTK_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/gtk/download/download_shelf_gtk.h ('k') | chrome/browser/ui/gtk/notifications/balloon_view_host_gtk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698