Remove Range.compareNode() as it is not in the spec

Source/core/dom/Range.h

Index: Source/core/dom/Range.h
diff --git a/Source/core/dom/Range.h b/Source/core/dom/Range.h
index 4530f52e6006462f9530906fbdb6ff83397d87f4..084c5c4880dfc0a29a8cb5ab6716a4e0ffaac6e7 100644
--- a/Source/core/dom/Range.h
+++ b/Source/core/dom/Range.h
@@ -81,9 +81,9 @@ public:
     bool isPointInRange(Node* refNode, int offset, ExceptionState&);
     short comparePoint(Node* refNode, int offset, ExceptionState&) const;
     enum CompareResults { NODE_BEFORE, NODE_AFTER, NODE_BEFORE_AND_AFTER, NODE_INSIDE };
-    CompareResults compareNode(Node* refNode, ExceptionState&) const;
     enum CompareHow { START_TO_START, START_TO_END, END_TO_END, END_TO_START };
     short compareBoundaryPoints(unsigned how, const Range* sourceRange, ExceptionState&) const;
+    static bool isNodeFullySelected(Node*, const Range* selectedRange);

[philipj_slow, 2015/06/08 11:50:31]

Having seen what the resulting code is like, I would change my suggestion to a
non-static |bool isNodeFullyContained(Node&)|, since you have a range at both
call sites and the Node* is never null, right? This should make the call sites
slightly more readable, I think.

     static short compareBoundaryPoints(Node* containerA, int offsetA, Node* containerB, int offsetB, ExceptionState&);
     static short compareBoundaryPoints(const RangeBoundaryPoint& boundaryA, const RangeBoundaryPoint& boundaryB, ExceptionState&);
     bool boundaryPointsValid() const;