[FLEX-33156] focusInHandler() in spark Scroller doesn't null check focusManager property before using

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Adobe Flex SDK 4.6 (Release)
• Fix Version/s: Adobe Flex SDK Next
• Component/s: Spark: Scroller
• Assignee: Carol Frampton
• Reporter: Roland Zwaga
• Created: 03/Aug/12 08:59
• Updated: 12/Sep/14 10:50
• Resolved: 04/Dec/12 20:01

Description

the focusHandler() method wrongfully assumes that the focusManager is always a valid instance. When checking the focusManager property in UIComponent it is clear that in certain cases this property can return a null value.

So in some cases this can lead to an RTE. I have encountered this once in my current project but find it very hard to reproduce. Basically a Scroller instance that was displayed in a popup window gave this RTE after the popup was removed.

Reproducable or not, I believe the null check should be added seeing the fact that the focusManager property explicitly returns null in some cases.

I have attached a patch file which adds the null check.