Noissue - Check for null and blank string instead of undefined

Noissue - Check for null and blank string instead of undefined

[Manish Jethani, 2018-03-30 13:55:49]

Patch Set 1

value is guaranteed never to be undefined here, so there's no point in checking
for undefined. If we really want a check, we should check for null and the blank
string.

[kzar, 2018-04-05 14:45:01]

On 2018/03/30 13:55:49, Manish Jethani wrote:
> Patch Set 1
> 
> value is guaranteed never to be undefined here, so there's no point in
checking
> for undefined. If we really want a check, we should check for null and the
blank
> string.

Yea, I nearly did `typeof value == "string"` when adding the $csp one but wanted
to be consistent with these other checks. But good point we should be checking
for null really anyway.

LGTM