DateFormatter should not attempt to load a format from storage if the format is "custom"

As discovered in #2846485: Terrible performance when rendering multi-value fields in views, DateForamtter::format attempts to load a date format from storage even when a custom date format is passed in. This leads to an unnecessary and expensive storage lookup. We can fix it just by checking of the format type is "custom" before attempting the lookup.