I'm not sure how many times this loop will run, but if it's a lot - it's worth declaring var iconsData = args.extraIconsData, and iterating over that - just to avoid the object lookup on each iteration.

however, if it's something that runs just a few times, then it's not critical.