Move user requested trigger conditions into policy object.

components/offline_pages/background/offliner_policy.h

Index: components/offline_pages/background/offliner_policy.h
diff --git a/components/offline_pages/background/offliner_policy.h b/components/offline_pages/background/offliner_policy.h
index ee2cc0db615b05a614ff0ce23379ac3e1fa54196..6e1aed34426e30fbdccbc71a2ecfaba86c4e8b6a 100644
--- a/components/offline_pages/background/offliner_policy.h
+++ b/components/offline_pages/background/offliner_policy.h
@@ -38,6 +38,21 @@ class OfflinerPolicy {
   // The max number of times we will retry a request.
   int GetMaxRetries() { return kMaxRetries; }
 
+  bool PowerRequiredForUserRequestedPage() { return false; }

[fgorski, 2016/07/26 16:42:14]

all these should be const methods top to bottom (this applies to methods outside
of this patch as well, but I believe you are already changing these in the other
patch.

[Pete Williamson, 2016/07/26 19:39:34]

Done (I changed the ones not already changed in the other patch).

+
+  bool PowerRequiredForNonUserRequestedPage() { return true; }
+
+  bool UnmeteredNetworkRequiredForUserRequestedPage() { return false; }
+
+  bool UnmeteredNetworkRequiredForNonUserRequestedPage() { return true; }
+
+  int BatteryPercentageRequiredForUserRequestedPage() { return 50; }
+
+  // This is so low because we require the device to be plugged in and charging.
+  // If we decide to allow non-user requested pages when not plugged in, we
+  // should raise this somewhat higher.
+  int BatteryPercentageRequiredForNonUserRequestedPage() { return 25; }
+
   // How many seconds to keep trying new pages for, before we give up,  and
   // return to the scheduler.
   int GetBackgroundProcessingTimeBudgetSeconds() {