[Offline pages] Implementation of OfflineBackgroundTask using background_task_scheduler

chrome/browser/android/offline_pages/background_scheduler_bridge.cc

 // Copyright 2016 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/android/offline_pages/background_scheduler_bridge.h"
 #include "base/android/callback_android.h"
 #include "base/android/scoped_java_ref.h"
-#include "chrome/browser/android/offline_pages/background_scheduler_bridge.h"
 #include "chrome/browser/android/offline_pages/offline_page_model_factory.h"
 #include "chrome/browser/android/offline_pages/request_coordinator_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "components/offline_pages/core/background/device_conditions.h"
+#include "components/offline_pages/core/background/offliner.h"
 #include "components/offline_pages/core/background/request_coordinator.h"
 #include "jni/BackgroundSchedulerBridge_jni.h"
 
 using base::android::JavaParamRef;
 using base::android::ScopedJavaGlobalRef;
 using base::android::ScopedJavaLocalRef;
 
 namespace offline_pages {
 namespace android {
 
@@ skipping 26 matching lines @@
       GetForBrowserContext(profile);
   DVLOG(2) << "resource_coordinator: " << coordinator;
   DeviceConditions device_conditions(
       j_power_connected, j_battery_percentage,
       static_cast<net::NetworkChangeNotifier::ConnectionType>(
           j_net_connection_type));
   return coordinator->StartScheduledProcessing(
       device_conditions, base::Bind(&ProcessingDoneCallback, j_callback_ref));
 }
 
+// JNI call to stop request processing in scheduled mode.
+static void StopScheduledProcessing(JNIEnv* env,
+                                    const JavaParamRef<jclass>& jcaller) {
+  Profile* profile = ProfileManager::GetLastUsedProfile();
+  RequestCoordinator* coordinator =
+      RequestCoordinatorFactory::GetInstance()->GetForBrowserContext(profile);
+  DVLOG(2) << "resource_coordinator: " << coordinator;
+  if (!coordinator)
+    return;
+  coordinator->StopProcessing(
+      Offliner::RequestStatus::BACKGROUND_SCHEDULER_CANCELED);

[Pete Williamson, 2017/02/28 20:17:49]

After thinking about it and looking at the current UMA, this looks good. 
Remember to update our Background Loading UMA pointers doc:
https://docs.google.com/document/d/1b__WjQPPnltJWHgCi-KoOC4ucEGRewQiCLPkM3CA-TE

[fgorski, 2017/02/28 21:24:09]

Done.

+}
+
 BackgroundSchedulerBridge::BackgroundSchedulerBridge() = default;
 
 BackgroundSchedulerBridge::~BackgroundSchedulerBridge() = default;
 
 void BackgroundSchedulerBridge::Schedule(
     const TriggerConditions& trigger_conditions) {
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jobject> j_conditions =
       CreateTriggerConditions(env, trigger_conditions.require_power_connected,
                               trigger_conditions.minimum_battery_percentage,
@@ skipping 47 matching lines @@
       power, battery, network_connection_type);
   return *device_conditions_;
 }
 
 bool RegisterBackgroundSchedulerBridge(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace android
 }  // namespace offline_pages