Even if we're not using template in updateCardData we need to regenerate the card data when the template changes since potentially the sourceComponent has changed and thus the new item needs new contents.
Now one could remove this line and put a call to updateCardData on onSourceComponentChanged or put a call to updateCardData in onLoaded but both result in more updateCardData than the current scheme.
Numbers when running make xvfbtestCard
without this branch: 352 calls
with this branch [broken]: 211 calls
with this branch + onSourceComponentChanged: 428 calls
with this branch + onLoaded: 428 calls
Branch is wrong :/
Even if we're not using template in updateCardData we need to regenerate the card data when the template changes since potentially the sourceComponent has changed and thus the new item needs new contents.
Now one could remove this line and put a call to updateCardData on onSourceCompone ntChanged or put a call to updateCardData in onLoaded but both result in more updateCardData than the current scheme.
Numbers when running make xvfbtestCard ntChanged: 428 calls
without this branch: 352 calls
with this branch [broken]: 211 calls
with this branch + onSourceCompone
with this branch + onLoaded: 428 calls