Move ExpandToParagraphBoundary to VisibleUnits

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

 /*
  * Copyright (C) 2006, 2007 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 15 matching lines @@
 
 #include "core/editing/spellcheck/TextCheckingParagraph.h"
 
 #include "core/dom/Range.h"
 #include "core/editing/VisiblePosition.h"
 #include "core/editing/VisibleUnits.h"
 #include "core/editing/iterators/CharacterIterator.h"
 
 namespace blink {
 
-static EphemeralRange ExpandToParagraphBoundary(const EphemeralRange& range) {
-  const VisiblePosition& start = CreateVisiblePosition(range.StartPosition());
-  DCHECK(start.IsNotNull()) << range.StartPosition();
-  const Position& paragraph_start = StartOfParagraph(start).DeepEquivalent();
-  DCHECK(paragraph_start.IsNotNull()) << range.StartPosition();
-
-  const VisiblePosition& end = CreateVisiblePosition(range.EndPosition());
-  DCHECK(end.IsNotNull()) << range.EndPosition();
-  const Position& paragraph_end = EndOfParagraph(end).DeepEquivalent();
-  DCHECK(paragraph_end.IsNotNull()) << range.EndPosition();
-
-  // TODO(xiaochengh): There are some cases (crbug.com/640112) where we get
-  // |paragraphStart > paragraphEnd|, which is the reason we cannot directly
-  // return |EphemeralRange(paragraphStart, paragraphEnd)|. This is not
-  // desired, though. We should do more investigation to ensure that why
-  // |paragraphStart <= paragraphEnd| is violated.
-  const Position& result_start =
-      paragraph_start.IsNotNull() && paragraph_start <= range.StartPosition()
-          ? paragraph_start
-          : range.StartPosition();
-  const Position& result_end =
-      paragraph_end.IsNotNull() && paragraph_end >= range.EndPosition()
-          ? paragraph_end
-          : range.EndPosition();
-  return EphemeralRange(result_start, result_end);
-}
-
 TextCheckingParagraph::TextCheckingParagraph(
     const EphemeralRange& checking_range)
     : checking_range_(checking_range),
       checking_start_(-1),
       checking_end_(-1),
       checking_length_(-1) {}
 
 TextCheckingParagraph::TextCheckingParagraph(
     const EphemeralRange& checking_range,
     const EphemeralRange& paragraph_range)
@@ skipping 105 matching lines @@
 
 int TextCheckingParagraph::CheckingLength() const {
   DCHECK(checking_range_.IsNotNull());
   if (-1 == checking_length_)
     checking_length_ = TextIterator::RangeLength(
         CheckingRange().StartPosition(), CheckingRange().EndPosition());
   return checking_length_;
 }
 
 }  // namespace blink