Add skeletons for the factories to create the request coordinator and

chrome/browser/android/offline_pages/request_coordinator_factory.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 "chrome/browser/android/offline_pages/request_coordinator_factory.h"
+
+#include "base/memory/singleton.h"
+#include "chrome/browser/android/offline_pages/prerendering_offliner_factory.h"
+#include "components/keyed_service/content/browser_context_dependency_manager.h"
+#include "components/keyed_service/content/browser_context_dependency_manager.h"

[dougarnett, 2016/05/03 22:30:44]

dupe

[Pete Williamson, 2016/05/04 00:39:21]

Done.

+#include "components/offline_pages/background/offliner_factory.h"
+#include "components/offline_pages/background/offliner_policy.h"
+#include "components/offline_pages/background/request_coordinator.h"
+
+namespace offline_pages {
+
+RequestCoordinatorFactory::RequestCoordinatorFactory()
+    : BrowserContextKeyedServiceFactory(
+          "RequestCoordinator",

[fgorski, 2016/05/03 22:48:16]

Can we include the offline in the name?
request is an overloaded term in chromium
wile coordinator sounds much like manager...

[Pete Williamson, 2016/05/04 00:39:21]

Done.

+          BrowserContextDependencyManager::GetInstance()) {
+}
+
+// static
+RequestCoordinatorFactory* RequestCoordinatorFactory::GetInstance() {
+  return base::Singleton<RequestCoordinatorFactory>::get();
+}
+
+// static
+RequestCoordinator* RequestCoordinatorFactory::GetForBrowserContext(
+    content::BrowserContext* context) {
+  return static_cast<RequestCoordinator*>(
+      GetInstance()->GetServiceForBrowserContext(context, true));
+}
+
+KeyedService* RequestCoordinatorFactory::BuildServiceInstanceFor(
+    content::BrowserContext* context) const {
+  OfflinerPolicy* policy = new OfflinerPolicy();
+  OfflinerFactory* prerendererOffliner =
+      new PrerenderingOfflinerFactory(context);
+  // TODO(petewil) Add support for server based offliner when it is ready.
+
+  return new RequestCoordinator(policy, prerendererOffliner);
+}
+
+content::BrowserContext* RequestCoordinatorFactory::GetBrowserContextToUse(
+    content::BrowserContext* context) const {
+  // TODO(petewil): Implement.
+  return nullptr;
+}
+
+}  // namespace offline_pages