Nodes with IDs should not be excluded as they might be the target of in-page links and attributes s…

third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp

 /*
  * Copyright (C) 2012, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 1007 matching lines @@
   Element* element = ToElement(node_);
   // We exclude elements that are in the shadow DOM.
   if (element->ContainingShadowRoot())
     return false;
 
   if (isHTMLAnchorElement(element)) {
     HTMLAnchorElement* html_element = toHTMLAnchorElement(element);
     return html_element->HasName() || html_element->HasID();
   }
 
-  if (element->HasID() && (IsLandmarkRelated() || isHTMLDivElement(element)))
+  if (element->HasID() && (IsLandmarkRelated() || isHTMLSpanElement(element) ||
+                           isHTMLDivElement(element))) {
     return true;
+  }
   return false;
 }
 
 bool AXNodeObject::IsMenu() const {
   return RoleValue() == kMenuRole;
 }
 
 bool AXNodeObject::IsMenuButton() const {
   return RoleValue() == kMenuButtonRole;
 }
@@ skipping 1442 matching lines @@
 
     // If this element is an ARIA text box or content editable, post a "value
     // changed" notification on it so that it behaves just like a native input
     // element or textarea.
     if (parent->IsNonNativeTextControl())
       cache.PostNotification(parent_node, AXObjectCacheImpl::kAXValueChanged);
   }
 }
 
 void AXNodeObject::UpdateAccessibilityRole() {
+  role_ = DetermineAccessibilityRole();
   bool ignored_status = AccessibilityIsIgnored();
-  role_ = DetermineAccessibilityRole();
 
   // The AX hierarchy only needs to be updated if the ignored status of an
   // element has changed.
   if (ignored_status != AccessibilityIsIgnored())
     ChildrenChanged();
 }
 
 void AXNodeObject::ComputeAriaOwnsChildren(
     HeapVector<Member<AXObject>>& owned_children) const {
   if (!HasAttribute(aria_ownsAttr))
@@ skipping 713 matching lines @@
     return String();
   return ToTextControlElement(node)->StrippedPlaceholder();
 }
 
 DEFINE_TRACE(AXNodeObject) {
   visitor->Trace(node_);
   AXObject::Trace(visitor);
 }
 
 }  // namespace blink