I see it does not make much of a difference, but wouldn't it be better to not construct a new instance, pass `null` and do `this.favicon = (favicon == null ? "" : favicon.getIcon())` in the constructor instead? This would also eliminate a possible NPE ;)