Merge lp:~azzar1/unity/fix-991637 into lp:unity

[Andrea Azzarone]

Fix lp bug 991637.

== Problem ==
unity launcher vanishes when switching to mirrored displays

== Test ==
Unit test added.

[Brandon Schaefer (brandontschaefer)]

Awesome, LGTM.

review: Approve

[Marco Trevisan (Treviño) (3v1n0)]

review: Approve

=== modified file 'launcher/LauncherController.cpp'
--- launcher/LauncherController.cpp	2013-02-09 15:16:41 +0000
+++ launcher/LauncherController.cpp	2013-02-14 19:33:25 +0000
@@ -216,7 +216,7 @@
       launchers[i] = nux::ObjectPtr<Launcher>(CreateLauncher());
     }
 
-    int monitor = (num_launchers == 1) ? primary : i;
+    int monitor = (num_launchers == 1 && num_monitors > 1) ? primary : i;
 
     if (launchers[i]->monitor() != monitor)
     {
 
=== modified file 'tests/test_launcher_controller.cpp'
--- tests/test_launcher_controller.cpp	2013-02-09 15:16:41 +0000
+++ tests/test_launcher_controller.cpp	2013-02-14 19:33:25 +0000
@@ -346,6 +346,16 @@
   }
 }
 
+TEST_F(TestLauncherController, MirroredMultimonitorSingleLauncherOnExternalMonitor)
+{
+  // See lp bug 991637
+  lc.multiple_launchers = false;
+  uscreen.SetPrimary(1);
+
+  ASSERT_EQ(lc.launchers().size(), 1);
+  ASSERT_EQ(lc.launcher().monitor(), 0);
+}
+
 TEST_F(TestLauncherController, MultimonitorSwitchToMultipleLaunchers)
 {
   lc.multiple_launchers = false;