Connect BackgroundFetch to the OfflineItemCollection

content/browser/background_fetch/background_fetch_context.cc

 // 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.
 
 #include "content/browser/background_fetch/background_fetch_context.h"
 
 #include "base/memory/ptr_util.h"
+#include "content/browser/background_fetch/background_fetch_client_proxy.h"
 #include "content/browser/background_fetch/background_fetch_data_manager.h"
 #include "content/browser/background_fetch/background_fetch_job_controller.h"
 #include "content/browser/background_fetch/background_fetch_registration_id.h"
 #include "content/browser/service_worker/service_worker_context_wrapper.h"
 #include "content/browser/storage_partition_impl.h"
+#include "content/public/browser/background_fetch_client.h"
 #include "content/public/browser/browser_context.h"
 
 namespace content {
 
 BackgroundFetchContext::BackgroundFetchContext(
     BrowserContext* browser_context,
     StoragePartitionImpl* storage_partition,
     const scoped_refptr<ServiceWorkerContextWrapper>& service_worker_context)
     : browser_context_(browser_context),
       storage_partition_(storage_partition),
       service_worker_context_(service_worker_context),
       background_fetch_data_manager_(
-          base::MakeUnique<BackgroundFetchDataManager>(browser_context)) {
+          base::MakeUnique<BackgroundFetchDataManager>(browser_context)),
+      client_proxy_(base::MakeUnique<BackgroundFetchClientProxy>(this)) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  BackgroundFetchClient* client = browser_context_->GetBackgroundFetchClient();
+  if (client)
+    client->SetDelegate(client_proxy_.get());
 }
 
 BackgroundFetchContext::~BackgroundFetchContext() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
+
+  BackgroundFetchClient* client = browser_context_->GetBackgroundFetchClient();
+  if (client)
+    client->SetDelegate(nullptr);

[Peter Beverloo, 2017/03/31 01:32:23]

Thinking more about this, we should probably do this in Shutdown() as that
happens *before* the profile goes, where this happens *when* the profile goes,
so we may end up in an unfortunate situation where this actually recreates the
client.

[harkness, 2017/03/31 10:11:44]

Excellent point. Done.

 }
 
 void BackgroundFetchContext::Shutdown() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&BackgroundFetchContext::ShutdownOnIO, this));
 }
 
 void BackgroundFetchContext::ShutdownOnIO() {
@@ skipping 53 matching lines @@
 void BackgroundFetchContext::DidFinishFetch(
     const BackgroundFetchRegistrationId& registration_id) {
   DCHECK_GT(active_fetches_.count(registration_id), 0u);
 
   // TODO(peter): Dispatch the `backgroundfetched` or the `backgroundfetchfail`
   // event to the Service Worker.
 
   active_fetches_.erase(registration_id);
 }
 
+void BackgroundFetchContext::CancelFetch(
+    const BackgroundFetchRegistrationId& registration_id) {
+  // TODO(harkness): pass the command to the JobController for the given task.

[Peter Beverloo, 2017/03/31 01:32:23]

DCHECK_CURRENTLY_ON(BrowserThread::IO);

[harkness, 2017/03/31 10:11:44]

Done.

+}
+
+void BackgroundFetchContext::PauseFetch(
+    const BackgroundFetchRegistrationId& registration_id) {
+  // TODO(harkness): pass the command to the JobController for the given task.
+}
+
+void BackgroundFetchContext::ResumeFetch(
+    const BackgroundFetchRegistrationId& registration_id) {
+  // TODO(harkness): pass the command to the JobController for the given task.
+}
+
 }  // namespace content