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

Side by Side Diff: chrome/browser/ui/cocoa/notifications/balloon_view_host_mac.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) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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_COCOA_NOTIFICATIONS_BALLOON_VIEW_HOST_MAC_H_ 5 #ifndef CHROME_BROWSER_UI_COCOA_NOTIFICATIONS_BALLOON_VIEW_HOST_MAC_H_
6 #define CHROME_BROWSER_UI_COCOA_NOTIFICATIONS_BALLOON_VIEW_HOST_MAC_H_ 6 #define CHROME_BROWSER_UI_COCOA_NOTIFICATIONS_BALLOON_VIEW_HOST_MAC_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 10
11 class RenderWidgetHostView; 11 class RenderWidgetHostView;
12 class RenderWidgetHostViewMac; 12 class RenderWidgetHostViewMac;
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 23
24 // Changes the size of the balloon. 24 // Changes the size of the balloon.
25 void UpdateActualSize(const gfx::Size& new_size); 25 void UpdateActualSize(const gfx::Size& new_size);
26 26
27 // Accessors. 27 // Accessors.
28 gfx::NativeView native_view() const; 28 gfx::NativeView native_view() const;
29 29
30 protected: 30 protected:
31 virtual void InitRenderWidgetHostView(); 31 virtual void InitRenderWidgetHostView();
32 virtual RenderWidgetHostView* render_widget_host_view() const; 32 virtual RenderWidgetHostView* render_widget_host_view() const;
33 33
34 private: 34 private:
35 // The Mac-specific widget host view. This is owned by its native view, 35 // The Mac-specific widget host view. This is owned by its native view,
36 // which this class frees in its destructor. 36 // which this class frees in its destructor.
37 RenderWidgetHostViewMac* render_widget_host_view_; 37 RenderWidgetHostViewMac* render_widget_host_view_;
38 38
39 DISALLOW_COPY_AND_ASSIGN(BalloonViewHost); 39 DISALLOW_COPY_AND_ASSIGN(BalloonViewHost);
40 }; 40 };
41 41
42 #endif // CHROME_BROWSER_UI_COCOA_NOTIFICATIONS_BALLOON_VIEW_HOST_MAC_H_ 42 #endif // CHROME_BROWSER_UI_COCOA_NOTIFICATIONS_BALLOON_VIEW_HOST_MAC_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/cocoa/notifications/balloon_view_bridge.h ('k') | chrome/browser/ui/cocoa/theme_install_bubble_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698