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

Unified Diff: third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp

Issue 2906953002: [DMC #26] Add CompositionMarker::Thickness enum (Closed)
Patch Set: Created 3 years, 7 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/markers/DocumentMarkerController.cpp
diff --git a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
index 623ef0d5873c054f0dc9e0904dcaf305719a90ec..388549dd06204b77d43809b117141750ada3779a 100644
--- a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
+++ b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
@@ -171,18 +171,20 @@ void DocumentMarkerController::AddTextMatchMarker(
// throttling algorithm. crbug.com/6819.
}
-void DocumentMarkerController::AddCompositionMarker(const EphemeralRange& range,
- Color underline_color,
- bool thick,
- Color background_color) {
+void DocumentMarkerController::AddCompositionMarker(
+ const EphemeralRange& range,
+ Color underline_color,
+ CompositionMarker::Thickness thickness,
+ Color background_color) {
DCHECK(!document_->NeedsLayoutTreeUpdate());
for (TextIterator marked_text(range.StartPosition(), range.EndPosition());
!marked_text.AtEnd(); marked_text.Advance()) {
- AddMarker(marked_text.CurrentContainer(),
- new CompositionMarker(marked_text.StartOffsetInCurrentContainer(),
- marked_text.EndOffsetInCurrentContainer(),
- underline_color, thick, background_color));
+ AddMarker(
+ marked_text.CurrentContainer(),
+ new CompositionMarker(marked_text.StartOffsetInCurrentContainer(),
+ marked_text.EndOffsetInCurrentContainer(),
+ underline_color, thickness, background_color));
}
}

Powered by Google App Engine
This is Rietveld 408576698