in DrawableButton::paintButton() :
TextButton::buttonOnColourId and TextButton::buttonColourId
should be replaced by :
DrawableButton::backgroundOnColourId and DrawableButton::backgroundColourId
No.. it shouldn't.
When you use the ImageOnButtonBackgrounds style, it acts like a TextButton, and uses those colours.