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

Unified Diff: ui/gfx/render_text_harfbuzz.h

Issue 2639493002: MacViews: Enable word lookup for selectable views::Labels and multi-line text. (Closed)
Patch Set: Address nits. Created 3 years, 11 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
« no previous file with comments | « ui/gfx/render_text.cc ('k') | ui/gfx/render_text_harfbuzz.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/gfx/render_text_harfbuzz.h
diff --git a/ui/gfx/render_text_harfbuzz.h b/ui/gfx/render_text_harfbuzz.h
index fd345d29217332148f6511e973637c33c8b8d9a1..4f52cdeefe95e0ad1c9ce2a4c5088bcddddea321 100644
--- a/ui/gfx/render_text_harfbuzz.h
+++ b/ui/gfx/render_text_harfbuzz.h
@@ -197,20 +197,6 @@ class GFX_EXPORT RenderTextHarfBuzz : public RenderText {
// |runs_| vector if argument exceeds the text length or width.
size_t GetRunContainingCaret(const SelectionModel& caret);
- // Returns the line index for the given argument. |text_y| is relative to
- // the text bounds. Returns -1 if |text_y| is above the text and
- // lines().size() if |text_y| is below it.
- int GetLineContainingYCoord(float text_y);
-
- // Returns the line segment index for the |line|, |text_x| pair. |text_x| is
- // relative to text in the given line. Returns -1 if |text_x| is to the left
- // of text in the line and |line|.segments.size() if it's to the right.
- // |offset_relative_segment| will contain the offset of |text_x| relative to
- // the start of the segment it is contained in.
- int GetLineSegmentContainingXCoord(const internal::Line& line,
- float text_x,
- float* offset_relative_segment);
-
// Given a |run|, returns the SelectionModel that contains the logical first
// or last caret position inside (not at a boundary of) the run.
// The returned value represents a cursor/caret position without a selection.
« no previous file with comments | « ui/gfx/render_text.cc ('k') | ui/gfx/render_text_harfbuzz.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698