Name calculation should not include nameFrom:author descendants.

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

 /*
  * Copyright (C) 2012 Apple 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 44 matching lines @@
 AXObjectImpl* AXSVGRoot::ComputeParent() const {
   DCHECK(!IsDetached());
   // If a parent was set because this is a remote SVG resource, use that
   // but otherwise, we should rely on the standard layout tree for the parent.
   if (parent_)
     return parent_;
 
   return AXLayoutObject::ComputeParent();
 }
 
 // SVG AAM 1.0 S8.2: the default role for an SVG root is "group".

[dmazzoni, 2017/05/16 16:25:39]

what about this comment?

 AccessibilityRole AXSVGRoot::DetermineAccessibilityRole() {
   AccessibilityRole role = AXLayoutObject::DetermineAccessibilityRole();
   if (role == kUnknownRole)
-    role = kGroupRole;
+    role = kGenericContainerRole;

[dmazzoni, 2017/05/16 16:25:39]

Maybe this one should be group?

[aleventhal1, 2017/05/16 18:11:26]

Here's my general rule:
role="group" or "fieldset" -> Group
Any other container that needs to be exposed for any reason -> Generic Container

How's that?

[dmazzoni, 2017/05/17 22:07:39]

Yeah, but the spec specifically says the default role for an SVG root is "group"
according to the comment above...

   return role;
 }
 
 // SVG elements are only ignored when a generic element would also be ignored.
 bool AXSVGRoot::ComputeAccessibilityIsIgnored(IgnoredReasons* reasons) const {
   return AccessibilityIsIgnoredByDefault(reasons);
 }
 
 }  // namespace blink