Wireframe scheduler implementation

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 <memory>
 
 #include "base/memory/singleton.h"
 #include "chrome/browser/android/offline_pages/background_scheduler_bridge.h"
@@ skipping 24 matching lines @@
     content::BrowserContext* context) {
   return static_cast<RequestCoordinator*>(
       GetInstance()->GetServiceForBrowserContext(context, true));
 }
 
 KeyedService* RequestCoordinatorFactory::BuildServiceInstanceFor(
     content::BrowserContext* context) const {
   std::unique_ptr<OfflinerPolicy> policy(new OfflinerPolicy());
   std::unique_ptr<OfflinerFactory> prerenderer_offliner(
       new PrerenderingOfflinerFactory(context));
+  // TODO(petewil) Add support for server based offliner when it is ready.

[dewittj, 2016/05/24 16:56:19]

I don't think this comment is really necessary here.

[Pete Williamson, 2016/05/24 18:49:32]

As I recall, I was asked to add it in a previous code review so we don't forget
about a server based offliner.  Since we aren't developing the server based
offliner, I could still remove it.

   std::unique_ptr<RequestQueueInMemoryStore> store(
       new RequestQueueInMemoryStore());
   std::unique_ptr<RequestQueue> queue(new RequestQueue(std::move(store)));
-  // TODO(petewil) Add support for server based offliner when it is ready.
-
+  // Create the Bridge from the C++ side (and it will initialize from the Java

[dewittj, 2016/05/24 16:56:19]

I think this is out of date now.

[Pete Williamson, 2016/05/24 18:49:32]

Done.

+  // side).
   std::unique_ptr<Scheduler>
       scheduler(new android::BackgroundSchedulerBridge());
-  // TODO(petewil) Add support for server based offliner when it is ready.
 
   return new RequestCoordinator(std::move(policy),
                                 std::move(prerenderer_offliner),
                                 std::move(queue),
                                 std::move(scheduler));
 }
 
 content::BrowserContext* RequestCoordinatorFactory::GetBrowserContextToUse(
     content::BrowserContext* context) const {
   // TODO(petewil): Make sure we support incognito properly.
   return context;
 }
 
 }  // namespace offline_pages