Add the request picker and a unit test

components/offline_pages/background/request_picker.cc

Index: components/offline_pages/background/request_picker.cc
diff --git a/components/offline_pages/background/request_picker.cc b/components/offline_pages/background/request_picker.cc
new file mode 100644
index 0000000000000000000000000000000000000000..401697211bde203fb748e34a1f0db1af534cbeec
--- /dev/null
+++ b/components/offline_pages/background/request_picker.cc
@@ -0,0 +1,47 @@
+// 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_picker.h"
+
+#include "base/bind.h"
+#include "base/logging.h"
+#include "components/offline_pages/background/save_page_request.h"
+
+namespace offline_pages {
+
+RequestPicker::RequestPicker(
+    RequestQueue* requestQueue,
+    RequestCoordinator::RequestPickedCallback picked_callback,
+    RequestCoordinator::RequestQueueEmptyCallback empty_callback)
+    : queue_(requestQueue),
+      picked_callback_(picked_callback),
+      empty_callback_(empty_callback),
+      weak_ptr_factory_(this) {}
+
+RequestPicker::~RequestPicker() {}
+
+void RequestPicker::ChooseNextRequest() {
+  // Get all requests from queue (there is no filtering mechanism).
+  queue_->GetRequests(base::Bind(&RequestPicker::GetRequestResultCallback,
+                                 weak_ptr_factory_.GetWeakPtr()));
+}
+
+void RequestPicker::GetRequestResultCallback(
+    RequestQueue::GetRequestsResult,
+    const std::vector<SavePageRequest>& requests) {
+  // If there is nothing to do, return right away.
+  if (requests.size() == 0) {
+    empty_callback_.Run();
+    return;
+  }
+
+  // Pick the most deserving request for our conditions.
+  const SavePageRequest& picked_request = requests[0];

[chili, 2016/05/27 23:54:13]

Should we have a TODO here to actually check all the conditions and 'manually'
filter? or are we going to be guaranteed in later RequestQueue cycles to add
filtering mechanism natively?

[Pete Williamson, 2016/05/28 00:08:52]

The whole purpose of this class is to make a more complicated decision, so there
is not much danger of forgetting to come back and fix it.  However, LMK if you
like, and I can add a TODO.

+
+  // When we have a best request to try next, get the request coodinator to
+  // start it.
+  picked_callback_.Run(picked_request);
+}
+
+}  // namespace offline_pages