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

Unified Diff: content/browser/accessibility/accessibility_tree_formatter.cc

Issue 811213002: Revert of Add DumpAccessibilityEvents test framework. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@dump_acc_events_1
Patch Set: Created 6 years 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: content/browser/accessibility/accessibility_tree_formatter.cc
diff --git a/content/browser/accessibility/accessibility_tree_formatter.cc b/content/browser/accessibility/accessibility_tree_formatter.cc
index 39956c3291ba4d8cd9f6c5ab098fe02cd110b99c..d4c189fb6f1b3a6ff51b5e42ce63775e6e9f33c8 100644
--- a/content/browser/accessibility/accessibility_tree_formatter.cc
+++ b/content/browser/accessibility/accessibility_tree_formatter.cc
@@ -24,8 +24,7 @@
AccessibilityTreeFormatter::AccessibilityTreeFormatter(
BrowserAccessibility* root)
- : root_(root),
- show_ids_(false) {
+ : root_(root) {
Initialize();
}
@@ -76,12 +75,12 @@
void AccessibilityTreeFormatter::RecursiveFormatAccessibilityTree(
const base::DictionaryValue& dict, base::string16* contents, int depth) {
- base::string16 indent = base::string16(depth * kIndentSpaces, ' ');
- base::string16 line = indent + ToString(dict);
+ base::string16 line =
+ ToString(dict, base::string16(depth * kIndentSpaces, ' '));
if (line.find(base::ASCIIToUTF16(kSkipString)) != base::string16::npos)
return;
- *contents += line + base::ASCIIToUTF16("\n");
+ *contents += line;
const base::ListValue* children;
dict.GetList(kChildrenDictAttr, &children);
const base::DictionaryValue* child_dict;
@@ -98,10 +97,12 @@
}
base::string16 AccessibilityTreeFormatter::ToString(
- const base::DictionaryValue& node) {
+ const base::DictionaryValue& node,
+ const base::string16& indent) {
int id_value;
node.GetInteger("id", &id_value);
- return base::IntToString16(id_value);
+ return indent + base::IntToString16(id_value) +
+ base::ASCIIToUTF16("\n");
}
void AccessibilityTreeFormatter::Initialize() {}
@@ -139,14 +140,11 @@
filters_ = filters;
}
-// static
bool AccessibilityTreeFormatter::MatchesFilters(
- const std::vector<Filter>& filters,
- const base::string16& text,
- bool default_result) {
- std::vector<Filter>::const_iterator iter = filters.begin();
+ const base::string16& text, bool default_result) const {
+ std::vector<Filter>::const_iterator iter = filters_.begin();
bool allow = default_result;
- for (iter = filters.begin(); iter != filters.end(); ++iter) {
+ for (iter = filters_.begin(); iter != filters_.end(); ++iter) {
if (MatchPattern(text, iter->match_str)) {
if (iter->type == Filter::ALLOW_EMPTY)
allow = true;
@@ -157,11 +155,6 @@
}
}
return allow;
-}
-
-bool AccessibilityTreeFormatter::MatchesFilters(
- const base::string16& text, bool default_result) const {
- return MatchesFilters(filters_, text, default_result);
}
base::string16 AccessibilityTreeFormatter::FormatCoordinates(

Powered by Google App Engine
This is Rietveld 408576698