Menu hook plumbed through to RequestCoordinator.

components/offline_pages/background/request_coordinator.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 "components/offline_pages/background/request_coordinator.h"
 
 #include <utility>
 
 #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 {
 
 RequestCoordinator::RequestCoordinator(
     std::unique_ptr<OfflinerPolicy> policy,
     std::unique_ptr<OfflinerFactory> factory) {
   // Do setup as needed.
   // TODO(petewil): Assert policy not null.
   policy_ = std::move(policy);
   factory_ = std::move(factory);
 }
 
 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;
 }
 
 bool RequestCoordinator::StartProcessing(
     const ProcessingDoneCallback& callback) {
   return false;
 }
 
 void RequestCoordinator::StopProcessing() {
 }
 
 }  // namespace offline_pages