Remove methods from BrowserAccessibilityComWin which aren't needed.

ui/accessibility/platform/ax_platform_node_win.cc

 // Copyright 2015 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 <atlbase.h>
 #include <atlcom.h>
 #include <limits.h>
 #include <oleacc.h>
 #include <stdint.h>
 
@@ skipping 912 matching lines @@
   return S_OK;
 }
 
 //
 // IAccessible2 methods not implemented.
 //
 
 STDMETHODIMP AXPlatformNodeWin::get_attribute(BSTR name, VARIANT* attribute) {
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_extendedRole(BSTR* extended_role) {
+  WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_EXTENDED_ROLE);

[dmazzoni, 2017/07/17 08:32:40]

Wait, how does this even compile if this macro is defined in
browser_accessibility_com_win.cc?

   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_nRelations(LONG* n_relations) {
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_relation(LONG relation_index,
                                              IAccessibleRelation** relation) {
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_relations(LONG max_relations,
                                               IAccessibleRelation** relations,
                                               LONG* n_relations) {
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::scrollTo(enum IA2ScrollType scroll_type) {
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::scrollToPoint(
     enum IA2CoordinateType coordinate_type,
     LONG x,
     LONG y) {
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_groupPosition(LONG* group_level,
                                                   LONG* similar_items_in_group,
                                                   LONG* position_in_group) {
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_localizedExtendedRole(
     BSTR* localized_extended_role) {
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_nExtendedStates(LONG* n_extended_states) {
+  WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_N_EXTENDED_STATES);
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_extendedStates(LONG max_extended_states,
                                                    BSTR** extended_states,
                                                    LONG* n_extended_states) {
+  WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_EXTENDED_STATES);
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::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);
+
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_locale(IA2Locale* locale) {
+  WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_LOCALE);
   return E_NOTIMPL;
 }
+
 STDMETHODIMP AXPlatformNodeWin::get_accessibleWithCaret(IUnknown** accessible,
                                                         long* caret_offset) {
   return E_NOTIMPL;
 }
 
 //
 // IAccessibleTable methods.
 //
 
 STDMETHODIMP AXPlatformNodeWin::get_accessibleAt(long row,
@@ skipping 828 matching lines @@
     REFGUID guidService, REFIID riid, void** object) {
   COM_OBJECT_VALIDATE_1_ARG(object);
 
   if (riid == IID_IAccessible2) {
     for (IAccessible2UsageObserver& observer :
          GetIAccessible2UsageObserverList()) {
       observer.OnIAccessible2Used();
     }
   }
 
-  if (guidService == IID_IAccessible ||
-      guidService == IID_IAccessible2 ||
+  if (guidService == IID_IAccessible || guidService == IID_IAccessible2 ||
       guidService == IID_IAccessible2_2 ||
+      guidService == IID_IAccessibleTable ||
+      guidService == IID_IAccessibleTable2 ||
+      guidService == IID_IAccessibleTableCell ||
       guidService == IID_IAccessibleText) {
     return QueryInterface(riid, object);
   }
 
   *object = nullptr;
   return E_FAIL;
 }
 
 //
 // Private member functions.
@@ skipping 955 matching lines @@
   if (container && container->GetData().role == ui::AX_ROLE_GROUP)
     container = FromNativeViewAccessible(container->GetParent());
 
   if (!container)
     return false;
 
   return container->GetData().role == ui::AX_ROLE_TREE_GRID;
 }
 
 }  // namespace ui