Add unit test for request coordiator

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 47991061dba95200a826cfac6abb9289ce0d0736..641926f1cc05ea50a8a3f570e38179ce5ae76813 100644
--- a/components/offline_pages/background/request_coordinator.h
+++ b/components/offline_pages/background/request_coordinator.h
@@ -6,6 +6,7 @@
 #define COMPONENTS_OFFLINE_PAGES_BACKGROUND_REQUEST_COORDINATOR_H_
 
 #include "base/callback.h"
+#include "components/offline_pages/background/save_page_request.h"

[fgorski, 2016/05/03 22:55:49]

forward declare instead and add the .h file in .cc

[Pete Williamson, 2016/05/04 17:29:27]

Done.

 
 namespace offline_pages {
 
@@ -15,10 +16,6 @@ class RequestCoordinator {
   // Callback to report when the processing of a triggered task is complete.
   typedef base::Callback<void()> ProcessingDoneCallback;
 
-  struct SavePageRequest {
-    // TODO(dougarnett): define and consider making stand-alone.
-  };
-
   // TODO(dougarnett): How to inject Offliner factories and policy objects.
   RequestCoordinator();