Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(429)

Unified Diff: third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp

Issue 2771603002: [WIP] Change DocumentMarkerController::add*Marker() methods to take an EphemeralRange (Closed)
Patch Set: Rebase Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
diff --git a/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp b/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
index 0439b7ca87ecea7c313fdd45b6764b05849c8ba2..e68a6e20f988c436a186190c6ff4a5cc7b6c2fab 100644
--- a/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
+++ b/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
@@ -302,8 +302,7 @@ void SpellChecker::advanceToNextMisspelling(bool startBeforeSelection) {
.build());
frame().selection().revealSelection();
spellCheckerClient().updateSpellingUIWithMisspelledWord(misspelledWord);
- frame().document()->markers().addSpellingMarker(
- misspellingRange.startPosition(), misspellingRange.endPosition());
+ frame().document()->markers().addSpellingMarker(misspellingRange);
}
}
@@ -543,13 +542,10 @@ static void addMarker(Document* document,
DCHECK(type == DocumentMarker::Grammar || type == DocumentMarker::Spelling)
<< type;
- if (type == DocumentMarker::Grammar) {
- document->markers().addGrammarMarker(
- rangeToMark.startPosition(), rangeToMark.endPosition(), description);
- } else {
- document->markers().addSpellingMarker(
- rangeToMark.startPosition(), rangeToMark.endPosition(), description);
- }
+ if (type == DocumentMarker::Grammar)
+ document->markers().addGrammarMarker(rangeToMark, description);
+ else
+ document->markers().addSpellingMarker(rangeToMark, description);
}
void SpellChecker::markAndReplaceFor(

Powered by Google App Engine
This is Rietveld 408576698