Add a big grab bag of missing web accessibility functionality...

content/browser/accessibility/browser_accessibility.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/accessibility/browser_accessibility.h"
 
 #include "base/logging.h"
 #include "base/string_number_conversions.h"
+#include "base/string_util.h"
 #include "content/browser/accessibility/browser_accessibility_manager.h"
+#include "content/common/view_messages.h"
 
+typedef WebAccessibility::BoolAttribute BoolAttribute;
+typedef WebAccessibility::FloatAttribute FloatAttribute;
 typedef WebAccessibility::IntAttribute IntAttribute;
 typedef WebAccessibility::StringAttribute StringAttribute;
 
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
 // There's no OS-specific implementation of BrowserAccessibilityManager
 // on Unix, so just instantiate the base class.
 // static
 BrowserAccessibility* BrowserAccessibility::Create() {
   return new BrowserAccessibility();
 }
@@ skipping 27 matching lines @@
     BrowserAccessibilityManager* manager,
     BrowserAccessibility* parent,
     int32 child_id,
     int32 index_in_parent,
     const webkit_glue::WebAccessibility& src) {
   manager_ = manager;
   parent_ = parent;
   child_id_ = child_id;
   index_in_parent_ = index_in_parent;
 
-  renderer_id_ = src.id;
+  // Update all of the rest of the attributes.
   name_ = src.name;
   value_ = src.value;
+  role_ = src.role;
+  state_ = src.state;
+  renderer_id_ = src.id;
   string_attributes_ = src.string_attributes;
   int_attributes_ = src.int_attributes;
+  float_attributes_ = src.float_attributes;
+  bool_attributes_ = src.bool_attributes;
   html_attributes_ = src.html_attributes;
   location_ = src.location;
-  role_ = src.role;
-  state_ = src.state;
   indirect_child_ids_ = src.indirect_child_ids;
   line_breaks_ = src.line_breaks;
   cell_ids_ = src.cell_ids;
+  unique_cell_ids_ = src.unique_cell_ids;
 
   Initialize();
 }
 
 void BrowserAccessibility::Initialize() {
   instance_active_ = true;
 }
 
 void BrowserAccessibility::AddChild(BrowserAccessibility* child) {
   children_.push_back(child);
@@ skipping 79 matching lines @@
   if (recursive || ref_count_ == 1) {
     for (std::vector<BrowserAccessibility*>::iterator iter = children_.begin();
          iter != children_.end();
          ++iter) {
       (*iter)->InternalReleaseReference(true);
     }
   }
 
   ref_count_--;
   if (ref_count_ == 0) {
+    // Allow the object to fire a TEXT_REMOVED notification.
+    name_.clear();
+    value_.clear();
+    SendNodeUpdateEvents();
+
+    manager_->NotifyAccessibilityEvent(
+        ViewHostMsg_AccessibilityNotification_Type::
+            NOTIFICATION_TYPE_OBJECT_HIDE,
+            this);

[David Tseng, 2011/08/26 16:14:37]

I think you want to align this with
"ViewHostMsg_AccessibilityNotification_Type::
"

[dmazzoni, 2011/08/29 18:08:51]

You're right, thanks!

+
     instance_active_ = false;
     children_.clear();
     manager_->Remove(child_id_, renderer_id_);
     NativeReleaseReference();
   }
 }
 
 void BrowserAccessibility::NativeReleaseReference() {
   delete this;
 }
@@ skipping 15 matching lines @@
     IntAttribute attribute, int* value) {
   std::map<IntAttribute, int32>::iterator iter =
       int_attributes_.find(attribute);
   if (iter != int_attributes_.end()) {
     *value = iter->second;
     return true;
   }
 
   return false;
 }
+
+bool BrowserAccessibility::GetFloatAttribute(
+    FloatAttribute attribute, float* value) {
+  std::map<FloatAttribute, float>::iterator iter =
+      float_attributes_.find(attribute);
+  if (iter != float_attributes_.end()) {
+    *value = iter->second;
+    return true;
+  }
+
+  return false;
+}
+
+bool BrowserAccessibility::GetBoolAttribute(
+    BoolAttribute attribute, bool* value) {
+  std::map<BoolAttribute, bool>::iterator iter =
+      bool_attributes_.find(attribute);
+  if (iter != bool_attributes_.end()) {
+    *value = iter->second;
+    return true;
+  }
+
+  return false;
+}
+
+bool BrowserAccessibility::GetHtmlAttribute(
+    const char* html_attr, string16* value) {
+  for (size_t i = 0; i < html_attributes_.size(); i++) {
+    string16& attr = html_attributes_[i].first;
+    if (LowerCaseEqualsASCII(attr, html_attr)) {
+      *value = html_attributes_[i].second;
+      return true;
+    }
+  }
+
+  return false;
+}
+
+bool BrowserAccessibility::IsEditableText() {

[David Tseng, 2011/08/26 16:14:37]

const method?

[dmazzoni, 2011/08/29 18:08:51]

Done. I made some other methods const as well.

+  return (role_ == WebAccessibility::ROLE_TEXT_FIELD ||
+          role_ == WebAccessibility::ROLE_TEXTAREA);
+}