Track text selection on the browser side (Mac)

content/browser/renderer_host/text_input_manager.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/text_input_manager.h"
 
 #include "base/strings/string16.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 #include "content/browser/renderer_host/render_widget_host_view_base.h"
 #include "content/common/view_messages.h"
@@ skipping 67 matching lines @@
 }
 
 const TextInputManager::TextSelection* TextInputManager::GetTextSelection(
     RenderWidgetHostViewBase* view) const {
   DCHECK(!view || IsRegistered(view));
   if (!view)
     view = active_view_;
   return !!view ? &text_selection_map_.at(view) : nullptr;
 }
 
+bool TextInputManager::GetSelectedText(RenderWidgetHostViewBase* view,
+                                       base::string16* selected_text) const {
+  const TextSelection* selection = GetTextSelection(view);
+  if (!selection)
+    return false;
+
+  if (selection->text.empty() || selection->range.is_empty())
+    return false;
+
+  selected_text->clear();
+  size_t pos = selection->range.GetMin() - selection->offset;
+  size_t n = selection->range.length();
+  DCHECK(pos + n <= selection->text.length())

[Avi (use Gerrit), 2016/08/12 15:53:10]

DCHECK_LE

[EhsanK, 2016/08/16 13:36:45]

Done.

+      << "The text can not fully cover range.";
+  if (pos >= selection->text.length()) {

[erikchen, 2016/08/12 19:33:43]

Is it ever possible for an instance of TextSelection to contain invalid
parameters? 

If so, we shouldn't have DCHECKs.
If not, we shouldn't have a conditional.

The code pattern:
"""
if (...) {
  NOTREACHED()
}
"""
is a red flag.

[EhsanK, 2016/08/16 13:36:45]

I am not sure but the selection information are coming from the renderer side,
so they could be wrong maybe?

I cannot DCHECK and not return early since it might throw exception in
text.substr() later on. So, if DCHECK is incorrect here, I think I should just
early return if the range values are invalid. WDYT?

[erikchen, 2016/08/16 16:36:22]

Yes, early return is appropriate. Short of memory corruption or someone breaking
this class in the future, nothing should make a DCHECK be hit. You might be want
a LOG(WARNING) instead? 

[EhsanK, 2016/08/17 06:20:50]

I think the DCHECK above should also change then. I did replace both checks with
early return and logs.

+    NOTREACHED() << "The text can not cover range.";
+    return false;
+  }
+  selected_text->append(selection->text.substr(pos, n));
+  return true;
+}
+
 void TextInputManager::UpdateTextInputState(
     RenderWidgetHostViewBase* view,
     const TextInputState& text_input_state) {
   DCHECK(IsRegistered(view));
 
   if (text_input_state.type == ui::TEXT_INPUT_TYPE_NONE &&
       active_view_ != view) {
     // We reached here because an IPC is received to reset the TextInputState
     // for |view|. But |view| != |active_view_|, which suggests that at least
     // one other view has become active and we have received the corresponding
@@ skipping 200 matching lines @@
 
 TextInputManager::TextSelection::TextSelection()
     : offset(0), range(gfx::Range::InvalidRange()), text(base::string16()) {}
 
 TextInputManager::TextSelection::TextSelection(const TextSelection& other) =
     default;
 
 TextInputManager::TextSelection::~TextSelection() {}
 
 }  // namespace content