Title: [232287] trunk/Source/WebCore
- Revision
- 232287
- Author
- n_w...@apple.com
- Date
- 2018-05-29 23:30:02 -0700 (Tue, 29 May 2018)
Log Message
Unreviewed, follow-up after r232285
https://bugs.webkit.org/show_bug.cgi?id=180871
Removed redundant null check.
* accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::isExpanded const):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (232286 => 232287)
--- trunk/Source/WebCore/ChangeLog 2018-05-30 04:36:07 UTC (rev 232286)
+++ trunk/Source/WebCore/ChangeLog 2018-05-30 06:30:02 UTC (rev 232287)
@@ -1,5 +1,15 @@
2018-05-29 Nan Wang <n_w...@apple.com>
+ Unreviewed, follow-up after r232285
+ https://bugs.webkit.org/show_bug.cgi?id=180871
+
+ Removed redundant null check.
+
+ * accessibility/AccessibilityObject.cpp:
+ (WebCore::AccessibilityObject::isExpanded const):
+
+2018-05-29 Nan Wang <n_w...@apple.com>
+
AX: macOS VoiceOver doesn't announce when details element is expanded when using role group
https://bugs.webkit.org/show_bug.cgi?id=180866
<rdar://problem/36074338>
Modified: trunk/Source/WebCore/accessibility/AccessibilityObject.cpp (232286 => 232287)
--- trunk/Source/WebCore/accessibility/AccessibilityObject.cpp 2018-05-30 04:36:07 UTC (rev 232286)
+++ trunk/Source/WebCore/accessibility/AccessibilityObject.cpp 2018-05-30 06:30:02 UTC (rev 232287)
@@ -2797,7 +2797,7 @@
// Summary element should use its details parent's expanded status.
if (isSummary()) {
if (const AccessibilityObject* parent = AccessibilityObject::matchedParent(*this, false, [] (const AccessibilityObject& object) {
- return object.node() && is<HTMLDetailsElement>(object.node());
+ return is<HTMLDetailsElement>(object.node());
}))
return parent->isExpanded();
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes