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

Unified Diff: chrome/browser/ui/gtk/browser_window_gtk.h

Issue 8608002: gtk: Add missing OVERRIDE annotations to BrowserWindowGtk. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/gtk/browser_window_gtk.h
diff --git a/chrome/browser/ui/gtk/browser_window_gtk.h b/chrome/browser/ui/gtk/browser_window_gtk.h
index 7cff44ac4bcf27b852926a60bbb2d90cb81e85b4..6bf2da4796f9bc03385200208a1fc1bf7f2fceb4 100644
--- a/chrome/browser/ui/gtk/browser_window_gtk.h
+++ b/chrome/browser/ui/gtk/browser_window_gtk.h
@@ -10,6 +10,8 @@
#include <string>
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
#include "base/memory/scoped_ptr.h"
#include "base/timer.h"
#include "build/build_config.h"
@@ -67,87 +69,91 @@ class BrowserWindowGtk : public BrowserWindow,
// Returns the type of text used for title.
virtual TitleDecoration GetWindowTitle(std::string* title) const;
- // Overridden from BrowserWindow
- virtual void Show();
- virtual void ShowInactive();
- virtual void SetBounds(const gfx::Rect& bounds);
- virtual void Close();
- virtual void Activate();
- virtual void Deactivate();
- virtual bool IsActive() const;
- virtual void FlashFrame();
- virtual gfx::NativeWindow GetNativeHandle();
- virtual BrowserWindowTesting* GetBrowserWindowTesting();
- virtual StatusBubble* GetStatusBubble();
- virtual void ToolbarSizeChanged(bool is_animating);
- virtual void UpdateTitleBar();
+ // Overridden from BrowserWindow:
+ virtual void Show() OVERRIDE;
+ virtual void ShowInactive() OVERRIDE;
+ virtual void SetBounds(const gfx::Rect& bounds) OVERRIDE;
+ virtual void Close() OVERRIDE;
+ virtual void Activate() OVERRIDE;
+ virtual void Deactivate() OVERRIDE;
+ virtual bool IsActive() const OVERRIDE;
+ virtual void FlashFrame() OVERRIDE;
+ virtual gfx::NativeWindow GetNativeHandle() OVERRIDE;
+ virtual BrowserWindowTesting* GetBrowserWindowTesting() OVERRIDE;
+ virtual StatusBubble* GetStatusBubble() OVERRIDE;
+ virtual void ToolbarSizeChanged(bool is_animating) OVERRIDE;
+ virtual void UpdateTitleBar() OVERRIDE;
virtual void BookmarkBarStateChanged(
- BookmarkBar::AnimateChangeType change_type);
- virtual void UpdateDevTools();
- virtual void UpdateLoadingAnimations(bool should_animate);
- virtual void SetStarredState(bool is_starred);
- virtual gfx::Rect GetRestoredBounds() const;
- virtual gfx::Rect GetBounds() const;
- virtual bool IsMaximized() const;
- virtual bool IsMinimized() const;
+ BookmarkBar::AnimateChangeType change_type) OVERRIDE;
+ virtual void UpdateDevTools() OVERRIDE;
+ virtual void UpdateLoadingAnimations(bool should_animate) OVERRIDE;
+ virtual void SetStarredState(bool is_starred) OVERRIDE;
+ virtual gfx::Rect GetRestoredBounds() const OVERRIDE;
+ virtual gfx::Rect GetBounds() const OVERRIDE;
+ virtual bool IsMaximized() const OVERRIDE;
+ virtual bool IsMinimized() const OVERRIDE;
virtual void EnterFullscreen(
const GURL& url, FullscreenExitBubbleType type) OVERRIDE;
virtual void ExitFullscreen() OVERRIDE;
virtual void UpdateFullscreenExitBubbleContent(
const GURL& url,
FullscreenExitBubbleType bubble_type) OVERRIDE;
- virtual bool IsFullscreen() const;
- virtual bool IsFullscreenBubbleVisible() const;
- virtual LocationBar* GetLocationBar() const;
- virtual void SetFocusToLocationBar(bool select_all);
- virtual void UpdateReloadStopState(bool is_loading, bool force);
+ virtual bool IsFullscreen() const OVERRIDE;
+ virtual bool IsFullscreenBubbleVisible() const OVERRIDE;
+ virtual LocationBar* GetLocationBar() const OVERRIDE;
+ virtual void SetFocusToLocationBar(bool select_all) OVERRIDE;
+ virtual void UpdateReloadStopState(bool is_loading, bool force) OVERRIDE;
virtual void UpdateToolbar(TabContentsWrapper* contents,
- bool should_restore_state);
- virtual void FocusToolbar();
- virtual void FocusAppMenu();
- virtual void FocusBookmarksToolbar();
- virtual void FocusChromeOSStatus();
- virtual void RotatePaneFocus(bool forwards);
- virtual bool IsBookmarkBarVisible() const;
- virtual bool IsBookmarkBarAnimating() const;
- virtual bool IsTabStripEditable() const;
- virtual bool IsToolbarVisible() const;
+ bool should_restore_state) OVERRIDE;
+ virtual void FocusToolbar() OVERRIDE;
+ virtual void FocusAppMenu() OVERRIDE;
+ virtual void FocusBookmarksToolbar() OVERRIDE;
+ virtual void FocusChromeOSStatus() OVERRIDE;
+ virtual void RotatePaneFocus(bool forwards) OVERRIDE;
+ virtual bool IsBookmarkBarVisible() const OVERRIDE;
+ virtual bool IsBookmarkBarAnimating() const OVERRIDE;
+ virtual bool IsTabStripEditable() const OVERRIDE;
+ virtual bool IsToolbarVisible() const OVERRIDE;
virtual void ConfirmAddSearchProvider(const TemplateURL* template_url,
- Profile* profile);
- virtual void ToggleBookmarkBar();
- virtual void ShowAboutChromeDialog();
- virtual void ShowUpdateChromeDialog();
- virtual void ShowTaskManager();
- virtual void ShowBackgroundPages();
- virtual void ShowBookmarkBubble(const GURL& url, bool already_bookmarked);
- virtual bool IsDownloadShelfVisible() const;
- virtual DownloadShelf* GetDownloadShelf();
- virtual void ShowRepostFormWarningDialog(TabContents* tab_contents);
- virtual void ShowCollectedCookiesDialog(TabContentsWrapper* tab_contents);
- virtual void ConfirmBrowserCloseWithPendingDownloads();
- virtual void UserChangedTheme();
- virtual int GetExtraRenderViewHeight() const;
- virtual void TabContentsFocused(TabContents* tab_contents);
+ Profile* profile) OVERRIDE;
+ virtual void ToggleBookmarkBar() OVERRIDE;
+ virtual void ShowAboutChromeDialog() OVERRIDE;
+ virtual void ShowUpdateChromeDialog() OVERRIDE;
+ virtual void ShowTaskManager() OVERRIDE;
+ virtual void ShowBackgroundPages() OVERRIDE;
+ virtual void ShowBookmarkBubble(const GURL& url,
+ bool already_bookmarked) OVERRIDE;
+ virtual bool IsDownloadShelfVisible() const OVERRIDE;
+ virtual DownloadShelf* GetDownloadShelf() OVERRIDE;
+ virtual void ShowRepostFormWarningDialog(TabContents* tab_contents) OVERRIDE;
+ virtual void ShowCollectedCookiesDialog(
+ TabContentsWrapper* tab_contents) OVERRIDE;
+ virtual void ConfirmBrowserCloseWithPendingDownloads() OVERRIDE;
+ virtual void UserChangedTheme() OVERRIDE;
+ virtual int GetExtraRenderViewHeight() const OVERRIDE;
+ virtual void TabContentsFocused(TabContents* tab_contents) OVERRIDE;
virtual void ShowPageInfo(Profile* profile,
const GURL& url,
const NavigationEntry::SSLStatus& ssl,
- bool show_history);
- virtual void ShowAppMenu();
+ bool show_history) OVERRIDE;
+ virtual void ShowAppMenu() OVERRIDE;
virtual bool PreHandleKeyboardEvent(const NativeWebKeyboardEvent& event,
- bool* is_keyboard_shortcut);
- virtual void HandleKeyboardEvent(const NativeWebKeyboardEvent& event);
+ bool* is_keyboard_shortcut) OVERRIDE;
+ virtual void HandleKeyboardEvent(
+ const NativeWebKeyboardEvent& event) OVERRIDE;
virtual void ShowCreateWebAppShortcutsDialog(
- TabContentsWrapper* tab_contents);
- virtual void ShowCreateChromeAppShortcutsDialog(Profile* profile,
- const Extension* app);
- virtual void Cut();
- virtual void Copy();
- virtual void Paste();
- virtual void ShowInstant(TabContentsWrapper* preview);
- virtual void HideInstant();
- virtual gfx::Rect GetInstantBounds();
+ TabContentsWrapper* tab_contents) OVERRIDE;
+ virtual void ShowCreateChromeAppShortcutsDialog(
+ Profile* profile,
+ const Extension* app) OVERRIDE;
+ virtual void Cut() OVERRIDE;
+ virtual void Copy() OVERRIDE;
+ virtual void Paste() OVERRIDE;
+ virtual void ShowInstant(TabContentsWrapper* preview) OVERRIDE;
+ virtual void HideInstant() OVERRIDE;
+ virtual gfx::Rect GetInstantBounds() OVERRIDE;
virtual WindowOpenDisposition GetDispositionForPopupBounds(
- const gfx::Rect& bounds);
+ const gfx::Rect& bounds) OVERRIDE;
virtual FindBar* CreateFindBar() OVERRIDE;
virtual void ShowAvatarBubble(TabContents* tab_contents,
const gfx::Rect& rect) OVERRIDE;
@@ -156,22 +162,22 @@ class BrowserWindowGtk : public BrowserWindow,
// Overridden from NotificationObserver:
virtual void Observe(int type,
const content::NotificationSource& source,
- const content::NotificationDetails& details);
+ const content::NotificationDetails& details) OVERRIDE;
// Overridden from TabStripModelObserver:
- virtual void TabDetachedAt(TabContentsWrapper* contents, int index);
+ virtual void TabDetachedAt(TabContentsWrapper* contents, int index) OVERRIDE;
virtual void ActiveTabChanged(TabContentsWrapper* old_contents,
TabContentsWrapper* new_contents,
int index,
- bool user_gesture);
+ bool user_gesture) OVERRIDE;
// Overridden from ActiveWindowWatcher::Observer.
- virtual void ActiveWindowChanged(GdkWindow* active_window);
+ virtual void ActiveWindowChanged(GdkWindow* active_window) OVERRIDE;
// Overridden from InfoBarContainer::Delegate:
- virtual SkColor GetInfoBarSeparatorColor() const;
- virtual void InfoBarContainerStateChanged(bool is_animating);
- virtual bool DrawInfoBarArrows(int* x) const;
+ virtual SkColor GetInfoBarSeparatorColor() const OVERRIDE;
+ virtual void InfoBarContainerStateChanged(bool is_animating) OVERRIDE;
+ virtual bool DrawInfoBarArrows(int* x) const OVERRIDE;
// Accessor for the tab strip.
TabStripGtk* tabstrip() const { return tabstrip_.get(); }
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698