Split the RequestPicker task into two separate tasks.

components/offline_pages/background/cleanup_task_factory.h

+// 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.
+
+#ifndef COMPONENTS_OFFLINE_PAGES_BACKGROUND_CLEANUP_TASK_FACTORY_H_
+#define COMPONENTS_OFFLINE_PAGES_BACKGROUND_CLEANUP_TASK_FACTORY_H_
+
+#include <stdint.h>
+
+#include <set>
+
+#include "components/offline_pages/background/cleanup_task.h"
+
+namespace offline_pages {
+
+class OfflinerPolicy;
+class RequestCoordinatorEventLogger;
+class RequestNotifier;
+class RequestQueueStore;
+
+// TODO: Should I rename this since it now has two uses?

[dougarnett, 2016/12/02 17:20:26]

Think just one at the moment

[fgorski, 2016/12/02 21:44:04]

What do you mean by 2 uses?

[Pete Williamson, 2016/12/05 20:39:45]

Oops, this is an old comment, I refactored so that there is only one factory
now, CleanupTask has one, PickerTask does not.  Comment removed.

+class CleanupTaskFactory {
+ public:
+  CleanupTaskFactory(OfflinerPolicy* policy,
+                     RequestNotifier* notifier,
+                     RequestCoordinatorEventLogger* event_logger);
+
+  ~CleanupTaskFactory();
+
+  std::unique_ptr<CleanupTask> CreatePickerTask(RequestQueueStore* store);

[dougarnett, 2016/12/02 17:20:26]

Remove this declaration, right? 

[fgorski, 2016/12/02 21:44:04]

+1

[Pete Williamson, 2016/12/05 20:39:45]

Done.

+
+  std::unique_ptr<CleanupTask> CreateCleanupTask(RequestQueueStore* store);
+
+ private:
+  // Unowned pointer to the Policy
+  OfflinerPolicy* policy_;
+  // Unowned pointer to the notifier
+  RequestNotifier* notifier_;
+  // Unowned pointer to the EventLogger
+  RequestCoordinatorEventLogger* event_logger_;
+};
+
+}  // namespace offline_pages
+
+#endif  // COMPONENTS_OFFLINE_PAGES_BACKGROUND_CLEANUP_TASK_FACTORY_H_