Android: Allow compiling browser without Java UI

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 895f8aeb28e5e2c565b42a3db22e0fb0a33e5c69..08fb008bd61184a3c1459e5d7c7a6699c95e73d3 100644
--- a/chrome/browser/background_sync/background_sync_controller_impl.cc
+++ b/chrome/browser/background_sync/background_sync_controller_impl.cc
@@ -8,7 +8,7 @@
 #include "chrome/browser/profiles/profile.h"
 #include "components/rappor/rappor_utils.h"
 
-#if defined(OS_ANDROID)
+#if defined(ANDROID_JAVA_UI)

[Ted C, 2015/11/30 19:39:05]

This class doesn't seem UI related either.  Don't we always want to do this?

[no sievers, 2015/11/30 23:56:34]

I think eventually we probably want it, but it needs refactoring.

JAVA_UI here also might mean JAVA_TAB_MODEL or what not (there is a web of
dependencies on the Java side in paricular).

I'd be happy to rename it also. Maybe JAVA_UI is not ideal, although a lot of it
ends up going back to this being the 'Java-based/Java-driven browser model /
frontend'. Can't really come up with a great name though.

 #include "chrome/browser/android/background_sync_launcher_android.h"
 #endif
 
@@ -39,7 +39,7 @@ void BackgroundSyncControllerImpl::RunInBackground(bool enabled,
 
   if (profile_->IsOffTheRecord())
     return;
-#if defined(OS_ANDROID)
+#if defined(ANDROID_JAVA_UI)
   BackgroundSyncLauncherAndroid::LaunchBrowserIfStopped(enabled, min_ms);
 #else
 // TODO(jkarlin): Use BackgroundModeManager to enter background mode. See