[Merge] lp:~michael-sheldon/webbrowser-app/ssl-status into lp:webbrowser-app

Olivier Tilloy olivier.tilloy at canonical.com
Mon Sep 22 11:30:25 UTC 2014


> > 285     +        target: certificateError ? certificateError : null
> >
> > isn’t this equivalent to:
> >
> >     target: certificateError
> 
>  This would cause a warning due to the certificateError being undefined on
> start up (whereas explicitly setting a null target won't).

Ah, ok. I would have expected it to be 'null' rather than 'undefined'. The conditional makes sense then.
-- 
https://code.launchpad.net/~michael-sheldon/webbrowser-app/ssl-status/+merge/235309
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.



More information about the Ubuntu-reviews mailing list