Mac:Dont always enable findbar buttons when restoring focus with search string

chrome/browser/ui/cocoa/find_bar/find_bar_cocoa_controller.mm

Index: chrome/browser/ui/cocoa/find_bar/find_bar_cocoa_controller.mm
diff --git a/chrome/browser/ui/cocoa/find_bar/find_bar_cocoa_controller.mm b/chrome/browser/ui/cocoa/find_bar/find_bar_cocoa_controller.mm
index 70013f041d886eb5aa94239a9046d75455f96ab7..22c5209a0cc98ef37db59589643dc6dab1c1b086 100644
--- a/chrome/browser/ui/cocoa/find_bar/find_bar_cocoa_controller.mm
+++ b/chrome/browser/ui/cocoa/find_bar/find_bar_cocoa_controller.mm
@@ -362,11 +362,6 @@ const int kMaxCharacters = 4000;
 
 - (void)setFocusAndSelection {
   [[findText_ window] makeFirstResponder:findText_];
-
-  // Enable the buttons if the find text is non-empty.

[rohitrao (ping after 24h), 2017/06/26 13:20:15]

Here's a case that appears to have regressed with this CL.  Can you also
reproduce this locally?

1) Open two tabs.
2) Search for something (with matches) in the first tab.
3) Close the find bar.  Switch to the second tab.  Switch back.
4) Press Cmd-F to open the find bar again.

At this point, I see the find bar, populated with a search string, but the
buttons are disabled.  I think I'd expect the buttons to be enabled at this
point?  Otherwise there's no way to trigger a search except by using the
keyboard shortcuts or deleting and retyping the text.

-  BOOL buttonsEnabled = ([[findText_ stringValue] length] > 0) ? YES : NO;

[rohitrao (ping after 24h), 2017/06/26 13:20:15]

So it seems that this function is called from at least one place where it's
important to set the button state correctly, and doing nothing doesn't work.

Maybe we need to continue setting the state, but use different logic for
|buttonsEnabled|?  We could look at whether or not there is an active find
operation, and if so, use the current results to set the enabled states.

[carloschilazo, 2017/06/28 22:50:22]

Thanks for looking into it Rohit! I agree, we need to then add logic for the
enablement instead of directly removing it. I'll make some time this weekend to
look into it.

-  [previousButton_ setEnabled:buttonsEnabled];
-  [nextButton_ setEnabled:buttonsEnabled];
 }
 
 - (void)restoreSavedFocus {