Adds propagation and handling of render-side focus events, for the benefit of...

webkit/glue/webaccessibilitymanager_impl.cc

 // Copyright (c) 2006-2008 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 "config.h"
 
 #include "AXObjectCache.h"
 #include "Document.h"
 #include "Frame.h"
 #include "RefPtr.h"
@@ skipping 16 matching lines @@
   RefPtr<GlueAccessibilityObject> acc_obj_root_;
 };
 
 /*static*/
 WebAccessibilityManager* WebAccessibilityManager::Create() {
   return new WebAccessibilityManagerImpl();
 }
 
 // class WebAccessibilityManagerImpl
 WebAccessibilityManagerImpl::WebAccessibilityManagerImpl()
-    : root_(new GlueAccessibilityObjectRoot) {
+    : root_(new GlueAccessibilityObjectRoot),
+      acc_obj_id_(0) {
+}
+
+WebAccessibilityManagerImpl::~WebAccessibilityManagerImpl() {
+  int_to_glue_acc_obj_map_.clear();
+  acc_obj_to_int_map_.clear();
 }
 
 bool WebAccessibilityManagerImpl::GetAccObjInfo(WebView* view,
   const WebAccessibility::InParams& in_params,
   WebAccessibility::OutParams* out_params) {
   if (!root_->acc_obj_root_ && !InitAccObjRoot(view)) {
     // Failure in retrieving or initializing the root.
     return false;
   }
 
-  // Find GlueAccessibilityObject requested by [in_params.object_id].
-  IntToAccObjMap::iterator it =
-      int_to_acc_obj_map_.find(in_params.object_id);
-  if (it == int_to_acc_obj_map_.end() || !it->second) {
+  // Function input parameters.
+  int object_id = in_params.object_id;
+  int child_id = in_params.child_id;
+
+  if (!in_params.direct_descendant) {
+    // Object is not a direct child, re-map the input parameters accordingly.
+    // The object to be retrived is referred to by the |in_params.child_id|, as

[jcampan, 2009/05/19 21:50:04]

typo: retrived -> retrieved

+    // a result of e.g. a focus event. The local |child_id| is set to 0, to
+    // indicate that any function call should refer to the object itself.
+    object_id = in_params.child_id;
+    child_id = 0;
+  }
+
+  // Find GlueAccessibilityObject requested by |object_id|.
+  IntToGlueAccObjMap::iterator it =
+      int_to_glue_acc_obj_map_.find(object_id);
+  if (it == int_to_glue_acc_obj_map_.end() || !it->second) {
     // Map did not contain a valid instance of the data requested.
     return false;
   }
   RefPtr<GlueAccessibilityObject> active_acc_obj = it->second;
 
-  // Function input parameters.
-  int child_id = in_params.child_id;
-
   // Temp paramters for holding output information.
   RefPtr<GlueAccessibilityObject> out_acc_obj = NULL;
   WebCore::String out_string;
 
   switch (in_params.function_id) {
     case WebAccessibility::FUNCTION_DODEFAULTACTION :
       if (!active_acc_obj->DoDefaultAction(child_id))
         return false;
       break;
     case WebAccessibility::FUNCTION_HITTEST :
@@ skipping 79 matching lines @@
     default:
       // Non-supported function id.
       return false;
   }
 
   // Output and hashmap assignments, as appropriate.
   if (!out_string.isEmpty())
     out_params->output_string = StringToString16(out_string);
 
   if (out_acc_obj) {
-    AccObjToIntMap::iterator it = acc_obj_to_int_map_.find(out_acc_obj.get());
+    AccObjToIntMap::iterator it =
+        acc_obj_to_int_map_.find(out_acc_obj->accessibilityObject());
 
     if (it != acc_obj_to_int_map_.end()) {
       // Data already present in map, return previously assigned id.
       out_params->object_id = it->second;
       out_params->output_long1 = -1;
     } else {
       // Insert new GlueAccessibilityObject in hashmaps.
-      int_to_acc_obj_map_[acc_obj_id_] = out_acc_obj.get();
-      acc_obj_to_int_map_[out_acc_obj.get()] = acc_obj_id_;
+      int_to_glue_acc_obj_map_[acc_obj_id_] = out_acc_obj.get();
+      acc_obj_to_int_map_[out_acc_obj->accessibilityObject()] = acc_obj_id_;
       out_params->object_id = acc_obj_id_++;
       out_params->output_long1 = -1;
     }
   }
   // TODO(klink): Handle simple objects returned.
   return true;
 }
 
 bool WebAccessibilityManagerImpl::InitAccObjRoot(WebView* view) {
-  // Root id is always 0.
-  acc_obj_id_ = 0;
-
   // Enable accessibility and retrieve Document.
   WebCore::AXObjectCache::enableAccessibility();
   WebFrameImpl* main_frame_impl =
       static_cast<WebFrameImpl*>(view->GetMainFrame());
   if (!main_frame_impl || !main_frame_impl->frame())
     return false;
 
   WebCore::Document* doc = main_frame_impl->frame()->document();
 
   if (!doc || !doc->renderer())
     return false;
 
   if (!root_->acc_obj_root_) {
     // Either we've never had a wrapper for this frame's top-level Document,
     // the Document renderer was destroyed and its wrapper was detached, or
     // the previous Document is in the page cache, and the current document
     // needs to be wrapped.
     root_->acc_obj_root_ = GlueAccessibilityObject::CreateInstance(doc->
         axObjectCache()->getOrCreate(doc->renderer()));
   }
   // Insert root in hashmaps.
-  int_to_acc_obj_map_[acc_obj_id_] = root_->acc_obj_root_.get();
-  acc_obj_to_int_map_[root_->acc_obj_root_.get()] = acc_obj_id_++;
+  int_to_glue_acc_obj_map_[acc_obj_id_] = root_->acc_obj_root_.get();
+  acc_obj_to_int_map_[root_->acc_obj_root_->accessibilityObject()] =
+      acc_obj_id_++;
 
   return true;
 }
 
 bool WebAccessibilityManagerImpl::ClearAccObjMap(int acc_obj_id,
                                                  bool clear_all) {
   if (clear_all) {
     // Clear maps and invalidate root.
-    int_to_acc_obj_map_.clear();
+    int_to_glue_acc_obj_map_.clear();
     acc_obj_to_int_map_.clear();
     root_->acc_obj_root_ = 0;
     return true;
   }
 
-  IntToAccObjMap::iterator it = int_to_acc_obj_map_.find(acc_obj_id);
+  IntToGlueAccObjMap::iterator it = int_to_glue_acc_obj_map_.find(acc_obj_id);
 
-  if (it == int_to_acc_obj_map_.end()) {
+  if (it == int_to_glue_acc_obj_map_.end()) {
     // Element not found.
     return false;
   }
 
   if (it->second) {
     // Erase element from reverse hashmap.
-    AccObjToIntMap::iterator it2 = acc_obj_to_int_map_.find(it->second);
+    AccObjToIntMap::iterator it2 =
+        acc_obj_to_int_map_.find(it->second->accessibilityObject());
 
     if (it2 != acc_obj_to_int_map_.end())
       acc_obj_to_int_map_.erase(it2);
   }
-  int_to_acc_obj_map_.erase(it);
+  int_to_glue_acc_obj_map_.erase(it);
 
   if (acc_obj_id == 0) {
     // Invalidate root.
     root_->acc_obj_root_ = 0;
   }
   return true;
 }
 
+int WebAccessibilityManagerImpl::FocusAccObj(
+    WebCore::AccessibilityObject* acc_obj) {
+  if (!acc_obj) {
+    // Return with failure.
+    return -1;
+  }
+
+  AccObjToIntMap::iterator it = acc_obj_to_int_map_.find(acc_obj);
+
+  if (it != acc_obj_to_int_map_.end()) {
+    return it->second;
+  } else {

[darin (slow to review), 2009/05/18 22:43:21]

nit: no need for else after return.  would be nice to reduce indentation of
remainder of function.

+    // Insert new accessibility object in hashmaps and return its newly
+    // assigned accessibility object id.
+    int_to_glue_acc_obj_map_[acc_obj_id_] =
+        GlueAccessibilityObject::CreateInstance(acc_obj);
+    acc_obj_to_int_map_[acc_obj] = acc_obj_id_;
+
+    return acc_obj_id_++;
+  }
+}
+
 }  // namespace webkit_glue