click-scope calls download-manager once per result

Bug #1215904 reported by Alejandro J. Cura
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-download-manager
Fix Released
High
Manuel de la Peña
ubuntu-download-manager (Ubuntu)
Fix Released
Undecided
Unassigned
unity-scope-click (Ubuntu)
Invalid
Medium
Alejandro J. Cura

Bug Description

Click scope is calling download-manager to get the DBus ObjectPaths of each app, whether they are downloading or not, and this is slow.

To fix this, we need to change the API of download manager so the DBus ObjectPaths for the downloads can be automatically inferred from some part of the metadata.

Related branches

Changed in unity-scope-click:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Alejandro J. Cura (alecu)
Changed in ubuntu-download-manager:
status: Confirmed → In Progress
Changed in ubuntu-download-manager:
milestone: none → 0.3
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-download-manager at revision 121, scheduled for release in ubuntu-download-manager, milestone 0.3

Changed in ubuntu-download-manager:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-download-manager - 0.2+13.10.20130920.1-0ubuntu1

---------------
ubuntu-download-manager (0.2+13.10.20130920.1-0ubuntu1) saucy; urgency=low

  [ Barry Warsaw ]
  * When extracting the QPair from _downloadProgress, use a reference
    local variable so that a new QPair isn't created, thus throwing away
    the update. (LP: #1226243)

  [ Manuel de la Pena ]
  * Fixes the irl example in the test folder. (LP: #1221598)
  * Added a class that provides a way to generate secure paths to
    appamor confined applications.
  * Make use of the apparmor class to build secure dbus paths. (LP:
    #1222901)
  * Increases the test coverage of the download factory class.
  * Added the new metadata key 'objectpath' where the client can propose
    a uuid for the download that will be used to create the path. (LP:
    #1215904)
  * Change the interface name so that is less confusing. (LP: #1224538)
  * Fixed the location where the file is stored. If the local-file
    metadata is provided and the app is not cofined that would be the
    used file else the XDG/APP_ID one will be used. (LP: #1224641,
    #1222914)
  * Provide two command line args to be used in tests. (LP: #1202809)
  * Set the accept-encoding header to 'identity' so that we do not
    expand any download. (LP: #1224678)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 126
 -- Ubuntu daily release <email address hidden> Fri, 20 Sep 2013 10:03:28 +0000

Changed in ubuntu-download-manager (Ubuntu):
status: New → Fix Released
Changed in ubuntu-download-manager:
status: Fix Committed → Fix Released
Changed in unity-scope-click:
importance: High → Medium
dobey (dobey)
affects: unity-scope-click → unity-scope-click (Ubuntu)
Leo Arias (elopio)
tags: added: qa-daily-testing rtm14
Revision history for this message
Alejandro J. Cura (alecu) wrote :

This affected the old vala scope, so I'm marking this bug as invalid.

Changed in unity-scope-click (Ubuntu):
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.