[Background Sync] Use GcmNetworkManager to start the browser for sync events.

chrome/browser/background_sync/background_sync_controller_impl.cc

Index: chrome/browser/background_sync/background_sync_controller_impl.cc
diff --git a/chrome/browser/background_sync/background_sync_controller_impl.cc b/chrome/browser/background_sync/background_sync_controller_impl.cc
index 7668229690e3eeca3f654ef0cb9ff39102c5b878..5932dcc65c9c7e6ed258dc0881bc7e9170a6caff 100644
--- a/chrome/browser/background_sync/background_sync_controller_impl.cc
+++ b/chrome/browser/background_sync/background_sync_controller_impl.cc
@@ -8,6 +8,10 @@
 #include "chrome/browser/profiles/profile.h"
 #include "components/rappor/rappor_utils.h"
 
+#if defined(OS_ANDROID)
+#include "chrome/browser/android/background_sync_launcher_android.h"
+#endif
+
 BackgroundSyncControllerImpl::BackgroundSyncControllerImpl(Profile* profile)
     : profile_(profile) {}
 
@@ -28,3 +32,12 @@ void BackgroundSyncControllerImpl::NotifyBackgroundSyncRegistered(
   rappor::SampleDomainAndRegistryFromGURL(
       GetRapporService(), "BackgroundSync.Register.Origin", origin);
 }
+
+#if defined(OS_ANDROID)

[jkarlin, 2015/10/14 18:30:29]

You'll need a non-android definition of this function as well.

[iclelland, 2015/10/14 19:31:24]

Currently the non-Android version is the default null implementation from the
parent class. Since we don't have code yet to start the desktop browsers (and
that wouldn't be part of this CL anyway) should I leave a TODO here?

+void BackgroundSyncControllerImpl::LaunchBrowserWhenNextOnline(
+    const content::BackgroundSyncManager* registrant,
+    bool launch_when_next_online) {

[jkarlin, 2015/10/14 18:30:29]

DCHECK thread

[iclelland, 2015/10/14 19:31:24]

Done.

+  chrome::android::BackgroundSyncLauncherAndroid::LaunchBrowserWhenNextOnline(
+      registrant, launch_when_next_online);
+}
+#endif