WIP: prefer using EphemeralRange for SpellChecker.

Source/web/WebLocalFrameImpl.cpp

 /*
  * Copyright (C) 2009 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 248 matching lines @@
     TrackExceptionState exceptionState;
     range->selectNodeContents(document->body(), exceptionState);
 
     if (!exceptionState.hadException()) {
         // The text iterator will walk nodes giving us text. This is similar to
         // the plainText() function in core/editing/TextIterator.h, but we implement the maximum
         // size and also copy the results directly into a wstring, avoiding the
         // string conversion.
         for (TextIterator it(range->startPosition(), range->endPosition()); !it.atEnd(); it.advance()) {
             it.text().appendTextToStringBuilder(output, 0, maxChars - output.length());
-            if (output.length() >= maxChars)
+            if (output.length() >= maxChars) {
+                range->dispose();

[yosin_UTC9, 2015/09/10 01:58:39]

Please introduce |ScopedRangeObject|, or better name, to avoid explicit call
of |Range::dispose()|.

[sof, 2015/09/10 06:07:49]

We have ScopedDisposal<T>, which is arguably more general. Switched.

                 return; // Filled up the buffer.
+            }
         }
     }
+    range->dispose();
 
     // The separator between frames when the frames are converted to plain text.
     const LChar frameSeparator[] = { '\n', '\n' };
     const size_t frameSeparatorLength = WTF_ARRAY_LENGTH(frameSeparator);
 
     // Recursively walk the children.
     const FrameTree& frameTree = frame->tree();
     for (Frame* curChild = frameTree.firstChild(); curChild; curChild = curChild->tree().nextSibling()) {
         if (!curChild->isLocalFrame())
             continue;
@@ skipping 1910 matching lines @@
 }
 
 WebSandboxFlags WebLocalFrameImpl::effectiveSandboxFlags() const
 {
     if (!frame())
         return WebSandboxFlags::None;
     return static_cast<WebSandboxFlags>(frame()->loader().effectiveSandboxFlags());
 }
 
 } // namespace blink