webbrowser-app

Code review comment for lp:~michael-sheldon/webbrowser-app/ssl-status

  1. ssl-status
  2. Merge into trunk
Revision history for this message
Michael Sheldon (michael-sheldon) wrote :

> 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).

« Back to merge proposal