Merge lp:~renatofilho/address-book-app/attach-to-sync-monitor into lp:~phablet-team/address-book-app/staging

[Renato Araujo Oliveira Filho]

Call "attach" on sync monitor server to notify a new client connection.
We need to notify sync-monitor that we are attached to him to start to receive signals.

You will need : lp:~renatofilho/sync-monitor/start-after-unity8 to test this.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:317
http://jenkins.qa.ubuntu.com/job/phablet-team-address-book-app-staging-ci/363/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/6337
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/4241
    SUCCESS: http://jenkins.qa.ubuntu.com/job/phablet-team-address-book-app-staging-utopic-i386-ci/363
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/5942
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/7589
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/7589/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/15245
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/3597
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/4629
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/4629/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/phablet-team-address-book-app-staging-ci/363/rebuild

review: Approve (continuous-integration)

=== modified file 'src/app/addressbookapp.cpp'
--- src/app/addressbookapp.cpp	2014-07-20 20:23:20 +0000
+++ src/app/addressbookapp.cpp	2014-10-30 20:04:38 +0000
@@ -221,6 +221,11 @@
 {
     unsetFirstRun();
 
+    if (m_syncMonitor) {
+        m_syncMonitor->call("detach");
+        delete m_syncMonitor;
+    }
+
     if (m_view) {
         delete m_view;
     }
@@ -438,9 +443,11 @@
                                        "com.canonical.SyncMonitor");
     if (m_syncMonitor->lastError().isValid()) {
         qWarning() << "Fail to connect with sync monitor:" << m_syncMonitor->lastError();
+    } else {
+        connect(m_syncMonitor, SIGNAL(stateChanged()), SIGNAL(syncingChanged()));
+        connect(m_syncMonitor, SIGNAL(enabledServicesChanged()), SIGNAL(syncEnabledChanged()));
+        m_syncMonitor->call("attach");
     }
-    connect(m_syncMonitor, SIGNAL(stateChanged()), SIGNAL(syncingChanged()));
-    connect(m_syncMonitor, SIGNAL(enabledServicesChanged()), SIGNAL(syncEnabledChanged()));
     Q_EMIT syncEnabledChanged();
     Q_EMIT syncingChanged();
 }