FWIW, I think calls to `doLayout` should generally be unnecessary, *except* where a child item is added or removed to an already rendered container, and that container's layout is one which doesn't support dynamic (re-)laying-out.

So, in this case, it probably shouldn't be required.

Having said that, I know sometimes it's hard to get ExtJS to do what you want, but yes it's probably a bit of a 'smell' to have doLayout in there.