Fix an issue with not clearing |selected_text_| when text selection range is empty (Mac)

content/browser/renderer_host/text_input_manager.cc

Index: content/browser/renderer_host/text_input_manager.cc
diff --git a/content/browser/renderer_host/text_input_manager.cc b/content/browser/renderer_host/text_input_manager.cc
index 16897f88438e018a4733f8098d6514b92e29cb35..dc7e1b6b6bfa6c213cb13ebc711a58a91cecca23 100644
--- a/content/browser/renderer_host/text_input_manager.cc
+++ b/content/browser/renderer_host/text_input_manager.cc
@@ -329,14 +329,13 @@ TextInputManager::TextSelection::~TextSelection() {}
 bool TextInputManager::TextSelection::GetSelectedText(
     base::string16* selected_text) const {
   if (text.empty() || range.is_empty())
-    return false;
+    return true;
 
   size_t pos = range.GetMin() - offset;
   size_t n = range.length();
   if (pos + n > text.length()) {

[EhsanK, 2016/12/22 20:06:22]

Before, we used to DCHECK on this so return false is not necessary.

     LOG(WARNING) << "The text can not fully cover range (selection's end point "
                     "exceeds text length).";
-    return false;
   }
 
   if (pos >= text.length()) {