Dedupe pubsubprocessor

@lloydwatkin can confirm that, but I think we're changing to ```String.isEmpty() ``` whenever we see those constructions preceded by a null check. Otherwise, I'd be happy to leave it as it is.