FocusManager tries to focus a node that is not visible

[tanneman]

The call this.grid.domNode.focus() in _FocusManager._delayedHeaderFocus can result in an error in IE (Can't move focus to the control because it is invisible).

I have attached an html file that exposes the bug. The steps to reproduce are in the attached html file. Just click on the buttons :).

Steps to reproduce

• Create a DataGrid? with selectable set tot true.
• Fill the table with rows and make sure you get a vertical scrollbar.
• Move the focus on one of the column header nodes (click on it)
• Hide the table (set style display to none)
• fill the table (or update the rowCount)

Expected

• No problems

Actual

• A script error occurs (on IE 6-8)

[evan]

(In [25473]) Refs #11867 add a safe check as - if(!cell){return} to avoid exceptions when cell is undefined, backport to 1.6/1.5 branches

[bill]

DojoX Grid and EnhancedGrid are deprecated in favor of ​dgrid and ​gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.