Android: Allow compiling browser without Java UI

chrome/browser/background_sync/background_sync_controller_impl.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/background_sync/background_sync_controller_impl.h"
 
 #include "chrome/browser/browser_process.h"
 #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
 
 BackgroundSyncControllerImpl::BackgroundSyncControllerImpl(Profile* profile)
     : profile_(profile) {}
 
 BackgroundSyncControllerImpl::~BackgroundSyncControllerImpl() = default;
 
 rappor::RapporService* BackgroundSyncControllerImpl::GetRapporService() {
   return g_browser_process->rappor_service();
@@ skipping 10 matching lines @@
   rappor::SampleDomainAndRegistryFromGURL(
       GetRapporService(), "BackgroundSync.Register.Origin", origin);
 }
 
 void BackgroundSyncControllerImpl::RunInBackground(bool enabled,
                                                    int64_t min_ms) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
   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
 // https://crbug.com/484201.
 #endif
 }