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

Side by Side Diff: ui/accessibility/ax_tree_combiner.cc

Issue 2301833005: Get rid of AX_LINE_BREAKS attribute to improve performance. (Closed)
Patch Set: Re-worded comment. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ui/accessibility/ax_tree.h" 5 #include "ui/accessibility/ax_tree.h"
6 #include "ui/accessibility/ax_tree_combiner.h" 6 #include "ui/accessibility/ax_tree_combiner.h"
7 #include "ui/gfx/geometry/rect_f.h" 7 #include "ui/gfx/geometry/rect_f.h"
8 8
9 namespace ui { 9 namespace ui {
10 namespace { 10 namespace {
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 case AX_ATTR_INDIRECT_CHILD_IDS: 75 case AX_ATTR_INDIRECT_CHILD_IDS:
76 case AX_ATTR_LABELLEDBY_IDS: 76 case AX_ATTR_LABELLEDBY_IDS:
77 case AX_ATTR_UNIQUE_CELL_IDS: 77 case AX_ATTR_UNIQUE_CELL_IDS:
78 return true; 78 return true;
79 79
80 // Note: all of the attributes are included here explicitly, 80 // Note: all of the attributes are included here explicitly,
81 // rather than using "default:", so that it's a compiler error to 81 // rather than using "default:", so that it's a compiler error to
82 // add a new attribute without explicitly considering whether it's 82 // add a new attribute without explicitly considering whether it's
83 // a node id attribute or not. 83 // a node id attribute or not.
84 case AX_INT_LIST_ATTRIBUTE_NONE: 84 case AX_INT_LIST_ATTRIBUTE_NONE:
85 case AX_ATTR_LINE_BREAKS:
86 case AX_ATTR_MARKER_TYPES: 85 case AX_ATTR_MARKER_TYPES:
87 case AX_ATTR_MARKER_STARTS: 86 case AX_ATTR_MARKER_STARTS:
88 case AX_ATTR_MARKER_ENDS: 87 case AX_ATTR_MARKER_ENDS:
89 case AX_ATTR_CHARACTER_OFFSETS: 88 case AX_ATTR_CHARACTER_OFFSETS:
89 case AX_ATTR_CACHED_LINE_STARTS:
90 case AX_ATTR_WORD_STARTS: 90 case AX_ATTR_WORD_STARTS:
91 case AX_ATTR_WORD_ENDS: 91 case AX_ATTR_WORD_ENDS:
92 return false; 92 return false;
93 } 93 }
94 94
95 NOTREACHED(); 95 NOTREACHED();
96 return false; 96 return false;
97 } 97 }
98 98
99 } // namespace 99 } // namespace
(...skipping 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
235 // Recurse into the child tree now, if any. 235 // Recurse into the child tree now, if any.
236 if (child_tree) 236 if (child_tree)
237 ProcessTree(child_tree); 237 ProcessTree(child_tree);
238 } 238 }
239 239
240 // Reset the transform. 240 // Reset the transform.
241 transform_ = old_transform; 241 transform_ = old_transform;
242 } 242 }
243 243
244 } // namespace ui 244 } // namespace ui
OLDNEW
« ui/accessibility/ax_node.cc ('K') | « ui/accessibility/ax_node_data.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698