Correctly fire active descendant changes for more roles

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

 /*
  * Copyright (C) 2008, 2009, 2011 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 999 matching lines @@
 bool AXObject::ariaPressedIsPresent() const
 {
     return !getAttribute(aria_pressedAttr).isEmpty();
 }
 
 bool AXObject::supportsActiveDescendant() const
 {
     // According to the ARIA Spec, all ARIA composite widgets, ARIA text boxes
     // and ARIA groups should be able to expose an active descendant.
     // Implicitly, <input> and <textarea> elements should also have this ability.
-    switch (ariaRoleAttribute()) {
+    switch (roleValue()) {
     case ComboBoxRole:
     case GridRole:
     case GroupRole:
     case ListBoxRole:
     case MenuRole:
     case MenuBarRole:
+    case PopUpButtonRole:
     case RadioGroupRole:
     case RowRole:
     case SearchBoxRole:
     case TabListRole:
     case TextFieldRole:
     case ToolbarRole:
     case TreeRole:
     case TreeGridRole:
         return true;
     default:
@@ skipping 706 matching lines @@
 
 DEFINE_TRACE(AXObject)
 {
     visitor->trace(m_children);
     visitor->trace(m_parent);
     visitor->trace(m_cachedLiveRegionRoot);
     visitor->trace(m_axObjectCache);
 }
 
 } // namespace blink