From 5efe92bfc5573933ce8e6ee01ba64352f9ae1d1d Mon Sep 17 00:00:00 2001 From: "pinkerton@chromium.org" Date: Tue, 1 Sep 2009 20:12:31 +0000 Subject: Remove un-needed method (updateLoadingAnimations). BUG=8281 TEST=none. Review URL: http://codereview.chromium.org/181037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@25077 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/cocoa/browser_window_cocoa.mm | 2 +- chrome/browser/cocoa/browser_window_controller.h | 3 --- chrome/browser/cocoa/browser_window_controller.mm | 12 ------------ 3 files changed, 1 insertion(+), 16 deletions(-) diff --git a/chrome/browser/cocoa/browser_window_cocoa.mm b/chrome/browser/cocoa/browser_window_cocoa.mm index 3f58e8c..377309c 100644 --- a/chrome/browser/cocoa/browser_window_cocoa.mm +++ b/chrome/browser/cocoa/browser_window_cocoa.mm @@ -113,7 +113,7 @@ void BrowserWindowCocoa::FocusDevTools() { } void BrowserWindowCocoa::UpdateLoadingAnimations(bool should_animate) { - [controller_ updateLoadingAnimations:should_animate ? YES : NO]; + // Do nothing on Mac. } void BrowserWindowCocoa::SetStarredState(bool is_starred) { diff --git a/chrome/browser/cocoa/browser_window_controller.h b/chrome/browser/cocoa/browser_window_controller.h index eb34d94..e7d9471 100644 --- a/chrome/browser/cocoa/browser_window_controller.h +++ b/chrome/browser/cocoa/browser_window_controller.h @@ -105,9 +105,6 @@ class TabStripModelObserverBridge; // Called to tell the selected tab to update its loading state. - (void)setIsLoading:(BOOL)isLoading; -// Called to start/stop the loading animations. -- (void)updateLoadingAnimations:(BOOL)animate; - // Make the location bar the first responder, if possible. - (void)focusLocationBar; diff --git a/chrome/browser/cocoa/browser_window_controller.mm b/chrome/browser/cocoa/browser_window_controller.mm index 33e5b6b..99d4329 100644 --- a/chrome/browser/cocoa/browser_window_controller.mm +++ b/chrome/browser/cocoa/browser_window_controller.mm @@ -666,18 +666,6 @@ willPositionSheet:(NSWindow*)sheet [toolbarController_ setIsLoading:isLoading]; } -// Called to start/stop the loading animations. -- (void)updateLoadingAnimations:(BOOL)animate { - if (animate) { - // TODO(pinkerton): determine what throbber animation is necessary and - // start a timer to periodically update. Windows tells the tab strip to - // do this. It uses a single timer to coalesce the multiple things that - // could be updating. http://crbug.com/8281 - } else { - // TODO(pinkerton): stop the timer. - } -} - // Make the location bar the first responder, if possible. - (void)focusLocationBar { [toolbarController_ focusLocationBar]; -- cgit v1.1