Merge lp:~3v1n0/unity/multimonitor-icons-redraw-fix into lp:unity

[Marco Trevisan (Treviño)]

Launcher: Connect the already present icons to signals when setting the model.

When a new launcher is added, we need to connect the icons that are already on the model to the needs_redraw signal, or the new launcher won't redraw when they ask that.

We need to connect the icons to the signals not to have redraw issues on newly created launchers...

Tests for this are already covered by the current manual tests, that were not fully passing on monitors hot-changes.

[Thomi Richards (thomir-deactivatedaccount)]

+1

review: Approve

=== modified file 'plugins/unityshell/src/Launcher.cpp'
--- plugins/unityshell/src/Launcher.cpp	2012-03-23 10:26:53 +0000
+++ plugins/unityshell/src/Launcher.cpp	2012-03-25 23:44:17 +0000
@@ -1814,6 +1814,9 @@
 {
   _model = model;
 
+  for (auto icon : *_model)
+    icon->needs_redraw.connect(sigc::mem_fun(this, &Launcher::OnIconNeedsRedraw));
+
   _model->icon_added.connect(sigc::mem_fun(this, &Launcher::OnIconAdded));
   _model->icon_removed.connect(sigc::mem_fun(this, &Launcher::OnIconRemoved));
   _model->order_changed.connect(sigc::mem_fun(this, &Launcher::OnOrderChanged));