nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1815959 - Check the content type for SVG shapes and images, not the frame r=emilio

Browse source 
Differential Revision: https://phabricator.services.mozilla.com/D169369
This commit is contained in:
Robert Longson 2023-02-14 05:59:41 +00:00
parent fb10525574
commit 986ccaa6d3
1 changed files with 7 additions and 6 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

13
accessible/base/nsAccessibilityService.cpp
View file

@ -75,7 +75,6 @@
#include "mozilla/ProfilerMarkers.h"
#include "mozilla/Services.h"
#include "mozilla/StaticPrefs_accessibility.h"
#include "mozilla/SVGGeometryFrame.h"
#include "XULAlertAccessible.h"
#include "XULComboboxAccessible.h"
@ -1236,12 +1235,14 @@ LocalAccessible* nsAccessibilityService::CreateAccessible(
if (!newAcc) {
if (content->IsSVGElement()) {
SVGGeometryFrame* geometryFrame = do_QueryFrame(frame);
if (geometryFrame && MustSVGElementBeAccessible(content)) {
// A graphic elements: rect, circle, ellipse, line, path, polygon,
// polyline and image. A 'use' and 'text' graphic elements require
if (content->IsNodeOfType(nsINode::eSHAPE) ||
content->IsSVGElement(nsGkAtoms::image)) {
// Shape elements: rect, circle, ellipse, line, path, polygon,
// and polyline. 'use' and 'text' graphic elements require
// special support.
newAcc = new EnumRoleAccessible<roles::GRAPHIC>(content, document);
if (MustSVGElementBeAccessible(content)) {
newAcc = new EnumRoleAccessible<roles::GRAPHIC>(content, document);
}
} else if (content->IsSVGElement(nsGkAtoms::text)) {
newAcc = new HyperTextAccessibleWrap(content->AsElement(), document);
} else if (content->IsSVGElement(nsGkAtoms::svg)) {