Connect BackgroundFetch to the OfflineItemCollection

chrome/browser/background_fetch/background_fetch_client_impl.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 <sstream>
+#include <string>
+
+#include "chrome/browser/background_fetch/background_fetch_client_impl.h"
+
+#include "chrome/browser/offline_items_collection/offline_content_aggregator_factory.h"
+#include "chrome/browser/profiles/profile.h"
+#include "components/offline_items_collection/core/offline_content_aggregator.h"
+#include "components/offline_items_collection/core/offline_item.h"
+
+namespace {
+
+const char kBackgroundFetchNamespace[] = "BackgroundFetchNamespace";
+
+}  // namespace
+
+using offline_items_collection::ContentId;
+using offline_items_collection::OfflineContentAggregator;
+using offline_items_collection::OfflineContentAggregatorFactory;
+using offline_items_collection::OfflineContentProvider;
+using offline_items_collection::OfflineItem;
+
+BackgroundFetchClientImpl::BackgroundFetchClientImpl(Profile* profile)
+    : profile_(profile), namespace_(kBackgroundFetchNamespace) {
+  DCHECK(profile_);
+  if (profile->IsOffTheRecord()) {
+    // There can be multiple incognito namespaces, so append the memory address
+    // of this client to provide a guaranteed unique namespace.
+    std::ostringstream suffix;
+    suffix << "Incognito" << this;
+    namespace_ += suffix.str();
+  }
+
+  OfflineContentAggregator* aggregator =
+      OfflineContentAggregatorFactory::GetForBrowserContext(profile_);
+  if (!aggregator)
+    return;
+  aggregator->RegisterProvider(namespace_, this);
+}
+
+BackgroundFetchClientImpl::~BackgroundFetchClientImpl() = default;
+
+void BackgroundFetchClientImpl::Shutdown() {
+  OfflineContentAggregator* aggregator =
+      OfflineContentAggregatorFactory::GetForBrowserContext(profile_);
+  if (!aggregator)
+    return;
+
+  aggregator->UnregisterProvider(namespace_);
+
+  // If the profile is an incognito profile, clean up all data.
+  if (delegate_ && namespace_ == kBackgroundFetchNamespace)

[Peter Beverloo, 2017/04/19 12:50:19]

!=  (+test maybe, since this is pretty critical?)

[harkness, 2017/04/19 13:59:42]

Done, thanks for the catch. Also added tests.

+    delegate_->CleanupAllTasks();
+}
+
+void BackgroundFetchClientImpl::SetDelegate(
+    content::BackgroundFetchClient::Delegate* delegate) {
+  delegate_ = delegate;
+}
+
+void BackgroundFetchClientImpl::CancelDownload(const ContentId& content_id) {
+  DCHECK_EQ(content_id.name_space, namespace_);
+
+  if (!delegate_)
+    return;
+
+  delegate_->CancelDownload(content_id.id);
+}
+
+void BackgroundFetchClientImpl::PauseDownload(const ContentId& content_id) {
+  DCHECK_EQ(content_id.name_space, namespace_);
+
+  if (!delegate_)
+    return;
+
+  delegate_->PauseDownload(content_id.id);
+}
+
+void BackgroundFetchClientImpl::ResumeDownload(const ContentId& content_id) {
+  DCHECK_EQ(content_id.name_space, namespace_);
+
+  if (!delegate_)
+    return;
+
+  delegate_->ResumeDownload(content_id.id);
+}
+
+void BackgroundFetchClientImpl::OpenItem(const ContentId& content_id) {
+  // TODO(harkness): Add another call to BackgroundFetchClient::Delegate and
+  // plumb this to 'onclickevent' to the ServiceWorker.
+}
+
+void BackgroundFetchClientImpl::GetVisualsForItem(
+    const ContentId& id,
+    const VisualsCallback& callback) {
+  // TODO(harkness): Update with icons.
+  callback.Run(id, nullptr);
+}
+
+void BackgroundFetchClientImpl::AddObserver(Observer* observer) {
+  // TODO(harkness): Add a path for the OfflineContentProvider to observe
+  // changes in available BackgroundFetch items.
+}
+
+void BackgroundFetchClientImpl::RemoveObserver(Observer* observer) {}
+
+bool BackgroundFetchClientImpl::AreItemsAvailable() {
+  // TODO(harkness): Follow up with dtrainor about what action to take for this.
+  return false;
+}
+
+void BackgroundFetchClientImpl::RemoveItem(const ContentId& content_id) {
+  // Not applicable for Background Fetch, since offline items don't exist in the
+  // offline content system beyond the duration of the download.
+}
+
+const OfflineItem* BackgroundFetchClientImpl::GetItemById(
+    const ContentId& content_id) {
+  // TODO(harkness): Follow up with dtrainor about what action to take for this.
+  return nullptr;
+}
+
+OfflineContentProvider::OfflineItemList
+BackgroundFetchClientImpl::GetAllItems() {
+  // TODO(harkness): Follow up with dtrainor about what action to take for this.
+  return OfflineContentProvider::OfflineItemList();
+}