diff options
author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-15 23:28:19 +0000 |
---|---|---|
committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-15 23:28:19 +0000 |
commit | c9bd2e8e19e8c5891d68e99b4c610c04a62046a0 (patch) | |
tree | 689e7d832227bd431aadd4698807edb246030fc8 /chrome/browser/ui/cocoa | |
parent | 97e521cba140ce053f33dd3e852e847c76781689 (diff) | |
download | chromium_src-c9bd2e8e19e8c5891d68e99b4c610c04a62046a0.zip chromium_src-c9bd2e8e19e8c5891d68e99b4c610c04a62046a0.tar.gz chromium_src-c9bd2e8e19e8c5891d68e99b4c610c04a62046a0.tar.bz2 |
Remove the TabContentsDelegate::ToolbarSizeChanged() method as no one was using it. Rename BrowserWindow::SelectedTabToolbarSizeChanged() to ToolbarSizeChanged() as there are no per-tab toolbars since we made the DownloadShelf per-window a while ago.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6870002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81834 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/cocoa')
4 files changed, 2 insertions, 9 deletions
diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.h b/chrome/browser/ui/cocoa/browser_window_cocoa.h index 3745556..11d09fd 100644 --- a/chrome/browser/ui/cocoa/browser_window_cocoa.h +++ b/chrome/browser/ui/cocoa/browser_window_cocoa.h @@ -43,7 +43,7 @@ class BrowserWindowCocoa : public BrowserWindow, virtual gfx::NativeWindow GetNativeHandle(); virtual BrowserWindowTesting* GetBrowserWindowTesting(); virtual StatusBubble* GetStatusBubble(); - virtual void SelectedTabToolbarSizeChanged(bool is_animating); + virtual void ToolbarSizeChanged(bool is_animating); virtual void UpdateTitleBar(); virtual void ShelfVisibilityChanged(); virtual void UpdateDevTools(); diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.mm b/chrome/browser/ui/cocoa/browser_window_cocoa.mm index 43fdb57..eb02ac8 100644 --- a/chrome/browser/ui/cocoa/browser_window_cocoa.mm +++ b/chrome/browser/ui/cocoa/browser_window_cocoa.mm @@ -139,7 +139,7 @@ StatusBubble* BrowserWindowCocoa::GetStatusBubble() { return [controller_ statusBubble]; } -void BrowserWindowCocoa::SelectedTabToolbarSizeChanged(bool is_animating) { +void BrowserWindowCocoa::ToolbarSizeChanged(bool is_animating) { // According to beng, this is an ugly method that comes from the days when the // download shelf was a ChromeView attached to the TabContents, and as its // size changed via animation it notified through TCD/etc to the browser view diff --git a/chrome/browser/ui/cocoa/constrained_html_delegate_mac.mm b/chrome/browser/ui/cocoa/constrained_html_delegate_mac.mm index 74778a0..d3cabc5 100644 --- a/chrome/browser/ui/cocoa/constrained_html_delegate_mac.mm +++ b/chrome/browser/ui/cocoa/constrained_html_delegate_mac.mm @@ -40,7 +40,6 @@ class ConstrainedHtmlDelegateMac : // HtmlDialogTabContentsDelegate --------------------------------------------- void MoveContents(TabContents* source, const gfx::Rect& pos) {} - void ToolbarSizeChanged(TabContents* source, bool is_animating) {} void HandleKeyboardEvent(const NativeWebKeyboardEvent& event) {} void set_window(ConstrainedWindow* window) { diff --git a/chrome/browser/ui/cocoa/html_dialog_window_controller.mm b/chrome/browser/ui/cocoa/html_dialog_window_controller.mm index 3b2d026..67c9537 100644 --- a/chrome/browser/ui/cocoa/html_dialog_window_controller.mm +++ b/chrome/browser/ui/cocoa/html_dialog_window_controller.mm @@ -48,7 +48,6 @@ public: // HtmlDialogTabContentsDelegate declarations. virtual void MoveContents(TabContents* source, const gfx::Rect& pos); - virtual void ToolbarSizeChanged(TabContents* source, bool is_animating); virtual void HandleKeyboardEvent(const NativeWebKeyboardEvent& event); private: @@ -171,11 +170,6 @@ void HtmlDialogWindowDelegateBridge::MoveContents(TabContents* source, // TODO(akalin): Actually set the window bounds. } -void HtmlDialogWindowDelegateBridge::ToolbarSizeChanged( - TabContents* source, bool is_animating) { - // TODO(akalin): Figure out what to do here. -} - // A simplified version of BrowserWindowCocoa::HandleKeyboardEvent(). // We don't handle global keyboard shortcuts here, but that's fine since // they're all browser-specific. (This may change in the future.) |