Convert WebRange to be a simple pair of numbers.

third_party/WebKit/Source/web/WebRange.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 12 matching lines @@
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "public/web/WebRange.h"
 
-#include "bindings/core/v8/ExceptionState.h"
-#include "bindings/core/v8/ExceptionStatePlaceholder.h"
 #include "core/dom/Document.h"
-#include "core/dom/Element.h"
 #include "core/dom/Range.h"
-#include "core/dom/shadow/ShadowRoot.h"
 #include "core/editing/FrameSelection.h"
 #include "core/editing/PlainTextRange.h"
-#include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
-#include "public/platform/WebString.h"
-#include "public/web/WebExceptionCode.h"
-#include "public/web/WebNode.h"
-#include "web/WebLocalFrameImpl.h"
-#include "wtf/PassRefPtr.h"
 
 namespace blink {
 
-void WebRange::reset()
+WebRange::WebRange(int start, int length)
+    : m_start(start)
+    , m_end(start + length)
 {
-    m_private.reset();
+    DCHECK(start != 1 && length != 0) << "These values are reserved to indicate that the range is null";

[ojan, 2016/08/09 18:15:05]

s/1/-1/ ?

[dglazkov, 2016/08/10 05:12:13]

LOL. YES.

 }
 
-void WebRange::assign(const WebRange& other)
+WebRange::WebRange(Range* range)
 {
-    m_private = other.m_private;
+    if (!range)
+        return;
+
+    m_start = range->startOffset();
+    m_end = range->endOffset();
 }
 
-int WebRange::startOffset() const
+Range* WebRange::createRange(LocalFrame* frame) const
 {
-    return m_private->startOffset();
-}
+    Element* selectionRoot = frame->selection().rootEditableElement();
+    ContainerNode* scope = selectionRoot ? selectionRoot : frame->document()->documentElement();
 
-int WebRange::endOffset() const
-{
-    return m_private->endOffset();
-}
-
-WebString WebRange::toPlainText() const
-{
-    return m_private->text();
-}
-
-// static
-WebRange WebRange::fromDocumentRange(WebLocalFrame* frame, int start, int length)
-{
-    LocalFrame* webFrame = toWebLocalFrameImpl(frame)->frame();
-    Element* selectionRoot = webFrame->selection().rootEditableElement();
-    ContainerNode* scope = selectionRoot ? selectionRoot : webFrame->document()->documentElement();
-
-    // TODO(dglazkov): The use of updateStyleAndLayoutIgnorePendingStylesheets needs to be audited.
-    // see http://crbug.com/590369 for more details.
-    scope->document().updateStyleAndLayoutIgnorePendingStylesheets();
-
-    return createRange(PlainTextRange(start, start + length).createRange(*scope));
-}
-
-WebRange::WebRange(Range*range)
-    : m_private(range)
-{
-}
-
-WebRange::operator Range*() const
-{
-    return m_private.get();
+    return blink::createRange(PlainTextRange(m_start, m_end).createRange(*scope));
 }
 
 } // namespace blink