Revert of Made surrounding text work for last word in a document

third_party/WebKit/Source/core/editing/SurroundingText.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 58 matching lines @@
   DCHECK_EQ(RootEditableElementOf(start_position),
             RootEditableElementOf(end_position));
   Element* const root_editable = RootEditableElementOf(start_position);
   Element* const root_element =
       root_editable ? root_editable : document->documentElement();
 
   CharacterIterator forward_iterator(
       end_position,
       Position::LastPositionInNode(root_element).ParentAnchoredEquivalent(),
       TextIteratorBehavior::Builder().SetStopsOnFormControls(true).Build());
+  // FIXME: why do we stop going trough the text if we were not able to select
+  // something on the right?
   if (!forward_iterator.AtEnd())
     forward_iterator.Advance(max_length - half_max_length);
 
+  EphemeralRange forward_range = forward_iterator.Range();
+  if (forward_range.IsNull() ||
+      !Range::Create(*document, end_position, forward_range.StartPosition())
+           ->GetText()
+           .length())
+    return;
+
   // Same as with the forward range but with the backward range. The range
   // starts at the document's or input element's start and ends at the selection
   // start and will be updated.
   BackwardsCharacterIterator backwards_iterator(
       Position::FirstPositionInNode(root_element).ParentAnchoredEquivalent(),
       start_position,
       TextIteratorBehavior::Builder().SetStopsOnFormControls(true).Build());
   if (!backwards_iterator.AtEnd())
     backwards_iterator.Advance(half_max_length);
 
   const TextIteratorBehavior behavior =
       TextIteratorBehavior::NoTrailingSpaceRangeLengthBehavior();
   start_offset_in_content_ = TextIterator::RangeLength(
       backwards_iterator.EndPosition(), start_position, behavior);
   end_offset_in_content_ = TextIterator::RangeLength(
       backwards_iterator.EndPosition(), end_position, behavior);
   content_range_ = Range::Create(*document, backwards_iterator.EndPosition(),
-                                 forward_iterator.StartPosition());
+                                 forward_range.StartPosition());
   DCHECK(content_range_);
 }
 
 String SurroundingText::Content() const {
   if (content_range_) {
     // SurroundingText is created with clean layout and must not be stored
     // through DOM or style changes, so layout must still be clean here.
     DCHECK(!content_range_->OwnerDocument().NeedsLayoutTreeUpdate());
     return content_range_->GetText();
   }
   return String();
 }
 
 unsigned SurroundingText::StartOffsetInContent() const {
   return start_offset_in_content_;
 }
 
 unsigned SurroundingText::EndOffsetInContent() const {
   return end_offset_in_content_;
 }
 
 }  // namespace blink