[webapp-container] For click packages with multiple desktop files, the single instance server prevents each instance to be run
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
webbrowser-app (Ubuntu) |
Fix Released
|
High
|
Alexandre Abreu |
Bug Description
The container (sharing some code with the webbrowser, uses the package name (ignoring the app specific name) to set the application name, which is then used in key paths for the running app:
- writable locations,
- cache folders,
- single instance server name,
etc.
The issue is that for click packages with multiple desktop files (that are supported by the platform), some of those names prevent the individual apps to run properly. the single instance server name is wrongly based on the writablelocation folder which is in turn based on the applicationname. This application name is set to ignore the app specific name and only care about the package name which makes those click package file to properly run each app individually.
Related branches
- system-apps-ci-bot: Needs Fixing (continuous-integration)
- Olivier Tilloy: Approve
-
Diff: 316 lines (+110/-37)10 files modifiedsrc/app/browserapplication.cpp (+32/-24)
src/app/browserapplication.h (+1/-2)
src/app/config.h.in (+0/-1)
src/app/single-instance-manager.cpp (+39/-3)
src/app/single-instance-manager.h (+1/-1)
src/app/webbrowser/webbrowser-app.cpp (+1/-1)
src/app/webcontainer/webapp-container.cpp (+27/-1)
src/app/webcontainer/webapp-container.h (+1/-0)
tests/autopilot/webapp_container/tests/__init__.py (+4/-0)
tests/unittests/single-instance-manager/tst_SingleInstanceManagerTests.cpp (+4/-4)
Changed in webbrowser-app (Ubuntu): | |
assignee: | nobody → Alexandre Abreu (abreu-alexandre) |
importance: | Undecided → High |
status: | New → Confirmed |
status: | Confirmed → In Progress |
This bug was fixed in the package webbrowser-app - 0.23+16. 10.20160526- 0ubuntu1
--------------- 10.20160526- 0ubuntu1) yakkety; urgency=medium
webbrowser-app (0.23+16.
[ Alexandre Abreu ] webbrowser app_id; Properly set applicationname based
* Fix account logic translation typo
* Fix container/
on package name and app name (LP: #1574884, #1577806)
[ CI Train Bot ]
* Resync trunk.
-- Alexandre Abreu <email address hidden> Thu, 26 May 2016 17:01:10 +0000