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

1 files changed, 3 insertions, 14 deletions

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];
 }