@Tony 
1) requestUpdate() should be left in, as the backend needs to tell other parts 
when the update request has been completed. I have tested this, so it does 
compile. In further testing, it does not look like any clients are requesting 
updates, so the increased power consumption is coming from somewhere else.

2) The crash is because of dead code, and included change is just a
quick fix. Proper fix is to go through this and remove the rest of the
dead and unneeded code.

3) If you want this delta as small as possible, it is safe to remove
this from the patch and only include #2. There is more dead code in
there now that it only relies on settings and active connections, which
I will remove in a different patch.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1480877

Title:
  Access points' "PropertiesChanged" dbus signals freeze UI on mobile
  devices

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1480877/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to