Major Description
Overriding isVisible is discouraged, visibility should be set just once. I know the rowcount is cached nowadays, but some time ago this actually bit us (admittedly in some way that I don't remember now, but I think by getRowCount being called too early in a request).