Avoid more temporary ranges in connection with various TextIterators.

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 55 matching lines @@
 
     // The forward range starts at the selection end and ends at the document's
     // end. It will then be updated to only contain the text in the text in the
     // right range around the selection.
     RefPtrWillBeRawPtr<Range> forwardRange = Range::create(*document, endPosition, lastPositionInNode(document->documentElement()).parentAnchoredEquivalent());
     CharacterIterator forwardIterator(forwardRange.get(), TextIteratorStopsOnFormControls);
     // FIXME: why do we stop going trough the text if we were not able to select something on the right?
     if (!forwardIterator.atEnd())
         forwardIterator.advance(maxLength - halfMaxLength);
 
-    forwardRange = forwardIterator.range();
+    forwardRange = forwardIterator.createRange();
     if (!forwardRange || !Range::create(*document, endPosition, forwardRange->startPosition())->text().length()) {
         ASSERT(forwardRange);
         return;
     }
 
     // Same as with the forward range but with the backward range. The range
     // starts at the document's start and ends at the selection start and will
     // be updated.
     RefPtrWillBeRawPtr<Range> backwardsRange = Range::create(*document, firstPositionInNode(document->documentElement()).parentAnchoredEquivalent(), startPosition);
     BackwardsCharacterIterator backwardsIterator(backwardsRange.get(), TextIteratorStopsOnFormControls);
     if (!backwardsIterator.atEnd())
         backwardsIterator.advance(halfMaxLength);
 
-    backwardsRange = backwardsIterator.range();
-    if (!backwardsRange) {
-        ASSERT(backwardsRange);
-        return;
-    }
-
-    m_startOffsetInContent = Range::create(*document, backwardsRange->endPosition(), startPosition)->text().length();
-    m_endOffsetInContent = Range::create(*document, backwardsRange->endPosition(), endPosition)->text().length();
-    m_contentRange = Range::create(*document, backwardsRange->endPosition(), forwardRange->startPosition());
+    m_startOffsetInContent = Range::create(*document, backwardsIterator.endPosition(), startPosition)->text().length();
+    m_endOffsetInContent = Range::create(*document, backwardsIterator.endPosition(), endPosition)->text().length();
+    m_contentRange = Range::create(*document, backwardsIterator.endPosition(), forwardRange->startPosition());
     ASSERT(m_contentRange);
 }
 
 PassRefPtrWillBeRawPtr<Range> SurroundingText::rangeFromContentOffsets(unsigned startOffsetInContent, unsigned endOffsetInContent)
 {
     if (startOffsetInContent >= endOffsetInContent || endOffsetInContent > content().length())
         return nullptr;
 
     CharacterIterator iterator(m_contentRange.get());
 
@@ skipping 22 matching lines @@
 {
     return m_startOffsetInContent;
 }
 
 unsigned SurroundingText::endOffsetInContent() const
 {
     return m_endOffsetInContent;
 }
 
 } // namespace blink