[WIP] Change DocumentMarkerController::add*Marker() methods to take an EphemeralRange

third_party/WebKit/Source/core/testing/Internals.cpp

Index: third_party/WebKit/Source/core/testing/Internals.cpp
diff --git a/third_party/WebKit/Source/core/testing/Internals.cpp b/third_party/WebKit/Source/core/testing/Internals.cpp
index 5498e7b808a6f232a95883d466b0a8c92d9a92cb..63c848e830df4630d4648e991aaf4588d7c02503 100644
--- a/third_party/WebKit/Source/core/testing/Internals.cpp
+++ b/third_party/WebKit/Source/core/testing/Internals.cpp
@@ -975,11 +975,9 @@ void Internals::setMarker(Document* document,
   document->updateStyleAndLayoutIgnorePendingStylesheets();
 
   if (type == DocumentMarker::Grammar) {

[Xiaocheng, 2017/03/22 21:21:01]

nit: no need to have braces now.

-    document->markers().addGrammarMarker(range->startPosition(),
-                                         range->endPosition());
+    document->markers().addGrammarMarker(EphemeralRange(range));
   } else {
-    document->markers().addSpellingMarker(range->startPosition(),
-                                          range->endPosition());
+    document->markers().addSpellingMarker(EphemeralRange(range));
   }
 }
 
@@ -1102,8 +1100,7 @@ void Internals::addCompositionMarker(const Range* range,
       parseColor(backgroundColorValue, backgroundColor, exceptionState,
                  "Invalid background color.")) {
     range->ownerDocument().markers().addCompositionMarker(
-        range->startPosition(), range->endPosition(), underlineColor, thick,
-        backgroundColor);
+        EphemeralRange(range), underlineColor, thick, backgroundColor);
   }
 }