This will break if there are currently any callsites which pass a tag (e.g. phutil_tag()) or view to setHeader() or setSubheader(). Your test plan should make some effort to verify that there aren't unless you're sure about this without further verification.