Added AddDownload communication path and observers

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/common/background_fetch/background_fetch_types.h"
+#include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 
 namespace content {
 
 BackgroundFetchClientProxy::BackgroundFetchClientProxy(
+    BrowserContext* browser_context,
     BackgroundFetchContext* background_fetch_context)
-    : background_fetch_context_(background_fetch_context) {
+    : browser_context_(browser_context),
+      background_fetch_context_(background_fetch_context),
+      weak_ptr_factory_(this) {
   DCHECK(background_fetch_context);
+  DCHECK(browser_context);
 }
 
 BackgroundFetchClientProxy::~BackgroundFetchClientProxy() = default;
 
+base::WeakPtr<BackgroundFetchClientProxy>
+BackgroundFetchClientProxy::GetWeakPtr() {
+  return weak_ptr_factory_.GetWeakPtr();
+}
+
+void BackgroundFetchClientProxy::AddDownload(

[Peter Beverloo, 2017/04/20 16:27:55]

Why do we call this "Download"?

[harkness, 2017/04/21 14:55:20]

I went with Download because all of the BackgroundFetchClient::Delegate  methods
use Download. I'm happy to change it if you want to suggest something different.

[Peter Beverloo, 2017/04/24 14:39:14]

I mean, yeah, this feature is not just about downloads. Use "Fetch".

+    const std::string& registration_id,
+    const BackgroundFetchOptions& options) {
+  BackgroundFetchClient* client = browser_context_->GetBackgroundFetchClient();
+  if (client) {
+    client->AddDownload(registration_id, options.title,
+                        options.total_download_size);
+  }
+}
+
 void BackgroundFetchClientProxy::CleanupAllTasks() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
                           base::Bind(&BackgroundFetchContext::CleanupAllTasks,
                                      background_fetch_context_));
 }
 
 void BackgroundFetchClientProxy::CancelDownload(
     const std::string& registration_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
@@ skipping 36 matching lines @@
     return;
   }
 
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&BackgroundFetchContext::ResumeFetch,
                  background_fetch_context_, deserialized_id));
 }
 
 }  // namespace content