I'd definitively not include any custom background image implementation. For VTE versions that support it natively, and we already have UI prefs for, it seems OK to leave it, but for newer VTEs it seems better to just guard it out.
IMO it would be worth dropping completely everywhere to reduce the amount of ifdef checks. There will be half of them if this gets removed and this mis-feature isn't worth cluttering the code.