[SelectMisspellingAsync #1] Move SelectMisspellingAsync() from ContextMenuClient.cpp to SpellChecker

third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 115 matching lines @@
 
   const Position& next_position =
       NextPositionOf(caret_position).DeepEquivalent();
 
   return EphemeralRange(
       previous_position.IsNull() ? caret_position.DeepEquivalent()
                                  : previous_position,
       next_position.IsNull() ? caret_position.DeepEquivalent() : next_position);
 }
 
+static bool IsWhiteSpaceOrPunctuation(UChar c) {
+  return IsSpaceOrNewline(c) || WTF::Unicode::IsPunct(c);
+}
+
 }  // namespace
 
 SpellChecker* SpellChecker::Create(LocalFrame& frame) {
   return new SpellChecker(frame);
 }
 
 static SpellCheckerClient& GetEmptySpellCheckerClient() {
   DEFINE_STATIC_LOCAL(EmptySpellCheckerClient, client, ());
   return client;
 }
@@ skipping 751 matching lines @@
         continue;
 
       return std::make_pair(&node, &ToSpellCheckMarker(*marker));
     }
   }
 
   // No marker found
   return Optional<std::pair<Node*, SpellCheckMarker*>>();
 }
 
+String SpellChecker::SelectMisspellingAsync(String& description) {
+  VisibleSelection selection =

[yosin_UTC9, 2017/07/20 00:56:32]

nit: s/VisbileSelection/const VisbileSelection/

+      GetFrame().Selection().ComputeVisibleSelectionInDOMTree();
+  if (selection.IsNone())
+    return String();
+
+  // Caret and range selections always return valid normalized ranges.
+  const EphemeralRange& selection_range =
+      selection.ToNormalizedEphemeralRange();
+
+  Node* const selection_start_container =
+      selection_range.StartPosition().ComputeContainerNode();
+  Node* const selection_end_container =
+      selection_range.EndPosition().ComputeContainerNode();
+
+  // We don't currently support the case where a misspelling spans multiple
+  // nodes. See crbug.com/720065
+  if (selection_start_container != selection_end_container)
+    return String();
+
+  const unsigned selection_start_offset =
+      selection_range.StartPosition().ComputeOffsetInContainerNode();
+  const unsigned selection_end_offset =
+      selection_range.EndPosition().ComputeOffsetInContainerNode();
+
+  const DocumentMarkerVector& markers_in_node =
+      GetFrame().GetDocument()->Markers().MarkersFor(
+          selection_start_container, DocumentMarker::MisspellingMarkers());
+
+  const auto marker_it =
+      std::find_if(markers_in_node.begin(), markers_in_node.end(),
+                   [=](const DocumentMarker* marker) {
+                     return marker->StartOffset() < selection_end_offset &&
+                            marker->EndOffset() > selection_start_offset;
+                   });
+  if (marker_it == markers_in_node.end())
+    return String();
+
+  const SpellCheckMarker* const found_marker = ToSpellCheckMarker(*marker_it);
+  description = found_marker->Description();
+
+  Range* const marker_range =

[yosin_UTC9, 2017/07/20 00:56:32]

We should avoid to use temporary Range object for Oilpan friendly.
We can use PlainText() with EphemeralRange here.

String Range::GetText() const {
  DCHECK(!owner_document_->NeedsLayoutTreeUpdate());
  return PlainText(EphemeralRange(this),
                   TextIteratorBehavior::Builder()
                       .SetEmitsObjectReplacementCharacter(true)
                       .Build());
}

[Xiaocheng, 2017/07/20 21:22:17]

Shouldn't we do this change in a separate CL?

Doing moving and refactoring in the same CL makes it harder to track code
changes.

+      Range::Create(*GetFrame().GetDocument(), selection_start_container,
+                    found_marker->StartOffset(), selection_start_container,
+                    found_marker->EndOffset());
+
+  if (marker_range->GetText().StripWhiteSpace(&IsWhiteSpaceOrPunctuation) !=
+      CreateRange(selection_range)
+          ->GetText()
+          .StripWhiteSpace(&IsWhiteSpaceOrPunctuation))
+    return String();
+
+  return marker_range->GetText();

[yosin_UTC9, 2017/07/20 00:56:31]

Please hold restul of Range::GetText() to avoid calling PlainText() twice.

[Xiaocheng, 2017/07/20 21:22:17]

Ditto.

+}
+
 void SpellChecker::ReplaceMisspelledRange(const String& text) {
   const Optional<std::pair<Node*, SpellCheckMarker*>>& node_and_marker =
       GetSpellCheckMarkerTouchingSelection();
   if (!node_and_marker)
     return;
 
   Node* const container_node = node_and_marker.value().first;
   const SpellCheckMarker* const marker = node_and_marker.value().second;
 
   GetFrame().Selection().SetSelection(
@@ skipping 384 matching lines @@
       if (!input.IsFocusedElementInDocument())
         return false;
     }
   }
   HTMLElement* element =
       Traversal<HTMLElement>::FirstAncestorOrSelf(*position.AnchorNode());
   return element && element->IsSpellCheckingEnabled();
 }
 
 }  // namespace blink