Commit 4f9fb41c authored by Yoichi Osato's avatar Yoichi Osato Committed by Commit Bot

Remove unused DocumentMarkerController::HasMarkers()

HasMarkers and its caller InlineTextBoxPainter::PaintsMarkerHighlights is
not called anywhere. Remove them.

Bug: 868229
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I2ad088d8637d461178e7e3af3017c923b6c1f4c3
Reviewed-on: https://chromium-review.googlesource.com/1248282Reviewed-by: default avatarYoshifumi Inoue <yosin@chromium.org>
Reviewed-by: default avatarXianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#594736}
parent 38545f4c
...@@ -93,7 +93,6 @@ class CORE_EXPORT DocumentMarkerController final ...@@ -93,7 +93,6 @@ class CORE_EXPORT DocumentMarkerController final
unsigned start_offset, unsigned start_offset,
unsigned end_offset, unsigned end_offset,
bool); bool);
bool HasMarkers(const Node* node) const { return markers_.Contains(node); }
// TODO(rlanday): can these methods for retrieving markers be consolidated // TODO(rlanday): can these methods for retrieving markers be consolidated
// without hurting efficiency? // without hurting efficiency?
......
...@@ -88,13 +88,6 @@ LayoutObject& InlineTextBoxPainter::InlineLayoutObject() const { ...@@ -88,13 +88,6 @@ LayoutObject& InlineTextBoxPainter::InlineLayoutObject() const {
inline_text_box_.GetLineLayoutItem()); inline_text_box_.GetLineLayoutItem());
} }
bool InlineTextBoxPainter::PaintsMarkerHighlights(
const LayoutObject& layout_object) {
return layout_object.GetNode() &&
layout_object.GetDocument().Markers().HasMarkers(
layout_object.GetNode());
}
static void ComputeOriginAndWidthForBox(const InlineTextBox& box, static void ComputeOriginAndWidthForBox(const InlineTextBox& box,
LayoutPoint& local_origin, LayoutPoint& local_origin,
LayoutUnit& width) { LayoutUnit& width) {
......
...@@ -65,8 +65,6 @@ class InlineTextBoxPainter { ...@@ -65,8 +65,6 @@ class InlineTextBoxPainter {
const ComputedStyle&, const ComputedStyle&,
const Font&); const Font&);
static bool PaintsMarkerHighlights(const LayoutObject&);
private: private:
enum class PaintOptions { kNormal, kCombinedText }; enum class PaintOptions { kNormal, kCombinedText };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment