Remove methods from BrowserAccessibilityComWin which aren't needed.

content/browser/accessibility/browser_accessibility_com_win.cc

 // Copyright (c) 2017 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_com_win.h"
 
 #include <UIAutomationClient.h>
 #include <UIAutomationCoreApi.h>
 
 #include <algorithm>
@@ skipping 220 matching lines @@
     : owner_(nullptr),
       win_attributes_(new WinAttributes()),
       previous_scroll_x_(0),
       previous_scroll_y_(0) {}
 
 BrowserAccessibilityComWin::~BrowserAccessibilityComWin() {
   for (BrowserAccessibilityRelation* relation : relations_)
     relation->Release();
 }
 
-//
-// IAccessible methods.
-//
-// Conventions:
-// * Always test for owner() first and return E_FAIL if it's false.
-// * Always check for invalid arguments first, even if they're unused.
-// * Return S_FALSE if the only output is a string argument and it's empty.
-// * There are some methods that don't touch any state such as get_toolkitName.
-//   For these rare cases, you may not need to call owner().
-//
-
-HRESULT BrowserAccessibilityComWin::accDoDefaultAction(VARIANT var_id) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::accDoDefaultAction(var_id);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::accHitTest(LONG x_left,
-                                                    LONG y_top,
-                                                    VARIANT* child) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::accHitTest(x_left, y_top, child);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::accLocation(LONG* x_left,
-                                                     LONG* y_top,
-                                                     LONG* width,
-                                                     LONG* height,
-                                                     VARIANT var_id) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::accLocation(x_left, y_top, width, height, var_id);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::accNavigate(LONG nav_dir,
-                                                     VARIANT start,
-                                                     VARIANT* end) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::accNavigate(nav_dir, start, end);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accChild(VARIANT var_child,
-                                                      IDispatch** disp_child) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accChild(var_child, disp_child);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accChildCount(LONG* child_count) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accChildCount(child_count);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accDefaultAction(
-    VARIANT var_id,
-    BSTR* def_action) {
-  if (!owner())
-    return E_FAIL;
-
-  AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  return AXPlatformNodeWin::get_accDefaultAction(var_id, def_action);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accDescription(VARIANT var_id,
-                                                            BSTR* desc) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accDescription(var_id, desc);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accFocus(VARIANT* focus_child) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accFocus(focus_child);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accHelp(VARIANT var_id,
-                                                     BSTR* help) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accHelp(var_id, help);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accKeyboardShortcut(
-    VARIANT var_id,
-    BSTR* acc_key) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accKeyboardShortcut(var_id, acc_key);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accName(VARIANT var_id,
-                                                     BSTR* name) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accName(var_id, name);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accParent(
-    IDispatch** disp_parent) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accParent(disp_parent);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accRole(VARIANT var_id,
-                                                     VARIANT* role) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accRole(var_id, role);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accState(VARIANT var_id,
-                                                      VARIANT* state) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accState(var_id, state);
-}
-
 bool BrowserAccessibilityComWin::IsRangeValueSupported() {
   switch (MSAARole()) {
     case ROLE_SYSTEM_PROGRESSBAR:
     case ROLE_SYSTEM_SLIDER:
     case ROLE_SYSTEM_SPINBUTTON:
     case ROLE_SYSTEM_SCROLLBAR:
       return true;
     case ROLE_SYSTEM_SEPARATOR:
       return owner()->HasState(ui::AX_STATE_FOCUSABLE);
     default:
       return false;
   }
 }
 
-STDMETHODIMP BrowserAccessibilityComWin::get_accValue(VARIANT var_id,
-                                                      BSTR* value) {
-  if (!owner())
-    return E_FAIL;
+//
+// IAccessible overrides:
+//
 
-  return AXPlatformNodeWin::get_accValue(var_id, value);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accHelpTopic(BSTR* help_file,
-                                                          VARIANT var_id,
-                                                          LONG* topic_id) {
-  return AXPlatformNodeWin::get_accHelpTopic(help_file, var_id, topic_id);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::get_accSelection(VARIANT* selected) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::get_accSelection(selected);
-}
-
-STDMETHODIMP BrowserAccessibilityComWin::accSelect(LONG flags_sel,
-                                                   VARIANT var_id) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::accSelect(flags_sel, var_id);
-}
-
-STDMETHODIMP
-BrowserAccessibilityComWin::put_accName(VARIANT var_id, BSTR put_name) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::put_accName(var_id, put_name);
-}
-STDMETHODIMP
-BrowserAccessibilityComWin::put_accValue(VARIANT var_id, BSTR put_val) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::put_accValue(var_id, put_val);
+STDMETHODIMP BrowserAccessibilityComWin::get_accDefaultAction(
+    VARIANT var_id,
+    BSTR* def_action) {
+  AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
+  return AXPlatformNodeWin::get_accDefaultAction(var_id, def_action);
 }
 
 //
-// IAccessible2 methods.
+// IAccessible2 overrides:
 //
 
-STDMETHODIMP BrowserAccessibilityComWin::role(LONG* role) {
-  if (!owner())
-    return E_FAIL;
-
-  return AXPlatformNodeWin::role(role);
-}
-
 STDMETHODIMP BrowserAccessibilityComWin::get_attributes(BSTR* attributes) {
   WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_IA2_GET_ATTRIBUTES);
   if (!owner())
     return E_FAIL;
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
   if (!attributes)
     return E_INVALIDARG;
   *attributes = nullptr;
 
   if (!owner())
     return E_FAIL;
 
   base::string16 str;
   for (const base::string16& attribute : ia2_attributes())
     str += attribute + L';';
 
   if (str.empty())
     return S_FALSE;
 
   *attributes = SysAllocString(str.c_str());
   DCHECK(*attributes);
   return S_OK;
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_states(AccessibleStates* states) {
-  if (!owner())
-    return E_FAIL;
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-
   return AXPlatformNodeWin::get_states(states);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_uniqueID(LONG* unique_id) {
   WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_UNIQUE_ID);
   if (!owner())
     return E_FAIL;
 
   if (!unique_id)
     return E_INVALIDARG;
@@ skipping 194 matching lines @@
     return E_FAIL;
 
   if (!localized_extended_role)
     return E_INVALIDARG;
 
   return GetStringAttributeAsBstr(ui::AX_ATTR_ROLE_DESCRIPTION,
                                   localized_extended_role);
 }
 
 //
-// IAccessible2 methods not implemented.
-//
-
-STDMETHODIMP BrowserAccessibilityComWin::get_extendedRole(BSTR* extended_role) {
-  WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_EXTENDED_ROLE);
-  AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  return E_NOTIMPL;
-}
-STDMETHODIMP
-BrowserAccessibilityComWin::get_nExtendedStates(LONG* n_extended_states) {
-  WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_N_EXTENDED_STATES);
-  AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  return E_NOTIMPL;
-}
-STDMETHODIMP
-BrowserAccessibilityComWin::get_extendedStates(LONG max_extended_states,
-                                               BSTR** extended_states,
-                                               LONG* n_extended_states) {
-  WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_EXTENDED_STATES);
-  AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  return E_NOTIMPL;
-}
-STDMETHODIMP
-BrowserAccessibilityComWin::get_localizedExtendedStates(
-    LONG max_localized_extended_states,
-    BSTR** localized_extended_states,
-    LONG* n_localized_extended_states) {
-  WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_LOCALIZED_EXTENDED_STATES);
-  AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  return E_NOTIMPL;
-}
-STDMETHODIMP BrowserAccessibilityComWin::get_locale(IA2Locale* locale) {
-  WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_LOCALE);
-  AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  return E_NOTIMPL;
-}
-
-//
 // IAccessibleApplication methods.
 //
 
 STDMETHODIMP BrowserAccessibilityComWin::get_appName(BSTR* app_name) {
   WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_APP_NAME);
 
   if (!app_name)
     return E_INVALIDARG;
 
   // GetProduct() returns a string like "Chrome/aa.bb.cc.dd", split out
@@ skipping 121 matching lines @@
 
 //
 // IAccessibleTable methods.
 //
 
 STDMETHODIMP BrowserAccessibilityComWin::get_accessibleAt(
     long row,
     long column,
     IUnknown** accessible) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_accessibleAt(row, column, accessible);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_caption(IUnknown** accessible) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_caption(accessible);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_childIndex(long row,
                                                         long column,
                                                         long* cell_index) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_childIndex(row, column, cell_index);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_columnDescription(
     long column,
     BSTR* description) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_columnDescription(column, description);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_columnExtentAt(
     long row,
     long column,
     long* n_columns_spanned) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_columnExtentAt(row, column, n_columns_spanned);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_columnHeader(
     IAccessibleTable** accessible_table,
     long* starting_row_index) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_columnHeader(accessible_table,
                                              starting_row_index);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_columnIndex(long cell_index,
                                                          long* column_index) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_columnIndex(cell_index, column_index);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_nColumns(long* column_count) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_nColumns(column_count);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_nRows(long* row_count) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_nRows(row_count);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_nSelectedChildren(
     long* cell_count) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_nSelectedChildren(cell_count);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_nSelectedColumns(
     long* column_count) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_nSelectedColumns(column_count);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_nSelectedRows(long* row_count) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_nSelectedRows(row_count);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_rowDescription(long row,
                                                             BSTR* description) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_rowDescription(row, description);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_rowExtentAt(long row,
                                                          long column,
                                                          long* n_rows_spanned) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_rowExtentAt(row, column, n_rows_spanned);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_rowHeader(
     IAccessibleTable** accessible_table,
     long* starting_column_index) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_rowHeader(accessible_table,
                                           starting_column_index);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_rowIndex(long cell_index,
                                                       long* row_index) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_rowIndex(cell_index, row_index);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_selectedChildren(
     long max_children,
     long** children,
     long* n_children) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())

[dmazzoni, 2017/07/17 17:05:36]

Is there ever a case where owner_ is not equal to delegate_? It seems like now
that
BrowserAccessibilityComWin inherits from AXPlatformNodeWin, we could replace
owner() with a function that takes delegate_ and casts it to a
BrowserAccessibilityWin*.
Then it would be more clear that this check isn't needed.

[dougt, 2017/07/17 19:45:50]

As I understand things, owner_ and delegate_ in this case are always a
BrowserAccessibilityWin.
My plan is to essentially to remove all of these methods when we sort out the
BrowserAccessibilityState stuff.  I want to be able to completely depend on
AXPlatformNodeWin for dealing with a null delegate.

-    return E_FAIL;
-
   return AXPlatformNodeWin::get_selectedChildren(max_children, children,
                                                  n_children);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_selectedColumns(long max_columns,
                                                              long** columns,
                                                              long* n_columns) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);

[dmazzoni, 2017/07/17 17:05:36]

For a future change, perhaps we should set up something in
ui/accessibility/platform that keeps track of calls to various accessibility
APIs that trigger mode flags, but in a generic way - then
BrowserAccessibilityState could register to be called when those
triggers happen.

Then we wouldn't need to override all of these methods at all,
since that's the only remaining reason we're overriding them.

[dougt, 2017/07/17 19:45:50]

YES!!

Lets discuss how to do this soon.

-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_selectedColumns(max_columns, columns,
                                                 n_columns);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_selectedRows(long max_rows,
                                                           long** rows,
                                                           long* n_rows) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_selectedRows(max_rows, rows, n_rows);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_summary(IUnknown** accessible) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_summary(accessible);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_isColumnSelected(
     long column,
     boolean* is_selected) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_isColumnSelected(column, is_selected);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_isRowSelected(
     long row,
     boolean* is_selected) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_isRowSelected(row, is_selected);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_isSelected(long row,
                                                         long column,
                                                         boolean* is_selected) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_isSelected(row, column, is_selected);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_rowColumnExtentsAtIndex(
     long index,
     long* row,
     long* column,
     long* row_extents,
     long* column_extents,
     boolean* is_selected) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_rowColumnExtentsAtIndex(
       index, row, column, row_extents, column_extents, is_selected);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::selectRow(long row) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::selectRow(row);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::selectColumn(long column) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::selectColumn(column);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::unselectRow(long row) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::unselectRow(row);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::unselectColumn(long column) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::unselectColumn(column);
 }
 
 STDMETHODIMP
 BrowserAccessibilityComWin::get_modelChange(IA2TableModelChange* model_change) {
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_modelChange(model_change);
 }
 
 //
 // IAccessibleTable2 methods.
 //
 
 STDMETHODIMP BrowserAccessibilityComWin::get_cellAt(long row,
                                                     long column,
                                                     IUnknown** cell) {
   AddAccessibilityModeFlags(AccessibilityMode::kScreenReader);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_cellAt(row, column, cell);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_nSelectedCells(long* cell_count) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_nSelectedCells(cell_count);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_selectedCells(
     IUnknown*** cells,
     long* n_selected_cells) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_selectedCells(cells, n_selected_cells);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_selectedColumns(long** columns,
                                                              long* n_columns) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_selectedColumns(columns, n_columns);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_selectedRows(long** rows,
                                                           long* n_rows) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_selectedRows(rows, n_rows);
 }
 
 //
 // IAccessibleTableCell methods.
 //
 
 STDMETHODIMP BrowserAccessibilityComWin::get_columnExtent(
     long* n_columns_spanned) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_columnExtent(n_columns_spanned);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_columnHeaderCells(
     IUnknown*** cell_accessibles,
     long* n_column_header_cells) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_columnHeaderCells(cell_accessibles,
                                                   n_column_header_cells);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_columnIndex(long* column_index) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_columnIndex(column_index);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_rowExtent(long* n_rows_spanned) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_rowExtent(n_rows_spanned);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_rowHeaderCells(
     IUnknown*** cell_accessibles,
     long* n_row_header_cells) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_rowHeaderCells(cell_accessibles,
                                                n_row_header_cells);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_rowIndex(long* row_index) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_rowIndex(row_index);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_isSelected(boolean* is_selected) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_isSelected(is_selected);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_rowColumnExtents(
     long* row_index,
     long* column_index,
     long* row_extents,
     long* column_extents,
     boolean* is_selected) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_rowColumnExtents(
       row_index, column_index, row_extents, column_extents, is_selected);
 }
 
 STDMETHODIMP BrowserAccessibilityComWin::get_table(IUnknown** table) {
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes);
-  if (!owner())
-    return E_FAIL;
-
   return AXPlatformNodeWin::get_table(table);
 }
 
 //
 // IAccessibleText methods.
 //
 
 STDMETHODIMP BrowserAccessibilityComWin::get_nCharacters(LONG* n_characters) {
   WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_N_CHARACTERS);
   AddAccessibilityModeFlags(kScreenReaderAndHTMLAccessibilityModes |
@@ skipping 3055 matching lines @@
 
 BrowserAccessibilityComWin* ToBrowserAccessibilityComWin(
     BrowserAccessibility* obj) {
   if (!obj || !obj->IsNative())
     return nullptr;
   auto* result = static_cast<BrowserAccessibilityWin*>(obj)->GetCOM();
   return result;
 }
 
 }  // namespace content