Add skeletons for the factories to create the request coordinator and

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..fe768b11778e4b35648149f60987f85422035a62
--- /dev/null
+++ b/components/offline_pages/background/request_coordinator.cc
@@ -0,0 +1,36 @@
+// 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 "components/offline_pages/background/offliner_policy.h"
+#include "components/offline_pages/background/scheduler.h"
+
+namespace offline_pages {
+
+RequestCoordinator::RequestCoordinator(
+    OfflinerPolicy* policy, OfflinerFactory* factory) {
+  // Do setup as needed.
+  // TODO(petewil): Assert policy not null.
+  policy_ = policy;
+  factory_ = factory;
+}
+
+RequestCoordinator::~RequestCoordinator() {
+  delete policy_;

[fgorski, 2016/04/29 04:16:46]

use unique_ptr which solves this.

[Pete Williamson, 2016/05/02 20:51:14]

I didn't want to use unique_ptr yet, because I think I will need to give a
reference to the policy object to the offliner factory.  When I nail down the
offliner factory a bit more, I'll know what I can do here.

+}
+
+bool RequestCoordinator::SavePageLater(const SavePageRequest& request) {
+  return true;
+}
+
+bool RequestCoordinator::StartProcessing(
+    const ProcessingDoneCallback& callback) {
+  return false;
+}
+
+void RequestCoordinator::StopProcessing() {
+}
+
+}  // namespace offline_pages