1 files changed, 254 insertions, 255 deletions

diff --git a/chrome/views/accessibility/view_accessibility.cc b/chrome/views/accessibility/view_accessibility.cc
index 5991218..4ba7d54 100644
--- a/chrome/views/accessibility/view_accessibility.cc
+++ b/chrome/views/accessibility/view_accessibility.cc
@@ -4,7 +4,7 @@
 
 #include "chrome/views/accessibility/view_accessibility.h"
 
-#include "chrome/views/accessibility/accessible_wrapper.h"
+#include "chrome/views/accessibility/view_accessibility_wrapper.h"
 #include "chrome/views/widget/widget.h"
 
 HRESULT ViewAccessibility::Initialize(views::View* view) {
@@ -16,106 +16,108 @@ HRESULT ViewAccessibility::Initialize(views::View* view) {
   return S_OK;
 }
 
-STDMETHODIMP ViewAccessibility::get_accChildCount(LONG* child_count) {
-  if (!child_count || !view_) {
+// TODO(klink): Handle case where child View is not contained by parent.
+STDMETHODIMP ViewAccessibility::accHitTest(LONG x_left, LONG y_top,
+                                           VARIANT* child) {
+  if (!child) {
     return E_INVALIDARG;
   }
 
-  *child_count = view_->GetChildViewCount();
-  return S_OK;
-}
-
-STDMETHODIMP ViewAccessibility::get_accChild(VARIANT var_child,
-                                             IDispatch** disp_child) {
-  if (var_child.vt != VT_I4 || !disp_child) {
-    return E_INVALIDARG;
-  }
+  gfx::Point pt(x_left, y_top);
+  views::View::ConvertPointToView(NULL, view_, &pt);
 
-  // If var_child is the parent, remain with the same IDispatch.
-  if (var_child.lVal == CHILDID_SELF) {
-    return S_OK;
+  if (!view_->HitTest(pt)) {
+    // If containing parent is not hit, return with failure.
+    child->vt = VT_EMPTY;
+    return S_FALSE;
   }
 
+  int child_count = view_->GetChildViewCount();
+  bool child_hit = false;
   views::View* child_view = NULL;
-  bool get_iaccessible = false;
-
-  // Check to see if child is out-of-bounds.
-  if (IsValidChild((var_child.lVal - 1), view_)) {
-    child_view = view_->GetChildViewAt(var_child.lVal - 1);
-  } else {
-    // Child is located elsewhere in the hierarchy, get ID and adjust for MSAA.
-    child_view = view_->GetViewByID(static_cast<int>(var_child.lVal));
-    get_iaccessible = true;
+  for (int child_id = 0; child_id < child_count; ++child_id) {
+    // Search for hit within any of the children.
+    child_view = view_->GetChildViewAt(child_id);
+    views::View::ConvertPointToView(view_, child_view, &pt);
+    if (child_view->HitTest(pt)) {
+      // Store child_id (adjusted with +1 to convert to MSAA indexing).
+      child->lVal = child_id + 1;
+      child_hit = true;
+      break;
+    }
+    // Convert point back to parent view to test next child.
+    views::View::ConvertPointToView(child_view, view_, &pt);
   }
 
-  if (!child_view) {
-    // No child found.
-    *disp_child = NULL;
-    return E_FAIL;
-  }
+  child->vt = VT_I4;
 
-  if (get_iaccessible || child_view->GetChildViewCount() != 0) {
-    // Retrieve the IUnknown interface for the requested child view, and
-    // assign the IDispatch returned.
-    if ((GetAccessibleWrapper(child_view))->
-        GetInstance(IID_IAccessible,
-                    reinterpret_cast<void**>(disp_child)) == S_OK) {
-      // Increment the reference count for the retrieved interface.
-      (*disp_child)->AddRef();
-      return S_OK;
-    } else {
-      // No interface, return failure.
-      return E_NOINTERFACE;
-    }
+  if (!child_hit) {
+    // No child hit, return parent id.
+    child->lVal = CHILDID_SELF;
   } else {
-    // When at a leaf, children are handled by the parent object.
-    *disp_child = NULL;
-    return S_FALSE;
+    if (child_view == NULL) {
+      return E_FAIL;
+    } else if (child_view->GetChildViewCount() != 0) {
+      // Retrieve IDispatch for child, if it is not a leaf.
+      child->vt = VT_DISPATCH;
+      if ((GetViewAccessibilityWrapper(child_view))->
+          GetInstance(IID_IAccessible,
+                      reinterpret_cast<void**>(&child->pdispVal)) == S_OK) {
+        // Increment the reference count for the retrieved interface.
+        child->pdispVal->AddRef();
+        return S_OK;
+      } else {
+        return E_NOINTERFACE;
+      }
+    }
   }
+
+  return S_OK;
 }
 
-STDMETHODIMP ViewAccessibility::get_accParent(IDispatch** disp_parent) {
-  if (!disp_parent) {
+STDMETHODIMP ViewAccessibility::accLocation(LONG* x_left, LONG* y_top,
+                                            LONG* width, LONG* height,
+                                            VARIANT var_id) {
+  if (var_id.vt != VT_I4 || !x_left || !y_top || !width || !height) {
     return E_INVALIDARG;
   }
 
-  views::View* parent_view = view_->GetParent();
-
-  if (!parent_view) {
-    // This function can get called during teardown of WidetWin so we
-    // should bail out if we fail to get the HWND.
-    if (!view_->GetWidget() || !view_->GetWidget()->GetNativeView()) {
-      *disp_parent = NULL;
-      return S_FALSE;
-    }
+  gfx::Rect view_bounds;
+  // Retrieving the parent View to be used for converting from view-to-screen
+  // coordinates.
+  views::View* parent = view_->GetParent();
 
-    // For a View that has no parent (e.g. root), point the accessible parent to
-    // the default implementation, to interface with Windows' hierarchy and to
-    // support calls from e.g. WindowFromAccessibleObject.
-    HRESULT hr =
-        ::AccessibleObjectFromWindow(view_->GetWidget()->GetNativeView(),
-                                     OBJID_WINDOW, IID_IAccessible,
-                                     reinterpret_cast<void**>(disp_parent));
+  if (parent == NULL) {
+    // If no parent, remain within the same View.
+    parent = view_;
+  }
 
-    if (!SUCCEEDED(hr)) {
-      *disp_parent = NULL;
-      return S_FALSE;
+  if (var_id.lVal == CHILDID_SELF) {
+    // Retrieve active View's bounds.
+    view_bounds = view_->bounds();
+  } else {
+    // Check to see if child is out-of-bounds.
+    if (!IsValidChild((var_id.lVal - 1), view_)) {
+      return E_INVALIDARG;
     }
-
-    return S_OK;
+    // Retrieve child bounds.
+    view_bounds = view_->GetChildViewAt(var_id.lVal - 1)->bounds();
+    // Parent View is current View.
+    parent = view_;
   }
 
-  // Retrieve the IUnknown interface for the parent view, and assign the
-  // IDispatch returned.
-  if ((GetAccessibleWrapper(parent_view))->
-      GetInstance(IID_IAccessible,
-                  reinterpret_cast<void**>(disp_parent)) == S_OK) {
-    // Increment the reference count for the retrieved interface.
-    (*disp_parent)->AddRef();
-    return S_OK;
+  if (!view_bounds.IsEmpty()) {
+    *width  = view_bounds.width();
+    *height = view_bounds.height();
+
+    gfx::Point topleft(view_bounds.origin());
+    views::View::ConvertPointToScreen(parent, &topleft);
+    *x_left = topleft.x();
+    *y_top  = topleft.y();
   } else {
-    return E_NOINTERFACE;
+    return E_FAIL;
   }
+  return S_OK;
 }
 
 STDMETHODIMP ViewAccessibility::accNavigate(LONG nav_dir, VARIANT start,
@@ -145,7 +147,7 @@ STDMETHODIMP ViewAccessibility::accNavigate(LONG nav_dir, VARIANT start,
 
       if (child->GetChildViewCount() != 0) {
         end->vt = VT_DISPATCH;
-        if ((GetAccessibleWrapper(child))->
+        if ((GetViewAccessibilityWrapper(child))->
             GetInstance(IID_IAccessible,
                         reinterpret_cast<void**>(&end->pdispVal)) == S_OK) {
           // Increment the reference count for the retrieved interface.
@@ -195,7 +197,7 @@ STDMETHODIMP ViewAccessibility::accNavigate(LONG nav_dir, VARIANT start,
         if (child->GetChildViewCount() != 0) {
           end->vt = VT_DISPATCH;
           // Retrieve IDispatch for non-leaf child.
-          if ((GetAccessibleWrapper(child))->
+          if ((GetViewAccessibilityWrapper(child))->
               GetInstance(IID_IAccessible,
                           reinterpret_cast<void**>(&end->pdispVal)) == S_OK) {
             // Increment the reference count for the retrieved interface.
@@ -248,65 +250,84 @@ STDMETHODIMP ViewAccessibility::accNavigate(LONG nav_dir, VARIANT start,
   return S_OK;
 }
 
-STDMETHODIMP ViewAccessibility::get_accFocus(VARIANT* focus_child) {
-  if (!focus_child) {
+STDMETHODIMP ViewAccessibility::get_accChild(VARIANT var_child,
+                                             IDispatch** disp_child) {
+  if (var_child.vt != VT_I4 || !disp_child) {
     return E_INVALIDARG;
   }
 
-  if (view_->GetChildViewCount() == 0 && view_->HasFocus()) {
-    // Parent view has focus.
-    focus_child->vt = VT_I4;
-    focus_child->lVal = CHILDID_SELF;
+  // If var_child is the parent, remain with the same IDispatch.
+  if (var_child.lVal == CHILDID_SELF) {
+    return S_OK;
+  }
+
+  views::View* child_view = NULL;
+  bool get_iaccessible = false;
+
+  // Check to see if child is out-of-bounds.
+  if (IsValidChild((var_child.lVal - 1), view_)) {
+    child_view = view_->GetChildViewAt(var_child.lVal - 1);
   } else {
-    bool has_focus = false;
-    int child_count = view_->GetChildViewCount();
-    // Search for child view with focus.
-    for (int child_id = 0; child_id < child_count; ++child_id) {
-      if (view_->GetChildViewAt(child_id)->HasFocus()) {
-        focus_child->vt = VT_I4;
-        focus_child->lVal = child_id + 1;
+    // Child is located elsewhere in the hierarchy, get ID and adjust for MSAA.
+    child_view = view_->GetViewByID(static_cast<int>(var_child.lVal));
+    get_iaccessible = true;
+  }
 
-        // If child view is no leaf, retrieve IDispatch.
-        if (view_->GetChildViewAt(child_id)->GetChildViewCount() != 0) {
-          focus_child->vt = VT_DISPATCH;
-          this->get_accChild(*focus_child, &focus_child->pdispVal);
-        }
-        has_focus = true;
-        break;
-      }
-    }
-    // No current focus on any of the children.
-    if (!has_focus) {
-      focus_child->vt = VT_EMPTY;
-      return S_FALSE;
+  if (!child_view) {
+    // No child found.
+    *disp_child = NULL;
+    return E_FAIL;
+  }
+
+  if (get_iaccessible || child_view->GetChildViewCount() != 0) {
+    // Retrieve the IUnknown interface for the requested child view, and
+    // assign the IDispatch returned.
+    if ((GetViewAccessibilityWrapper(child_view))->
+        GetInstance(IID_IAccessible,
+                    reinterpret_cast<void**>(disp_child)) == S_OK) {
+      // Increment the reference count for the retrieved interface.
+      (*disp_child)->AddRef();
+      return S_OK;
+    } else {
+      // No interface, return failure.
+      return E_NOINTERFACE;
     }
+  } else {
+    // When at a leaf, children are handled by the parent object.
+    *disp_child = NULL;
+    return S_FALSE;
   }
+}
 
+STDMETHODIMP ViewAccessibility::get_accChildCount(LONG* child_count) {
+  if (!child_count || !view_) {
+    return E_INVALIDARG;
+  }
+
+  *child_count = view_->GetChildViewCount();
   return S_OK;
 }
 
-STDMETHODIMP ViewAccessibility::get_accName(VARIANT var_id, BSTR* name) {
-  if (var_id.vt != VT_I4 || !name) {
+STDMETHODIMP ViewAccessibility::get_accDefaultAction(VARIANT var_id,
+                                                     BSTR* def_action) {
+  if (var_id.vt != VT_I4 || !def_action) {
     return E_INVALIDARG;
   }
 
-  std::wstring temp_name;
+  std::wstring temp_action;
 
   if (var_id.lVal == CHILDID_SELF) {
-    // Retrieve the parent view's name.
-    view_->GetAccessibleName(&temp_name);
+    view_->GetAccessibleDefaultAction(&temp_action);
   } else {
     if (!IsValidChild((var_id.lVal - 1), view_)) {
       return E_INVALIDARG;
     }
-    // Retrieve the child view's name.
-    view_->GetChildViewAt(var_id.lVal - 1)->GetAccessibleName(&temp_name);
+    view_->GetChildViewAt(var_id.lVal - 1)->
+        GetAccessibleDefaultAction(&temp_action);
   }
-  if (!temp_name.empty()) {
-    // Return name retrieved.
-    *name = CComBSTR(temp_name.c_str()).Detach();
+  if (!temp_action.empty()) {
+    *def_action = CComBSTR(temp_action.c_str()).Detach();
   } else {
-    // If view has no name, return S_FALSE.
     return S_FALSE;
   }
 
@@ -337,212 +358,190 @@ STDMETHODIMP ViewAccessibility::get_accDescription(VARIANT var_id, BSTR* desc) {
   return S_OK;
 }
 
-STDMETHODIMP ViewAccessibility::get_accState(VARIANT var_id, VARIANT* state) {
-  if (var_id.vt != VT_I4 || !state) {
+STDMETHODIMP ViewAccessibility::get_accFocus(VARIANT* focus_child) {
+  if (!focus_child) {
     return E_INVALIDARG;
   }
 
-  state->vt = VT_I4;
-
-  if (var_id.lVal == CHILDID_SELF) {
-    // Retrieve all currently applicable states of the parent.
-    this->SetState(state, view_);
+  if (view_->GetChildViewCount() == 0 && view_->HasFocus()) {
+    // Parent view has focus.
+    focus_child->vt = VT_I4;
+    focus_child->lVal = CHILDID_SELF;
   } else {
-    if (!IsValidChild((var_id.lVal - 1), view_)) {
-      return E_INVALIDARG;
+    bool has_focus = false;
+    int child_count = view_->GetChildViewCount();
+    // Search for child view with focus.
+    for (int child_id = 0; child_id < child_count; ++child_id) {
+      if (view_->GetChildViewAt(child_id)->HasFocus()) {
+        focus_child->vt = VT_I4;
+        focus_child->lVal = child_id + 1;
+
+        // If child view is no leaf, retrieve IDispatch.
+        if (view_->GetChildViewAt(child_id)->GetChildViewCount() != 0) {
+          focus_child->vt = VT_DISPATCH;
+          this->get_accChild(*focus_child, &focus_child->pdispVal);
+        }
+        has_focus = true;
+        break;
+      }
+    }
+    // No current focus on any of the children.
+    if (!has_focus) {
+      focus_child->vt = VT_EMPTY;
+      return S_FALSE;
     }
-    // Retrieve all currently applicable states of the child.
-    this->SetState(state, view_->GetChildViewAt(var_id.lVal - 1));
   }
 
-  // Make sure that state is not empty, and has the proper type.
-  if (state->vt == VT_EMPTY)
-    return E_FAIL;
-
   return S_OK;
 }
 
-STDMETHODIMP ViewAccessibility::get_accRole(VARIANT var_id, VARIANT* role) {
-  if (var_id.vt != VT_I4 || !role) {
+STDMETHODIMP ViewAccessibility::get_accKeyboardShortcut(VARIANT var_id,
+                                                        BSTR* acc_key) {
+  if (var_id.vt != VT_I4 || !acc_key) {
     return E_INVALIDARG;
   }
 
+  std::wstring temp_key;
+
   if (var_id.lVal == CHILDID_SELF) {
-    // Retrieve parent role.
-    if (!view_->GetAccessibleRole(role)) {
-      return E_FAIL;
-    }
+    view_->GetAccessibleKeyboardShortcut(&temp_key);
   } else {
     if (!IsValidChild((var_id.lVal - 1), view_)) {
       return E_INVALIDARG;
     }
-    // Retrieve child role.
-    if (!view_->GetChildViewAt(var_id.lVal - 1)->GetAccessibleRole(role)) {
-      return E_FAIL;
-    }
+    view_->GetChildViewAt(var_id.lVal - 1)->
+        GetAccessibleKeyboardShortcut(&temp_key);
+  }
+  if (!temp_key.empty()) {
+    *acc_key = CComBSTR(temp_key.c_str()).Detach();
+  } else {
+    return S_FALSE;
   }
-
-  // Make sure that role is not empty, and has the proper type.
-  if (role->vt == VT_EMPTY)
-    return E_FAIL;
 
   return S_OK;
 }
 
-STDMETHODIMP ViewAccessibility::get_accDefaultAction(VARIANT var_id,
-                                                     BSTR* def_action) {
-  if (var_id.vt != VT_I4 || !def_action) {
+STDMETHODIMP ViewAccessibility::get_accName(VARIANT var_id, BSTR* name) {
+  if (var_id.vt != VT_I4 || !name) {
     return E_INVALIDARG;
   }
 
-  std::wstring temp_action;
+  std::wstring temp_name;
 
   if (var_id.lVal == CHILDID_SELF) {
-    view_->GetAccessibleDefaultAction(&temp_action);
+    // Retrieve the parent view's name.
+    view_->GetAccessibleName(&temp_name);
   } else {
     if (!IsValidChild((var_id.lVal - 1), view_)) {
       return E_INVALIDARG;
     }
-    view_->GetChildViewAt(var_id.lVal - 1)->
-        GetAccessibleDefaultAction(&temp_action);
+    // Retrieve the child view's name.
+    view_->GetChildViewAt(var_id.lVal - 1)->GetAccessibleName(&temp_name);
   }
-  if (!temp_action.empty()) {
-    *def_action = CComBSTR(temp_action.c_str()).Detach();
+  if (!temp_name.empty()) {
+    // Return name retrieved.
+    *name = CComBSTR(temp_name.c_str()).Detach();
   } else {
+    // If view has no name, return S_FALSE.
     return S_FALSE;
   }
 
   return S_OK;
 }
 
-STDMETHODIMP ViewAccessibility::accLocation(LONG* x_left, LONG* y_top,
-                                            LONG* width, LONG* height,
-                                            VARIANT var_id) {
-  if (var_id.vt != VT_I4 || !x_left || !y_top || !width || !height) {
+STDMETHODIMP ViewAccessibility::get_accParent(IDispatch** disp_parent) {
+  if (!disp_parent) {
     return E_INVALIDARG;
   }
 
-  gfx::Rect view_bounds;
-  // Retrieving the parent View to be used for converting from view-to-screen
-  // coordinates.
-  views::View* parent = view_->GetParent();
+  views::View* parent_view = view_->GetParent();
 
-  if (parent == NULL) {
-    // If no parent, remain within the same View.
-    parent = view_;
-  }
+  if (!parent_view) {
+    // This function can get called during teardown of WidetWin so we
+    // should bail out if we fail to get the HWND.
+    if (!view_->GetWidget() || !view_->GetWidget()->GetNativeView()) {
+      *disp_parent = NULL;
+      return S_FALSE;
+    }
 
-  if (var_id.lVal == CHILDID_SELF) {
-    // Retrieve active View's bounds.
-    view_bounds = view_->bounds();
-  } else {
-    // Check to see if child is out-of-bounds.
-    if (!IsValidChild((var_id.lVal - 1), view_)) {
-      return E_INVALIDARG;
+    // For a View that has no parent (e.g. root), point the accessible parent to
+    // the default implementation, to interface with Windows' hierarchy and to
+    // support calls from e.g. WindowFromAccessibleObject.
+    HRESULT hr =
+        ::AccessibleObjectFromWindow(view_->GetWidget()->GetNativeView(),
+                                     OBJID_WINDOW, IID_IAccessible,
+                                     reinterpret_cast<void**>(disp_parent));
+
+    if (!SUCCEEDED(hr)) {
+      *disp_parent = NULL;
+      return S_FALSE;
     }
-    // Retrieve child bounds.
-    view_bounds = view_->GetChildViewAt(var_id.lVal - 1)->bounds();
-    // Parent View is current View.
-    parent = view_;
-  }
 
-  if (!view_bounds.IsEmpty()) {
-    *width  = view_bounds.width();
-    *height = view_bounds.height();
+    return S_OK;
+  }
 
-    gfx::Point topleft(view_bounds.origin());
-    views::View::ConvertPointToScreen(parent, &topleft);
-    *x_left = topleft.x();
-    *y_top  = topleft.y();
+  // Retrieve the IUnknown interface for the parent view, and assign the
+  // IDispatch returned.
+  if ((GetViewAccessibilityWrapper(parent_view))->
+      GetInstance(IID_IAccessible,
+                  reinterpret_cast<void**>(disp_parent)) == S_OK) {
+    // Increment the reference count for the retrieved interface.
+    (*disp_parent)->AddRef();
+    return S_OK;
   } else {
-    return E_FAIL;
+    return E_NOINTERFACE;
   }
-  return S_OK;
 }
 
-
-// TODO(klink): Handle case where child View is not contained by parent.
-STDMETHODIMP ViewAccessibility::accHitTest(LONG x_left, LONG y_top,
-                                           VARIANT* child) {
-  if (!child) {
+STDMETHODIMP ViewAccessibility::get_accRole(VARIANT var_id, VARIANT* role) {
+  if (var_id.vt != VT_I4 || !role) {
     return E_INVALIDARG;
   }
 
-  gfx::Point pt(x_left, y_top);
-  views::View::ConvertPointToView(NULL, view_, &pt);
-
-  if (!view_->HitTest(pt)) {
-    // If containing parent is not hit, return with failure.
-    child->vt = VT_EMPTY;
-    return S_FALSE;
-  }
-
-  int child_count = view_->GetChildViewCount();
-  bool child_hit = false;
-  views::View* child_view = NULL;
-  for (int child_id = 0; child_id < child_count; ++child_id) {
-    // Search for hit within any of the children.
-    child_view = view_->GetChildViewAt(child_id);
-    views::View::ConvertPointToView(view_, child_view, &pt);
-    if (child_view->HitTest(pt)) {
-      // Store child_id (adjusted with +1 to convert to MSAA indexing).
-      child->lVal = child_id + 1;
-      child_hit = true;
-      break;
+  if (var_id.lVal == CHILDID_SELF) {
+    // Retrieve parent role.
+    if (!view_->GetAccessibleRole(role)) {
+      return E_FAIL;
     }
-    // Convert point back to parent view to test next child.
-    views::View::ConvertPointToView(child_view, view_, &pt);
-  }
-
-  child->vt = VT_I4;
-
-  if (!child_hit) {
-    // No child hit, return parent id.
-    child->lVal = CHILDID_SELF;
   } else {
-    if (child_view == NULL) {
+    if (!IsValidChild((var_id.lVal - 1), view_)) {
+      return E_INVALIDARG;
+    }
+    // Retrieve child role.
+    if (!view_->GetChildViewAt(var_id.lVal - 1)->GetAccessibleRole(role)) {
       return E_FAIL;
-    } else if (child_view->GetChildViewCount() != 0) {
-      // Retrieve IDispatch for child, if it is not a leaf.
-      child->vt = VT_DISPATCH;
-      if ((GetAccessibleWrapper(child_view))->
-          GetInstance(IID_IAccessible,
-                      reinterpret_cast<void**>(&child->pdispVal)) == S_OK) {
-        // Increment the reference count for the retrieved interface.
-        child->pdispVal->AddRef();
-        return S_OK;
-      } else {
-        return E_NOINTERFACE;
-      }
     }
   }
 
+  // Make sure that role is not empty, and has the proper type.
+  if (role->vt == VT_EMPTY)
+    return E_FAIL;
+
   return S_OK;
 }
 
-STDMETHODIMP ViewAccessibility::get_accKeyboardShortcut(VARIANT var_id,
-                                                        BSTR* acc_key) {
-  if (var_id.vt != VT_I4 || !acc_key) {
+STDMETHODIMP ViewAccessibility::get_accState(VARIANT var_id, VARIANT* state) {
+  if (var_id.vt != VT_I4 || !state) {
     return E_INVALIDARG;
   }
 
-  std::wstring temp_key;
+  state->vt = VT_I4;
 
   if (var_id.lVal == CHILDID_SELF) {
-    view_->GetAccessibleKeyboardShortcut(&temp_key);
+    // Retrieve all currently applicable states of the parent.
+    this->SetState(state, view_);
   } else {
     if (!IsValidChild((var_id.lVal - 1), view_)) {
       return E_INVALIDARG;
     }
-    view_->GetChildViewAt(var_id.lVal - 1)->
-        GetAccessibleKeyboardShortcut(&temp_key);
-  }
-  if (!temp_key.empty()) {
-    *acc_key = CComBSTR(temp_key.c_str()).Detach();
-  } else {
-    return S_FALSE;
+    // Retrieve all currently applicable states of the child.
+    this->SetState(state, view_->GetChildViewAt(var_id.lVal - 1));
   }
 
+  // Make sure that state is not empty, and has the proper type.
+  if (state->vt == VT_EMPTY)
+    return E_FAIL;
+
   return S_OK;
 }
 
@@ -612,29 +611,29 @@ void ViewAccessibility::SetState(VARIANT* state, views::View* view) {
 // IAccessible functions not supported.
 
 HRESULT ViewAccessibility::accDoDefaultAction(VARIANT var_id) {
-  return DISP_E_MEMBERNOTFOUND;
+  return E_NOTIMPL;
 }
 
 STDMETHODIMP ViewAccessibility::get_accValue(VARIANT var_id, BSTR* value) {
   if (value)
     *value = NULL;
-  return DISP_E_MEMBERNOTFOUND;
+  return E_NOTIMPL;
 }
 
 STDMETHODIMP ViewAccessibility::get_accSelection(VARIANT* selected) {
   if (selected)
     selected->vt = VT_EMPTY;
-  return DISP_E_MEMBERNOTFOUND;
+  return E_NOTIMPL;
 }
 
 STDMETHODIMP ViewAccessibility::accSelect(LONG flagsSelect, VARIANT var_id) {
-  return DISP_E_MEMBERNOTFOUND;
+  return E_NOTIMPL;
 }
 
 STDMETHODIMP ViewAccessibility::get_accHelp(VARIANT var_id, BSTR* help) {
   if (help)
     *help = NULL;
-  return DISP_E_MEMBERNOTFOUND;
+  return E_NOTIMPL;
 }
 
 STDMETHODIMP ViewAccessibility::get_accHelpTopic(BSTR* help_file,
@@ -646,15 +645,15 @@ STDMETHODIMP ViewAccessibility::get_accHelpTopic(BSTR* help_file,
   if (topic_id) {
     *topic_id = static_cast<LONG>(-1);
   }
-  return DISP_E_MEMBERNOTFOUND;
+  return E_NOTIMPL;
 }
 
 STDMETHODIMP ViewAccessibility::put_accName(VARIANT var_id, BSTR put_name) {
   // Deprecated.
-  return DISP_E_MEMBERNOTFOUND;
+  return E_NOTIMPL;
 }
 
 STDMETHODIMP ViewAccessibility::put_accValue(VARIANT var_id, BSTR put_val) {
   // Deprecated.
-  return DISP_E_MEMBERNOTFOUND;
+  return E_NOTIMPL;
 }