[Offline Pages] Fix official build failure (using clang) caused by unused variables.

chrome/browser/android/offline_pages/evaluation/offline_page_evaluation_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/evaluation/offline_page_evaluation_bridge.h"
 
 #include "base/android/callback_android.h"
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
+#include "base/memory/ptr_util.h"
+#include "base/sequenced_task_runner.h"
+#include "chrome/browser/android/offline_pages/background_scheduler_bridge.h"
+#include "chrome/browser/android/offline_pages/downloads/offline_page_notification_bridge.h"
+#include "chrome/browser/android/offline_pages/evaluation/evaluation_test_scheduler.h"
 #include "chrome/browser/android/offline_pages/offline_page_model_factory.h"
+#include "chrome/browser/android/offline_pages/prerendering_offliner_factory.h"
 #include "chrome/browser/android/offline_pages/request_coordinator_factory.h"
+#include "chrome/browser/net/nqe/ui_network_quality_estimator_service.h"
+#include "chrome/browser/net/nqe/ui_network_quality_estimator_service_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_android.h"
+#include "chrome/common/chrome_constants.h"
+#include "components/offline_pages/background/offliner_factory.h"
+#include "components/offline_pages/background/offliner_policy.h"
 #include "components/offline_pages/background/request_coordinator.h"
 #include "components/offline_pages/background/request_notifier.h"
+#include "components/offline_pages/background/request_queue.h"
+#include "components/offline_pages/background/request_queue_store.h"
+#include "components/offline_pages/background/request_queue_store_sql.h"
 #include "components/offline_pages/background/save_page_request.h"
+#include "components/offline_pages/downloads/download_notifying_observer.h"
 #include "components/offline_pages/offline_page_item.h"
 #include "components/offline_pages/offline_page_model.h"
 #include "content/public/browser/browser_context.h"
+#include "content/public/browser/browser_thread.h"
 #include "jni/OfflinePageEvaluationBridge_jni.h"
 #include "jni/SavePageRequest_jni.h"
 
 using base::android::ConvertJavaStringToUTF8;
 using base::android::ConvertUTF8ToJavaString;
 using base::android::JavaParamRef;
 using base::android::ScopedJavaGlobalRef;
 using base::android::ScopedJavaLocalRef;
 
 namespace offline_pages {
 namespace android {
 
 namespace {
+const bool kPreferUntriedRequest = false;
+const bool kPreferEarlierRequest = true;
+const bool kPreferRetryCountOverRecency = false;
+const int kMaxStartedTries = 4;
+const int kMaxCompletedTries = 1;
+const int kImmediateRequestExpirationTimeInSeconds = 3600;
 
 void ToJavaOfflinePageList(JNIEnv* env,
                            jobject j_result_obj,
                            const std::vector<OfflinePageItem>& offline_pages) {
   for (const OfflinePageItem& offline_page : offline_pages) {
     Java_OfflinePageEvaluationBridge_createOfflinePageAndAddToList(
         env, j_result_obj,
         ConvertUTF8ToJavaString(env, offline_page.url.spec()),
         offline_page.offline_id,
         ConvertUTF8ToJavaString(env, offline_page.client_id.name_space),
@@ skipping 38 matching lines @@
   Profile* profile = ProfileAndroid::FromProfileAndroid(j_profile);
 
   OfflinePageModel* offline_page_model =
       OfflinePageModelFactory::GetForBrowserContext(profile);
 
   RequestCoordinator* request_coordinator = nullptr;
 
   if (j_use_evaluation_scheduler) {
     request_coordinator = static_cast<RequestCoordinator*>(
         RequestCoordinatorFactory::GetInstance()->SetTestingFactoryAndUse(
-            profile, &RequestCoordinatorFactory::GetTestingFactory));
+            profile,
+            &OfflinePageEvaluationBridge::GetTestingRequestCoordinator));
   } else {
     request_coordinator =
         RequestCoordinatorFactory::GetForBrowserContext(profile);
   }
 
   if (offline_page_model == nullptr || request_coordinator == nullptr)
     return ScopedJavaLocalRef<jobject>();
 
   OfflinePageEvaluationBridge* bridge = new OfflinePageEvaluationBridge(
       env, profile, offline_page_model, request_coordinator);
 
   return ScopedJavaLocalRef<jobject>(bridge->java_ref());
 }
 
 // static
 bool OfflinePageEvaluationBridge::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
+// static
+std::unique_ptr<KeyedService>
+OfflinePageEvaluationBridge::GetTestingRequestCoordinator(
+    content::BrowserContext* context) {
+  // Create a new OfflinerPolicy with a larger background processing
+  // budget (3600 sec). Other values are the same with default ones.
+  std::unique_ptr<OfflinerPolicy> policy(new OfflinerPolicy(
+      kPreferUntriedRequest, kPreferEarlierRequest,
+      kPreferRetryCountOverRecency, kMaxStartedTries, kMaxCompletedTries,
+      kImmediateRequestExpirationTimeInSeconds));
+  std::unique_ptr<OfflinerFactory> prerenderer_offliner(
+      new PrerenderingOfflinerFactory(context));
+
+  scoped_refptr<base::SequencedTaskRunner> background_task_runner =
+      content::BrowserThread::GetBlockingPool()->GetSequencedTaskRunner(
+          content::BrowserThread::GetBlockingPool()->GetSequenceToken());
+  Profile* profile = Profile::FromBrowserContext(context);
+  base::FilePath queue_store_path =
+      profile->GetPath().Append(chrome::kOfflinePageRequestQueueDirname);
+
+  std::unique_ptr<RequestQueueStoreSQL> queue_store(
+      new RequestQueueStoreSQL(background_task_runner, queue_store_path));
+  std::unique_ptr<RequestQueue> queue(new RequestQueue(std::move(queue_store)));
+  std::unique_ptr<android::EvaluationTestScheduler> scheduler(
+      new android::EvaluationTestScheduler());
+  net::NetworkQualityEstimator::NetworkQualityProvider*
+      network_quality_estimator =
+          UINetworkQualityEstimatorServiceFactory::GetForProfile(profile);
+  // TODO(fgorski): Something needs to keep the handle to the Notification
+  // dispatcher.
+  std::unique_ptr<RequestCoordinator> request_coordinator =
+      base::MakeUnique<RequestCoordinator>(
+          std::move(policy), std::move(prerenderer_offliner), std::move(queue),
+          std::move(scheduler), network_quality_estimator);
+  request_coordinator->SetImmediateScheduleCallbackForTest(
+      base::Bind(&android::EvaluationTestScheduler::ImmediateScheduleCallback,
+                 base::Unretained(scheduler.get())));
+
+  DownloadNotifyingObserver::CreateAndStartObserving(
+      request_coordinator.get(),
+      base::MakeUnique<android::OfflinePageNotificationBridge>());
+
+  return std::move(request_coordinator);
+}
+
 OfflinePageEvaluationBridge::OfflinePageEvaluationBridge(
     JNIEnv* env,
     content::BrowserContext* browser_context,
     OfflinePageModel* offline_page_model,
     RequestCoordinator* request_coordinator)
     : browser_context_(browser_context),
       offline_page_model_(offline_page_model),
       request_coordinator_(request_coordinator) {
   java_ref_.Reset(Java_OfflinePageEvaluationBridge_create(
       env, reinterpret_cast<jlong>(this)));
@@ skipping 94 matching lines @@
 
 void OfflinePageEvaluationBridge::NotifyIfDoneLoading() const {
   if (!offline_page_model_->is_loaded())
     return;
   JNIEnv* env = base::android::AttachCurrentThread();
   Java_OfflinePageEvaluationBridge_offlinePageModelLoaded(env, java_ref_);
 }
 
 }  // namespace android
 }  // namespace offline_pages