Connect BackgroundFetch to the OfflineItemCollection

content/public/browser/background_fetch_client.h

Index: content/public/browser/background_fetch_client.h
diff --git a/content/public/browser/background_fetch_client.h b/content/public/browser/background_fetch_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..f85bc76dd16a09ebc6e46a5629ad0eb14fdbb2f6
--- /dev/null
+++ b/content/public/browser/background_fetch_client.h
@@ -0,0 +1,22 @@
+// 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 CONTENT_PUBLIC_BROWSER_BACKGROUND_FETCH_CLIENT_H_
+#define CONTENT_PUBLIC_BROWSER_BACKGROUND_FETCH_CLIENT_H_
+
+#include "content/common/content_export.h"
+
+namespace content {
+
+class BackgroundFetchDelegate;
+
+class CONTENT_EXPORT BackgroundFetchClient {

[Peter Beverloo, 2017/03/29 14:32:13]

++docs on this interface. Especially things like ownership of the pointers.

[harkness, 2017/03/30 12:42:36]

Done.

+ public:
+  virtual void SetDelegate(BackgroundFetchDelegate* delegate) = 0;
+  virtual BackgroundFetchDelegate* GetDelegate() const = 0;

[Peter Beverloo, 2017/03/29 14:32:13]

Why do we need GetDelegate()?

[harkness, 2017/03/30 12:42:36]

I thought we were going to need one, but we don't. Removed.

+};

[Peter Beverloo, 2017/03/29 14:32:13]

Virtual base classes need to have a virtual destructor. Mind defining one?

[harkness, 2017/03/30 12:42:36]

Done.

+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_BROWSER_BACKGROUND_FETCH_CLIENT_H_