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

Unified Diff: ui/accessibility/ax_enums.idl

Issue 2301833005: Get rid of AX_LINE_BREAKS attribute to improve performance. (Closed)
Patch Set: Fixed line length issue. Created 4 years, 3 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: ui/accessibility/ax_enums.idl
diff --git a/ui/accessibility/ax_enums.idl b/ui/accessibility/ax_enums.idl
index e706e1350d64de4e1ed038327b2c2f13eb40c81c..f0e2b1ec339b8920a03634e73d323e685a3a803e 100644
--- a/ui/accessibility/ax_enums.idl
+++ b/ui/accessibility/ax_enums.idl
@@ -390,9 +390,6 @@
flowto_ids,
labelledby_ids,
- // For static text. Character indices where line breaks occur.
- line_breaks,
-
// For static text. These int lists must be the same size; they represent
// the start and end character offset of each marker. Examples of markers
// include spelling and grammar errors, and find-in-page matches.
@@ -416,6 +413,11 @@
// within the object's bounds, the second offset is the right coordinate
// of the second character, and so on.
character_offsets,
+
+ // Used for caching. Do not read directly. Use |AXNode::GetLineStartOffsets|
+ // For all objects with text content. A list of the start offset of each
+ // line of text inside this object.
+ line_starts,
dmazzoni 2016/09/13 17:29:38 Maybe it'd be more clear to call it cached_line_st
// For inline text. These int lists must be the same size; they represent
// the start and end character offset of each word within this text.

Powered by Google App Engine
This is Rietveld 408576698