Wireframe scheduler implementation

components/offline_pages/background/request_coordinator.h

Index: components/offline_pages/background/request_coordinator.h
diff --git a/components/offline_pages/background/request_coordinator.h b/components/offline_pages/background/request_coordinator.h
index 4ab1e5a0680833a31c30f79eec2e1cec28cbb932..d2da172fd3bce4b17639a73c872ed9a62b7c9743 100644
--- a/components/offline_pages/background/request_coordinator.h
+++ b/components/offline_pages/background/request_coordinator.h
@@ -57,7 +57,8 @@ class RequestCoordinator :
   // Returns the request queue used for requests.  Coordinator keeps ownership.
   RequestQueue* GetQueue() { return queue_.get(); }
 
-  Scheduler* GetSchedulerForTesting() { return scheduler_.get(); }
+  // Return an unowned pointer to the Scheduler.
+  Scheduler* GetScheduler() { return scheduler_.get(); }

[dewittj, 2016/05/25 23:16:45]

I think that any functions that are inlined like this should be lower_case(), so
s/GetScheduler/scheduler/

Also should do GetQueue() -> queue()

[Pete Williamson, 2016/05/25 23:59:58]

Done.

 
   // Returns the status of the most recent offlining.
   Offliner::CompletionStatus last_offlining_status() {