OmniboxView: Attempt to fix a crash in GetSelectedText().

chrome/browser/ui/views/omnibox/omnibox_view_win.cc

Index: chrome/browser/ui/views/omnibox/omnibox_view_win.cc
diff --git a/chrome/browser/ui/views/omnibox/omnibox_view_win.cc b/chrome/browser/ui/views/omnibox/omnibox_view_win.cc
index 806cd51e3d1e9bffe39848c063d83bac3d50d247..2c9c8d20b0cccbdfa2e9a600230d30f339a2ef8c 100644
--- a/chrome/browser/ui/views/omnibox/omnibox_view_win.cc
+++ b/chrome/browser/ui/views/omnibox/omnibox_view_win.cc
@@ -2109,6 +2109,8 @@ string16 OmniboxViewWin::GetSelectedText() const {
   // Figure out the length of the selection.
   CHARRANGE sel;
   GetSel(sel);
+  if (sel.cpMin == sel.cpMax)
+    return string16();

[Peter Kasting, 2011/10/27 18:17:53]

Nit: Might want to add an EOL comment: "// GetSelText() will crash if we pass it
NULL."

[James Hawkins, 2011/10/27 18:26:42]

Done.

 
   // Grab the selected text.
   string16 str;