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

Side by Side Diff: chrome/browser/ui/gtk/tab_contents_container_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_TAB_CONTENTS_CONTAINER_GTK_H_ 5 #ifndef CHROME_BROWSER_UI_GTK_TAB_CONTENTS_CONTAINER_GTK_H_
6 #define CHROME_BROWSER_UI_GTK_TAB_CONTENTS_CONTAINER_GTK_H_ 6 #define CHROME_BROWSER_UI_GTK_TAB_CONTENTS_CONTAINER_GTK_H_
7 #pragma once 7 #pragma once
8 8
9 #include <gtk/gtk.h> 9 #include <gtk/gtk.h>
10 10
11 #include "base/basictypes.h" 11 #include "base/basictypes.h"
12 #include "chrome/browser/ui/gtk/owned_widget_gtk.h" 12 #include "chrome/browser/ui/gtk/owned_widget_gtk.h"
13 #include "chrome/browser/ui/gtk/view_id_util.h" 13 #include "chrome/browser/ui/gtk/view_id_util.h"
14 #include "content/common/notification_observer.h" 14 #include "content/common/notification_observer.h"
15 #include "content/common/notification_registrar.h" 15 #include "content/common/notification_registrar.h"
16 #include "ui/base/gtk/gtk_signal.h" 16 #include "ui/base/gtk/gtk_signal.h"
17 17
18 class RenderViewHost; 18 class RenderViewHost;
19 class StatusBubbleGtk; 19 class StatusBubbleGtk;
20 class TabContents; 20 class TabContents;
21 class TabContentsWrapper; 21 class TabContentsWrapper;
22 22
23 typedef struct _GtkFloatingContainer GtkFloatingContainer; 23 typedef struct _GtkFloatingContainer GtkFloatingContainer;
24 24
25 class TabContentsContainerGtk : public NotificationObserver, 25 class TabContentsContainerGtk : public NotificationObserver,
26 public ViewIDUtil::Delegate { 26 public ViewIDUtil::Delegate {
27 public: 27 public:
28 explicit TabContentsContainerGtk(StatusBubbleGtk* status_bubble); 28 explicit TabContentsContainerGtk(StatusBubbleGtk* status_bubble);
29 ~TabContentsContainerGtk(); 29 virtual ~TabContentsContainerGtk();
30 30
31 void Init(); 31 void Init();
32 32
33 // Make the specified tab visible. 33 // Make the specified tab visible.
34 void SetTab(TabContentsWrapper* tab); 34 void SetTab(TabContentsWrapper* tab);
35 TabContentsWrapper* tab() const { return tab_; } 35 TabContentsWrapper* tab() const { return tab_; }
36 36
37 // Gets the tab contents currently being displayed (either |tab_contents_| or 37 // Gets the tab contents currently being displayed (either |tab_contents_| or
38 // |preview_contents_|). 38 // |preview_contents_|).
39 TabContents* GetVisibleTabContents(); 39 TabContents* GetVisibleTabContents();
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 // We insert and remove TabContents GtkWidgets into this expanded_. This 102 // We insert and remove TabContents GtkWidgets into this expanded_. This
103 // should not be a GtkVBox since there were errors with timing where the vbox 103 // should not be a GtkVBox since there were errors with timing where the vbox
104 // was horizontally split with the top half displaying the current TabContents 104 // was horizontally split with the top half displaying the current TabContents
105 // and bottom half displaying the loading page. 105 // and bottom half displaying the loading page.
106 GtkWidget* expanded_; 106 GtkWidget* expanded_;
107 107
108 DISALLOW_COPY_AND_ASSIGN(TabContentsContainerGtk); 108 DISALLOW_COPY_AND_ASSIGN(TabContentsContainerGtk);
109 }; 109 };
110 110
111 #endif // CHROME_BROWSER_UI_GTK_TAB_CONTENTS_CONTAINER_GTK_H_ 111 #endif // CHROME_BROWSER_UI_GTK_TAB_CONTENTS_CONTAINER_GTK_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/gtk/status_icons/status_tray_gtk.h ('k') | chrome/browser/ui/gtk/tab_contents_drag_source.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698