Ignore selections where the user dragged off the popup.

http://crbug.com/13107
TEST=Start typing in omnibox.  Click in the popup and drag mouse off before releasing.  Shouldn't crash.

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

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

1 files changed, 11 insertions, 3 deletions

diff --git a/chrome/browser/autocomplete/autocomplete_popup_view_mac.mm b/chrome/browser/autocomplete/autocomplete_popup_view_mac.mm
index 68b48c6..ca4c8ec 100644
--- a/chrome/browser/autocomplete/autocomplete_popup_view_mac.mm
+++ b/chrome/browser/autocomplete/autocomplete_popup_view_mac.mm
@@ -319,9 +319,17 @@ AutocompletePopupModel* AutocompletePopupViewMac::GetModel() {
 }
 
 void AutocompletePopupViewMac::AcceptInput() {
-  AutocompleteMatrix* matrix = [popup_ contentView];
-  model_->SetSelectedLine([matrix selectedRow], false);
-  edit_view_->AcceptInput(CURRENT_TAB, false);
+  const NSInteger selectedRow = [[popup_ contentView] selectedRow];
+
+  // -1 means no cells were selected.  This can happen if the user
+  // clicked and then dragged their mouse off the popup before
+  // releasing, so reset the selection and ignore the event.
+  if (selectedRow == -1) {
+    PaintUpdatesNow();
+  } else {
+    model_->SetSelectedLine(selectedRow, false);
+    edit_view_->AcceptInput(CURRENT_TAB, false);
+  }
 }
 
 @implementation AutocompleteButtonCell