Drop AccessibilityTreeFormatterStub

No value to produce a bare accessible tree on platforms having no accessibility support. Native blink accessibility tree can be used instead

Bug: 1070310
Change-Id: I8aa537a6440e6da34f70defb1b8d72b11a1874e1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2149500Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Alexander Surkkov <asurkov@igalia.com>
Cr-Commit-Position: refs/heads/master@{#759317}

content/browser/accessibility/accessibility_event_recorder.cc

@@ -35,7 +35,6 @@ AccessibilityEventRecorder::GetTestPasses() {
 #else   // defined(OS_ANDROID)
   return {
       {"blink", &AccessibilityEventRecorder::Create},
-      {"native", &AccessibilityEventRecorder::Create},
   };
 #endif  // defined(OS_ANDROID)
 }

content/browser/accessibility/accessibility_tree_formatter_stub.cc

@@ -4,34 +4,14 @@
 
 #include "base/strings/string_number_conversions.h"
 #include "content/browser/accessibility/accessibility_tree_formatter_blink.h"
-#include "content/browser/accessibility/accessibility_tree_formatter_browser.h"
 
 namespace content {
 
-class AccessibilityTreeFormatterStub
-    : public AccessibilityTreeFormatterBrowser {
- public:
-  explicit AccessibilityTreeFormatterStub();
-  ~AccessibilityTreeFormatterStub() override;
-
- private:
-  base::FilePath::StringType GetExpectedFileSuffix() override;
-  const std::string GetAllowEmptyString() override;
-  const std::string GetAllowString() override;
-  const std::string GetDenyString() override;
-  const std::string GetDenyNodeString() override;
-  void AddProperties(const BrowserAccessibility& node,
-                     base::DictionaryValue* dict) override;
-  base::string16 ProcessTreeForOutput(
-      const base::DictionaryValue& node,
-      base::DictionaryValue* filtered_dict_result = nullptr) override;
-};
-
 #if !defined(PLATFORM_HAS_NATIVE_ACCESSIBILITY_IMPL)
 // static
 std::unique_ptr<AccessibilityTreeFormatter>
 AccessibilityTreeFormatter::Create() {
-  return std::make_unique<AccessibilityTreeFormatterStub>();
+  return AccessibilityTreeFormatterBlink::CreateBlink();
 }
 
 // static
@@ -39,49 +19,8 @@ std::vector<AccessibilityTreeFormatter::TestPass>
 AccessibilityTreeFormatter::GetTestPasses() {
   return {
       {"blink", &AccessibilityTreeFormatterBlink::CreateBlink},
-      {"native", &AccessibilityTreeFormatter::Create},
   };
 }
 #endif
 
-AccessibilityTreeFormatterStub::AccessibilityTreeFormatterStub()
-    : AccessibilityTreeFormatterBrowser() {}
-
-AccessibilityTreeFormatterStub::~AccessibilityTreeFormatterStub() {}
-
-void AccessibilityTreeFormatterStub::AddProperties(
-    const BrowserAccessibility& node,
-    base::DictionaryValue* dict) {
-  dict->SetInteger("id", node.GetId());
-}
-
-base::string16 AccessibilityTreeFormatterStub::ProcessTreeForOutput(
-    const base::DictionaryValue& node,
-    base::DictionaryValue* filtered_dict_result) {
-  int id_value;
-  node.GetInteger("id", &id_value);
-  return base::NumberToString16(id_value);
-}
-
-base::FilePath::StringType
-AccessibilityTreeFormatterStub::GetExpectedFileSuffix() {
-  return base::FilePath::StringType();
-}
-
-const std::string AccessibilityTreeFormatterStub::GetAllowEmptyString() {
-  return std::string();
-}
-
-const std::string AccessibilityTreeFormatterStub::GetAllowString() {
-  return std::string();
-}
-
-const std::string AccessibilityTreeFormatterStub::GetDenyString() {
-  return std::string();
-}
-
-const std::string AccessibilityTreeFormatterStub::GetDenyNodeString() {
-  return std::string();
-}
-
 }  // namespace content