Change to remove dead code from AccessibilityRenderObject.
Created attachment 63683 [details] Remove dead code from AccessibilityRenderObject
i don't think this is dead code. this would be used by role="range" if i'm not mistaken
Which object has the range role? The win bot succeeded.
you can set an element to a range <div role="range"></div>
Looks like AccessibilityObjectWrapper.mm is using these functions. Too bad the mac bot doesn't run for non-committers.
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?
(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
(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"