Bug 43604 – AX: Remove some dead code from AccessibilityRenderObject

• Status: RESOLVED
• Resolution: INVALID
• Component: Accessibility

[Chris Guillory]

Change to remove dead code from AccessibilityRenderObject.

[Chris Guillory]

Created attachment 63683 [details]
Remove dead code from AccessibilityRenderObject

[chris fleizach]

i don't think this is dead code. this would be used by role="range" if i'm not mistaken

[Chris Guillory]

Which object has the range role? The win bot succeeded.

[chris fleizach]

you can set an element to a range

<div role="range"></div>

[Chris Guillory]

Looks like AccessibilityObjectWrapper.mm is using these functions. Too bad the mac bot doesn't run for non-committers.

[Eric Seidel (no email)]

Comment on attachment 63683 [details]
Remove dead code from AccessibilityRenderObject

We should remove any non-tested code. :p

If this is in use, could one of your chrises add a test?

[chris fleizach]

(In reply to comment #6)
> (From update of attachment 63683 [details])
> We should remove any non-tested code. :p
> 
> If this is in use, could one of your chrises add a test?

LayoutTests/platform/mac/accessibility/aria-range.html
LayoutTests/platform/mac/accessibility/aria-range-value.html

[chris fleizach]

(In reply to comment #7)
> (In reply to comment #6)
> > (From update of attachment 63683 [details] [details])
> > We should remove any non-tested code. :p
> > 
> > If this is in use, could one of your chrises add a test?
> 
> LayoutTests/platform/mac/accessibility/aria-range.html
> LayoutTests/platform/mac/accessibility/aria-range-value.html

also meant to say, "already exists"