From fb6de399a7eead8338e4f49920feed47d122fec3 Mon Sep 17 00:00:00 2001 From: "oshima@chromium.org" Date: Fri, 8 Jan 2010 18:35:54 +0000 Subject: Simplified the api by replacing them with single Show(false) call and cleaned up linux/mac code. BUG=22036 TEST=None Review URL: http://codereview.chromium.org/523125 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35803 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/cocoa/find_bar_bridge.mm | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) (limited to 'chrome/browser/cocoa/find_bar_bridge.mm') diff --git a/chrome/browser/cocoa/find_bar_bridge.mm b/chrome/browser/cocoa/find_bar_bridge.mm index c12a830..9b73254 100644 --- a/chrome/browser/cocoa/find_bar_bridge.mm +++ b/chrome/browser/cocoa/find_bar_bridge.mm @@ -16,8 +16,8 @@ FindBarBridge::FindBarBridge() FindBarBridge::~FindBarBridge() { } -void FindBarBridge::Show() { - [cocoa_controller_ showFindBar:YES]; // with animation. +void FindBarBridge::Show(bool animate) { + [cocoa_controller_ showFindBar:(animate ? YES : NO)]; } void FindBarBridge::Hide(bool animate) { @@ -52,24 +52,13 @@ bool FindBarBridge::IsFindBarVisible() { void FindBarBridge::MoveWindowIfNecessary(const gfx::Rect& selection_rect, bool no_redraw) { // http://crbug.com/11084 + // http://crbug.com/22036 } void FindBarBridge::StopAnimation() { [cocoa_controller_ stopAnimation]; } -gfx::Rect FindBarBridge::GetDialogPosition(gfx::Rect avoid_overlapping_rect) { - // http://crbug.com/22036 - return gfx::Rect(); -} - -void FindBarBridge::SetDialogPosition(const gfx::Rect& new_pos, - bool no_redraw) { - // TODO(rohitrao): Do something useful here. For now, just show the findbar. - // http://crbug.com/22036 - [cocoa_controller_ showFindBar:NO]; // Do not animate. -} - void FindBarBridge::RestoreSavedFocus() { [cocoa_controller_ restoreSavedFocus]; } -- cgit v1.1