Added DownloadItem::Observer to JobController.

content/browser/background_fetch/background_fetch_context.h

 // 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_BROWSER_BACKGROUND_FETCH_BACKGROUND_FETCH_CONTEXT_H_
 #define CONTENT_BROWSER_BACKGROUND_FETCH_BACKGROUND_FETCH_CONTEXT_H_
 
 #include <unordered_map>
 
 #include "base/macros.h"
@@ skipping 23 matching lines @@
       StoragePartition* storage_partition,
       const scoped_refptr<ServiceWorkerContextWrapper>& context);
 
   // Init and Shutdown are for use on the UI thread when the StoragePartition is
   // being setup and torn down.
   void Init();
 
   // Shutdown must be called before deleting this. Call on the UI thread.
   void Shutdown();
 
+  // Called by the JobController to inform the context that the job is complete.
+  // TODO(harkness): Pass back enough information that the context can construct
+  // the Mojo response.
+  void JobComplete(const std::string& job_guid);

[Peter Beverloo, 2017/03/08 14:58:19]

Could we instead give the JobController a base::callback that it calls when a
job is complete? That would allow you to make this private, and would help
testing down the road too.

[harkness, 2017/03/10 13:33:53]

I agree with using callbacks, but this hadn't been hooked up at all, so I've
just removed it for now. When I add it back it will be with callbacky goodness.

+
   BackgroundFetchDataManager* GetDataManagerForTesting() {
     return &background_fetch_data_manager_;
   }
 
  private:
   void CreateRequest(const BackgroundFetchJobInfo& job_info,
                      std::vector<BackgroundFetchRequestInfo>& request_infos);
 
   void ShutdownOnIO();
 
   friend class base::RefCountedThreadSafe<BackgroundFetchContext>;
   ~BackgroundFetchContext();
 
   // The Context is owned by the BrowserContext via the StoragePartition.
   BrowserContext* browser_context_;
   StoragePartition* storage_partition_;
 
   scoped_refptr<ServiceWorkerContextWrapper> service_worker_context_;
   BackgroundFetchDataManager background_fetch_data_manager_;
 
   std::unordered_map<std::string, std::unique_ptr<BackgroundFetchJobController>>
       job_map_;
 
   DISALLOW_COPY_AND_ASSIGN(BackgroundFetchContext);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_BACKGROUND_FETCH_BACKGROUND_FETCH_CONTEXT_H_