[Mac] AcceptInput() doesn't really need focus (as when called via "Go" button).

http://crbug.com/15811
TEST=Browse to www.google.com.  Focus is in document.  Click Go.  No more crash.

Review URL: http://codereview.chromium.org/149487

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20495 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 5 deletions

diff --git a/chrome/browser/autocomplete/autocomplete_edit_view_mac.mm b/chrome/browser/autocomplete/autocomplete_edit_view_mac.mm
index 71c11b0..9fe6ab9 100644
--- a/chrome/browser/autocomplete/autocomplete_edit_view_mac.mm
+++ b/chrome/browser/autocomplete/autocomplete_edit_view_mac.mm
@@ -290,9 +290,11 @@ NSRange AutocompleteEditViewMac::GetSelectedRange() const {
 }
 
 void AutocompleteEditViewMac::SetSelectedRange(const NSRange range) {
+  // This can be called when we don't have focus.  For instance, when
+  // the user clicks the "Go" button.
   if (model_->has_focus()) {
-    // TODO(shess): This should not be necessary.  Try to convert to
-    // DCHECK(IsFirstResponder()).
+    // TODO(shess): If |model_| thinks we have focus, this should not
+    // be necessary.  Try to convert to DCHECK(IsFirstResponder()).
     FocusLocation();
 
     // TODO(shess): What if it didn't get first responder, and there is
@@ -543,9 +545,6 @@ void AutocompleteEditViewMac::OnDidResignKey() {
 
 void AutocompleteEditViewMac::AcceptInput(
     WindowOpenDisposition disposition, bool for_drop) {
-  // We should only arrive here when the field is focussed.
-  DCHECK([field_ currentEditor]);
-
   model_->AcceptInput(disposition, for_drop);
 }