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

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

Issue 10837065: Revert 149510 - Allow filters in accessibility tests to specify which attributes to check. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 5 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: content/browser/accessibility/dump_accessibility_tree_helper.cc
===================================================================
--- content/browser/accessibility/dump_accessibility_tree_helper.cc (revision 149548)
+++ content/browser/accessibility/dump_accessibility_tree_helper.cc (working copy)
@@ -4,21 +4,12 @@
#include "content/browser/accessibility/dump_accessibility_tree_helper.h"
-#include "base/logging.h"
#include "base/memory/scoped_ptr.h"
-#include "base/string_util.h"
namespace {
const int kIndentSpaces = 4;
}
-DumpAccessibilityTreeHelper::DumpAccessibilityTreeHelper() {
- Initialize();
-}
-
-DumpAccessibilityTreeHelper::~DumpAccessibilityTreeHelper() {
-}
-
void DumpAccessibilityTreeHelper::DumpAccessibilityTree(
BrowserAccessibility* node, string16* contents) {
RecursiveDumpAccessibilityTree(node, contents, 0);
@@ -37,41 +28,3 @@
indent + kIndentSpaces);
}
}
-
-void DumpAccessibilityTreeHelper::SetFilters(
- const std::set<string16>& allow_filters,
- const std::set<string16>& deny_filters) {
- allow_filters_ = allow_filters;
- deny_filters_ = deny_filters;
-}
-
-bool DumpAccessibilityTreeHelper::MatchesFilters(
- const string16& text, bool default_result) {
- std::set<string16>::const_iterator iter = allow_filters_.begin();
- for (iter = allow_filters_.begin(); iter != allow_filters_.end(); ++iter) {
- if (MatchPattern(text, *iter))
- return true;
- }
- for (iter = deny_filters_.begin(); iter != deny_filters_.end(); ++iter) {
- if (MatchPattern(text, *iter))
- return false;
- }
- return default_result;
-}
-
-void DumpAccessibilityTreeHelper::StartLine() {
- line_.clear();
-}
-
-void DumpAccessibilityTreeHelper::Add(
- bool include_by_default, const string16& attr) {
- if (!MatchesFilters(attr, include_by_default))
- return;
- if (!line_.empty())
- line_ += ASCIIToUTF16(" ");
- line_ += attr;
-}
-
-string16 DumpAccessibilityTreeHelper::FinishLine() {
- return line_;
-}

Powered by Google App Engine
This is Rietveld 408576698