Comment 4 for bug 1669907

Pushed to master and rel_2_12. Thanks, Cesar and Terran.

I note that the underlying cause of the bug is that the grid provider.get() is getting called two times in quick succession when the user switch to the View Holds tab. That's, at minimum, inefficient, since the same set of holds is getting fetched twice. I'll open a separate bug.