Code review comment for lp:~mardy/webbrowser-app/add-onlineaccount-support-for-container2

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

(apparently, when the initial version of this branch was reverted in trunk, the new C++ files were not removed)

421 + dbPath: ".local/share/" + applicationName + "/.QtWebKit/cookies.db"

Apparently this dbPath property is relative to the user’s home directory. This is not how local files to the applications should be accessed. Instead, use the "dataLocation" global context property being exported by the Ubuntu.Components.Extra.Browser plugin.

« Back to merge proposal