[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