I was wrong before about needing it - we actually want to iterate over all the results of crash, and put each product+version combination into ```top_crashes[signame]``` - having the ```break``` there caused a regression where we only show one product+version per signature, when really there should be more (see the associated bug for details)