Looks reasonable, since (afaics) that method doesn't do anything with the data. It'd be a different matter if it took the variables and encoded them into the body, for instance.
Considering that it doesn't do anything with the data at all, I'd say it really has no business setting a content type, but I guess it's better left in place for compatibility.
Patch applied in 7.8. Thanks.