Add skeletons for the factories to create the request coordinator and

components/offline_pages/background/request_coordinator.cc

 // 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"
 
 // TODO(fgorski): include file with SavePageRequest definition.

[fgorski, 2016/05/04 05:20:58]

I think you can resolve the TODO now.

[Pete Williamson, 2016/05/04 21:23:21]

Done, also moved added the save_page_request.h include.
(I made this same change in my other changelist, so I'll have to merge one of
these.)

+#include "components/offline_pages/background/offliner_policy.h"
 
 namespace offline_pages {
 
-// TODO(dougarnett): How to inject Offliner factories and policy objects.
-RequestCoordinator::RequestCoordinator() {
+RequestCoordinator::RequestCoordinator(
+    OfflinerPolicy* policy, OfflinerFactory* factory) {
   // Do setup as needed.
+  // TODO(petewil): Assert policy not null.
+  policy_ = policy;
+  factory_ = factory;
 }
 
 RequestCoordinator::~RequestCoordinator() {
-  // Do cleanup as needed.
+  delete policy_;
 }
 
 bool RequestCoordinator::SavePageLater(const SavePageRequest& request) {
   return true;
 }
 
 bool RequestCoordinator::StartProcessing(
     const ProcessingDoneCallback& callback) {
   return false;
 }
 
 void RequestCoordinator::StopProcessing() {
 }
 
 }  // namespace offline_pages