Connect BackgroundFetch to the OfflineItemCollection

content/browser/background_fetch/background_fetch_client_proxy.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_client_proxy.h"
+
+#include "content/browser/background_fetch/background_fetch_context.h"
+#include "content/browser/background_fetch/background_fetch_registration_id.h"
+#include "content/public/browser/browser_thread.h"
+
+namespace content {
+
+BackgroundFetchClientProxy::BackgroundFetchClientProxy(
+    BackgroundFetchContext* background_fetch_context)
+    : background_fetch_context_(background_fetch_context) {
+  DCHECK(background_fetch_context);
+}
+
+BackgroundFetchClientProxy::~BackgroundFetchClientProxy() = default;
+
+void BackgroundFetchClientProxy::CancelDownload(
+    const std::string& registration_id) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  BackgroundFetchRegistrationId deserialized_id;
+  if (!BackgroundFetchRegistrationId::Deserialize(registration_id,
+                                                  deserialized_id))
+    return;

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

nit: {} (condition is multiple lines, elsewhere too)

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

LOG(ERROR)? This shouldn't crash, but it also shouldn't happen. Potentially
we'll want UMA for this at some point, too.

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

I added a log without printing the string. I wasn't sure of the rules about the
content/embedder boundary.

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

Ah git cl format, you fail me!

+
+  BrowserThread::PostTask(
+      BrowserThread::IO, FROM_HERE,
+      base::Bind(&BackgroundFetchContext::CancelFetch,
+                 background_fetch_context_, deserialized_id));

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

I guess we're getting scoped_refptr lifetime semantics for the callback anyway
because it recognizes that the type is refcounted?

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

That's what I was counting on. Do you want me to make it explicit?

+}
+
+void BackgroundFetchClientProxy::PauseDownload(
+    const std::string& registration_id) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  BackgroundFetchRegistrationId deserialized_id;
+  if (!BackgroundFetchRegistrationId::Deserialize(registration_id,
+                                                  deserialized_id))
+    return;
+
+  BrowserThread::PostTask(
+      BrowserThread::IO, FROM_HERE,
+      base::Bind(&BackgroundFetchContext::PauseFetch, background_fetch_context_,
+                 deserialized_id));
+}
+
+void BackgroundFetchClientProxy::ResumeDownload(
+    const std::string& registration_id) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  BackgroundFetchRegistrationId deserialized_id;
+  if (!BackgroundFetchRegistrationId::Deserialize(registration_id,
+                                                  deserialized_id))
+    return;
+
+  BrowserThread::PostTask(
+      BrowserThread::IO, FROM_HERE,
+      base::Bind(&BackgroundFetchContext::ResumeFetch,
+                 background_fetch_context_, deserialized_id));
+}
+
+}  // namespace content