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

Revision history for this message
Olivier Tilloy (osomon) wrote :

There seems to be a regression in what the address bar displays, probably not introduced by this MR (maybe by http://bazaar.launchpad.net/~phablet-team/webbrowser-app/trunk/revision/710), which becomes very visible when trying to browse a site with an invalid security certificate: browse to any site, wait for it to fully load, then tap on the address bar and try to browse to e.g. https://www.pcwebshop.co.uk/ : you’re presented with the security warning page, but the shortened address in the address bar is that of the previously browsed site.

We might want to address this issue as part of this MR, considering that it makes it much more visible to the user.

« Back to merge proposal