396603 - Remove MustPrune() from accHitTest and similar

Status: RESOLVED WONTFIX

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

AccessibleObjectFromPoint() already handles the pruning, so the callers of it should not need to use it.

Comment 1

[David Bolter [:davidb] (NeedInfo me for attention)]

Mass un-assigning bugs assigned to Aaron.

Comment 2

[Trevor Saunders (:tbsaunde)]

Surkov anything left to fix here? we don't call MustPrune() in accHitTest()

Comment 3

[alexander :surkov (:asurkov)]

(In reply to Trevor Saunders (:tbsaunde) from comment #2)
> Surkov anything left to fix here? we don't call MustPrune() in accHitTest()

MustPrune is part of ChildAtPoint. This is common code path shared between platforms. Even if AccessibleObjectFromPoint takes care to prune not allowed accessibles but then we need to check what other platforms expect. I think this is low priority bug since this call shouldn't be expensive at all and the bug requires some investigations. We can keep it open and get back to it by a chance.

Comment 4

[Marco Zehe (:MarcoZ)]

Surkov, still valid?

Comment 5

[alexander :surkov (:asurkov)]

perhaps not worth to keep open, there's no big harm to have extra call of MustPrune() on some platforms, no one touched this for years, suggesting to close it as wontfix