Adding a BUILD file for both gyp and gn for the new background_offlining

components/offline_pages/background/request_coordinator.cc

Index: components/offline_pages/background/request_coordinator.cc
diff --git a/components/offline_pages/background/request_coordinator.cc b/components/offline_pages/background/request_coordinator.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1e443097b555a7248baa179725ad98478acc54c4
--- /dev/null
+++ b/components/offline_pages/background/request_coordinator.cc
@@ -0,0 +1,38 @@
+// 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"
+

[dougarnett, 2016/04/27 19:15:35]

nit - drop this empty line

[Pete Williamson, 2016/04/27 21:03:46]

There is supposed to be a blank line here.

Normal practice is to include the associated .h file first, and other headers
after a blank line.  The practice is meant to assure that each header can be
standalone, and there are no include order dependencies.

+#include "components/offline_pages/background/scheduler.h"
+
+// TODO: include file with SavePageRequest definition.
+
+namespace offline_pages {
+
+// TODO(dougarnett): How to inject Offliner factories and policy objects.
+RequestCoordinator::RequestCoordinator() {
+  // Do setup as needed.
+}
+
+RequestCoordinator::~RequestCoordinator() {
+  // Do cleanup as needed.
+}
+
+// Queues |request| to later load and save when system conditions allow.
+bool RequestCoordinator::SavePageLater(const SavePageRequest& request) {
+  return true;
+}
+
+// Starts processing of one or more queued save page later requests.

[dougarnett, 2016/04/27 19:15:35]

might be good to include rest of .h comment wrt contract in calling callback if
returns true

[Pete Williamson, 2016/04/27 21:03:46]

Done.

+bool RequestCoordinator::StartProcessing(
+    const ProcessingDoneCallback& callback) {
+  return true;

[dougarnett, 2016/04/27 19:15:35]

false until/unless this will call callback

[Pete Williamson, 2016/04/27 21:03:46]

Done.

+}
+
+// Stops the current request processing if active.
+void RequestCoordinator::StopProcessing() {
+
+}
+
+}  // namespace offline_pages