[Merge] lp:~michael-sheldon/webbrowser-app/fix-1354388 into lp:webbrowser-app

Olivier Tilloy olivier.tilloy at canonical.com
Wed Sep 24 15:47:58 UTC 2014


Review: Approve

Looks good, and works as expected. Thanks!

I’m seeing an additional use case that doesn’t seem to be covered (although I don’t think it’s a new regression, so it may be fine to tackle it separately):

  - browse to any page that triggers the display of a certificate warning page (e.g. https://testssl-expire.disig.sk/) and choose to proceed anyway
  - close the browser and open it again, the previously open page is re-loaded
  - the certificate warning page is displayed, but the address bar is empty, until you choose to proceed anyway

If you think it can be addressed easily and with minimal changes to this MR, then go ahead, otherwise I’m happy to report it separately afterwards.
-- 
https://code.launchpad.net/~michael-sheldon/webbrowser-app/fix-1354388/+merge/235796
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.



More information about the Ubuntu-reviews mailing list