[Merge] lp:~phablet-team/network-manager/lp1480877-wifi-rm-dup-scan-signals into lp:~network-manager/network-manager/ubuntu

[Tony Espy]

My changes have no effect on whether or not scans are scheduled.

For the gory details see:

https://bugs.launchpad.net/ubuntu-rtm/+source/network-manager/+bug/1480877/comments/41

I checked, and the change that introduced DUP signals in first place was a location services fix ( see: 0002-wifi-cull-the-scan-list-before-signalling-ScanDone-b.patch ), which hasn't been up-streamed, so my changes aren't really directly applicable.

I've test three different phones, and a laptop running wily, and the only time bss_updated_cb is ever fired is after a scan completes.  As scan_done_cb always calls cull_scan_list directly, which in turn updates all of the last-seen properties of the know AccessPoints, the removal of bss_updated_cb is basically a no-op.

The removal of the schedule_scan_list call from new_bss_cb again is also predicated on the fact that new_bss_cb only ever gets triggered when a scan happens.  As it was the last function that used schedule_scanlist_cull, I removed it and the associated functions/priv member.






-- 
https://code.launchpad.net/~phablet-team/network-manager/lp1480877-wifi-rm-dup-scan-signals/+merge/271575
Your team Ubuntu Phablet Team is subscribed to branch lp:~phablet-team/network-manager/lp1480877-wifi-rm-dup-scan-signals.