Move user requested trigger conditions into policy object.

components/offline_pages/background/request_coordinator.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 "components/offline_pages/background/request_coordinator.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 11 matching lines @@
 namespace {
 
 // Records the final request status UMA for an offlining request. This should
 // only be called once per Offliner::LoadAndSave request.
 void RecordOfflinerResultUMA(Offliner::RequestStatus request_status) {
   UMA_HISTOGRAM_ENUMERATION("OfflinePages.Background.OfflinerRequestStatus",
                             request_status,
                             Offliner::RequestStatus::STATUS_COUNT);
 }
 
-// TODO(dougarnett/petewil): Move to Policy object. Also consider lower minimum
-// battery percentage once there is some processing time limits in place.
-const Scheduler::TriggerConditions kUserRequestTriggerConditions(
-    false /* require_power_connected */,
-    50 /* minimum_battery_percentage */,
-    false /* require_unmetered_network */);
-
 // Timeout is 2.5 minutes based on the size of Marshmallow doze mode
 // maintenance window (3 minutes)
 // TODO(petewil): Find the optimal timeout based on data for 2G connections and
 // common EM websites. crbug.com/625204
 // TODO(petewil): Move this value into the policy object.
 long OFFLINER_TIMEOUT_SECONDS = 150;
 
 }  // namespace
 
 RequestCoordinator::RequestCoordinator(std::unique_ptr<OfflinerPolicy> policy,
@@ skipping 178 matching lines @@
         updated_request,
         base::Bind(&RequestCoordinator::UpdateRequestCallback,
                    weak_ptr_factory_.GetWeakPtr()));
   }
 
   // TODO(petewil): Check time budget. Return to the scheduler if we are out.
   // Start another request if we have time.
   TryNextRequest();
 }
 
-const Scheduler::TriggerConditions&
+const Scheduler::TriggerConditions
 RequestCoordinator::GetTriggerConditionsForUserRequest() {
-  return kUserRequestTriggerConditions;
+  Scheduler::TriggerConditions trigger_conditions(
+      policy_->PowerRequiredForUserRequestedPage(),
+      policy_->BatteryPercentageRequiredForUserRequestedPage(),
+      policy_->UnmeteredNetworkRequiredForUserRequestedPage());
+  return trigger_conditions;
 }
 
 void RequestCoordinator::GetOffliner() {
   if (!offliner_) {
     offliner_ = factory_->GetOffliner(policy_.get());
   }
 }
 
 }  // namespace offline_pages