Complete the Additional Drivers implementation

Specifically; points 6 and 7 on the design wiki page need to be addressed:

If the changes required a restart, the “Software & Updates” panel is still open, and the “Additional Drivers” tab is still frontmost, the System Settings window should request attention.

If the changes required a restart, but you have since navigated away from the “Additional Drivers” tab or closed System Settings altogether, a parent-less alert box should appear with the title “Additional Drivers”, the “Software & Updates” icon, the text “ Warning /!\ You need to restart the computer to complete the driver changes.”, and “Restart Later” and “Restart Now” buttons (fixing bug 306708).

"Prompting to install additional drivers" is also not done yet.

apt-daemon tends to die sometimes, after multiple install/removes have been done in succession:
Traceback (most recent call last):
  File "/home/mtrudel/Public/software-properties/softwareproperties/gtk/SoftwarePropertiesGtk.py", line 1017, in on_driver_changes_apply
    self.transaction.run()
  File "/usr/lib/python3/dist-packages/defer/utils.py", line 107, in _deferable
    return func(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/aptdaemon/errors.py", line 192, in _convert_dbus_exception
    raise get_native_exception(error)
  File "/usr/lib/python3/dist-packages/aptdaemon/errors.py", line 190, in _convert_dbus_exception
    return func(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/aptdaemon/client.py", line 745, in run
    reply_handler=reply_handler)
  File "/usr/lib/python3/dist-packages/dbus/proxies.py", line 145, in __call__
    **keywords)
  File "/usr/lib/python3/dist-packages/dbus/connection.py", line 651, in call_blocking
    message, timeout)
dbus.exceptions.DBusException: org.freedesktop.DBus.Error.NoReply: Message did not receive a reply (timeout by message bus)
ERROR
It seems that the daemon died.
---

Definitely something to investigate.

As for NoReply, there are three common causes:

- aptdaemon crashes in the middle of an operation. Do you get an apport crash report when this happens?

- You call the operation synchronously. This should not be done, call it async and then have aptdaemon call a result callback.

- You keep the aptdaemon d-bus object around for a long time, while aptdaemon timed out regularly (after some minutes of inactivity). However, this actually ought to result in a ServiceNotFound exception, or should even re-activate the daemon through D-BUS.

NB I haven't looked at the source code at all.

Note, in ubuntu-drivers-common 0.2.61 I added a simple wrapper tool which pretends that you have an nvidia, fglrx, and broadcom wifi card. Run it like this in the s-p checkout:

 PYTHONPATH=. /usr/share/ubuntu-drivers-common/fake-devices-wrapper ./software-properties-gtk --data-dir=data

For s-p to work with the wrapper you need a tiny patch to s-p:

--- softwareproperties/gtk/SoftwarePropertiesGtk.py 2012-07-06 20:21:44 +0000
+++ softwareproperties/gtk/SoftwarePropertiesGtk.py 2012-07-12 14:46:42 +0000
@@ -935,7 +935,7 @@

     try:
       name = subprocess.check_output([db_command,
- device_info['syspath'].replace('/sys', '')
+ device_info['syspath'].replace(os.getenv('SYSFS_PATH', '/sys'), '')
                                      ],
                                      universal_newlines=True)
       name = " ".join(name.split('\n'))

Patch is applied to lp:~mathieu-tl/software-properties/additional-drivers.

FYI: You can remove the pci-db/usb-db stuff from s-p-common now and just use the 'vendor' and 'model' fields:

ubuntu-drivers-common (1:0.2.62) quantal; urgency=low

  * debian/control: As the PackageKit apt backend is going away, drop the
    Conflicts to packagekit-backend-aptcc. Instead, recommend
    python3-aptdaemon.pkcompat and update the description to point out that
    the PackageKit API does not work with the aptcc backend.
    (LP: #1023953)
  * UbuntuDrivers.detect.py, system_driver_packages(): Add 'vendor' and
    'model' fields for the human readable names from pci/usb databases.

 -- Martin Pitt <email address hidden> Fri, 13 Jul 2012 10:41:12 +0200

This bug was fixed in the package software-properties - 0.89

---------------
software-properties (0.89) quantal; urgency=low

  * Add the jockey replacement tab for software-properties-gtk:
    - finish the work started by cyphermox (LP: #1021733)
    - add dep on latest ubuntu-drivers-common for the backend
  * Wrap all dbus calls under a try/except catch for unauthorized calls.
    (original work by Edward Donovan) (LP: #828850)
 -- Didier Roche <email address hidden> Wed, 18 Jul 2012 11:28:09 +0200

See bug #1059227.