Create skeleton for the Prefetching store and initial pipeline step.

components/offline_pages/core/prefetch/prefetch_dispatcher_impl.h

 // Copyright 2017 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.
 
 #ifndef COMPONENTS_OFFLINE_PAGES_CORE_PREFETCH_PREFETCH_DISPATCHER_IMPL_H_
 #define COMPONENTS_OFFLINE_PAGES_CORE_PREFETCH_PREFETCH_DISPATCHER_IMPL_H_
 
+#include <memory>
+
 #include "base/macros.h"
 #include "components/offline_pages/core/prefetch/prefetch_dispatcher.h"
+#include "components/offline_pages/core/prefetch/prefetch_store.h"
+#include "components/offline_pages/core/task_queue.h"
 
 namespace offline_pages {
 
 class PrefetchDispatcherImpl : public PrefetchDispatcher {
  public:
-  PrefetchDispatcherImpl();
+  PrefetchDispatcherImpl(std::unique_ptr<PrefetchStore> store);
   ~PrefetchDispatcherImpl() override;
 
   // PrefetchDispatcher implementation:
   void AddCandidatePrefetchURLs(
-      const std::vector<PrefetchURL>& suggested_urls) override;
+      const std::string& name_space,
+      const std::vector<PrefetchURL>& prefetch_urls) override;
   void RemoveAllUnprocessedPrefetchURLs(const std::string& name_space) override;
-  void RemovePrefetchURLsByClientId(const ClientId& client_id) override;
+  void RemovePrefetchURLsByClientId(const std::string& name_space,
+                                    const std::string& client_id) override;
   void BeginBackgroundTask(std::unique_ptr<ScopedBackgroundTask> task) override;
   void StopBackgroundTask(ScopedBackgroundTask* task) override;
 
  private:
+  friend class PrefetchDispatcherTest;
+
+  std::unique_ptr<PrefetchStore> store_;

[Dmitry Titov, 2017/05/18 21:24:02]

Should these rather live on PrefetchService and be injected in its constructor?
The PrefetchDispatcher may just keep a pointer back to PrefetchService to
retrieve other subcomponents.

Sooner or later it will have such a pointer and will pull other subcomponents
form the Service, so it might be good to just adopt this pattern now.

[carlosk, 2017/05/24 20:57:26]

Done.

+  TaskQueue task_queue_;
+
   DISALLOW_COPY_AND_ASSIGN(PrefetchDispatcherImpl);
 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_CORE_PREFETCH_PREFETCH_DISPATCHER_IMPL_H_