Menu hook plumbed through to RequestCoordinator.

components/offline_pages/background/request_coordinator.cc

Index: components/offline_pages/background/request_coordinator.cc
diff --git a/components/offline_pages/background/request_coordinator.cc b/components/offline_pages/background/request_coordinator.cc
index ae5a3a2e15d5115e0543581c4879c1db2bd3ef9c..781fbb2101c3bcfe0d9c365e3d82b105d2a5009f 100644
--- a/components/offline_pages/background/request_coordinator.cc
+++ b/components/offline_pages/background/request_coordinator.cc
@@ -9,6 +9,7 @@
 #include "components/offline_pages/background/offliner_factory.h"
 #include "components/offline_pages/background/offliner_policy.h"
 #include "components/offline_pages/background/save_page_request.h"
+#include "components/offline_pages/offline_page_item.h"
 
 namespace offline_pages {
 
@@ -23,7 +24,14 @@ RequestCoordinator::RequestCoordinator(
 
 RequestCoordinator::~RequestCoordinator() {}
 
-bool RequestCoordinator::SavePageLater(const SavePageRequest& request) {
+bool RequestCoordinator::SavePageLater(
+    const GURL url, const ClientId& client_id) {
+  DVLOG(0) << "URL is " << url << " " << __FUNCTION__;

[jianli, 2016/05/06 21:48:05]

nit: avoid adding this in production code

[Pete Williamson, 2016/05/06 23:56:27]

Done.

+
+  // TODO(petewil): We need a robust scheme for allocating new IDs.

[jianli, 2016/05/06 21:48:05]

Is this comment conflict with ID being passed in?

[Pete Williamson, 2016/05/06 23:56:27]

As it turns out, there are two IDs.
The ClientID is a pair of <namespace, id-unique-within-that-namespace>.
There is also a request ID.

The client (we don't have a real one yet, just a test hook) is responsible for
making sure it's requests have unique ids.

The RequestCoordinator is responsible for making sure that every request has a
unique requestID, which is what this part of the code is doing.  What I have now
is not very sophisticated, I hope to improve this before we are done (hence the
TODO).

+  // TODO(petewil): Build a SavePageRequest.
+  // TODO(petewil): Put the request on the request queue, nudge the scheduler.
+
   return true;
 }