[Merge] lp:~artmello/webbrowser-app/webbrowser-app-statesaver into lp:webbrowser-app
Olivier Tilloy
olivier.tilloy at canonical.com
Wed Jul 23 11:24:00 UTC 2014
Review: Needs Fixing
I’ve tested only the browser app on desktop so far, and the general use case seems to work fine, however I’m seeing a case that doesn’t:
webbrowser-app http://example.org
killall -9 webbrowser-app
webbrowser-app http://ubuntu.com
I expect the second invokation of webbrowser-app to open ubuntu.com, however the previous state is restored and it browses to example.org instead.
Note that this issue is most likely affecting desktop only, not phone, where this kind of URL opening will go through the UriHandler, which will ignore the saved state, according to documentation.
To address this, you probably need a mechanism similar to what you did for the webapp-container, where you recognize whether a URL was explicitly requested on the command line.
--
https://code.launchpad.net/~artmello/webbrowser-app/webbrowser-app-statesaver/+merge/227737
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.
More information about the Ubuntu-reviews
mailing list