~abreu-alexandre/oxide/add-ua-to-downloadrequested

Viewing all changes in revision 711.

  • Committer: Chris Coulson
  • Date: 2014-09-02 11:11:08 UTC
  • Revision ID: chris.coulson@canonical.com-20140902111108-fr8fqi1klfgen36k
Some minor tweaks to the CookieManager API:
- Bring back setNetworkCookies as this is being used from C++ code in webbrowser-app. Note that the url parameter is no longer a QString though (it's a QUrl)
- Remove the status parameter entirely from the response to setCookies/setNetworkCookies - the problem with this is that it provides no way to know which cookies failed and which ones were set successfully. Instead, replace it by a new parameter (failedCookies) which returns a list of cookies that failed to set.
- Having setCookies + cookiesSet and getCookies + gotCookies is a bit confusing. These are now changed to setCookies + setCookiesResponse and getCookies + getCookiesResponse.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: