Hook up the other Background Fetch Mojo methods with the system

content/browser/background_fetch/background_fetch_job_controller_unittest.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_job_controller.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind_helpers.h"
@@ skipping 137 matching lines @@
     // BrowserContext will take care of deallocating it.
     BrowserContext::SetDownloadManagerForTesting(&browser_context_,
                                                  download_manager_);
   }
 
   void TearDown() override { job_controller_->Shutdown(); }
 
   void InitializeJobController(
       const BackgroundFetchRegistrationId& registration_id) {
     job_controller_ = base::MakeUnique<BackgroundFetchJobController>(
-        registration_id, &browser_context_,
+        registration_id, BackgroundFetchOptions(), &browser_context_,
         BrowserContext::GetDefaultStoragePartition(&browser_context_),
         data_manager_.get(),
         base::BindOnce(&BackgroundFetchJobControllerTest::DidCompleteJob,
                        base::Unretained(this), registration_id));
   }
 
   void DidCompleteJob(
+      const BackgroundFetchRegistrationId& original_registration_id,
       const BackgroundFetchRegistrationId& registration_id,
-      const BackgroundFetchRegistrationId& original_registration_id) {
+      bool aborted_by_developer) {
     ASSERT_EQ(registration_id, original_registration_id);
     did_complete_job_ = true;
+    did_abort_by_developer_ = true;

[harkness, 2017/03/27 16:34:12]

Shouldn't this be did_abort_by_developer_ = aborted_by_developer?

[Peter Beverloo, 2017/03/28 13:34:22]

Done.

   }
 
   void StartProcessing() {
     base::RunLoop run_loop;
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         base::Bind(&BackgroundFetchJobControllerTest::StartProcessingOnIO,
                    base::Unretained(this), run_loop.QuitClosure()));
     run_loop.Run();
   }
 
   void StartProcessingOnIO(const base::Closure& closure) {
     job_controller_->StartProcessing();
     BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, closure);
   }
 
   BackgroundFetchJobController* job_controller() {
     return job_controller_.get();
   }
   MockDownloadManagerWithCallback* download_manager() {
     return download_manager_;
   }
 
   DownloadItem::Observer* ItemObserver() const { return job_controller_.get(); }
 
   bool did_complete_job() const { return did_complete_job_; }
+  bool did_abort_by_developer() const { return did_abort_by_developer_; }
 
   FakeBackgroundFetchDataManager* data_manager() const {
     return data_manager_.get();
   }
 
  private:
   bool did_complete_job_ = false;
+  bool did_abort_by_developer_ = false;
   TestBrowserThreadBundle thread_bundle_;
   TestBrowserContext browser_context_;
   std::unique_ptr<FakeBackgroundFetchDataManager> data_manager_;
   std::unique_ptr<BackgroundFetchJobController> job_controller_;
   MockDownloadManagerWithCallback* download_manager_;
 };
 
 TEST_F(BackgroundFetchJobControllerTest, SingleRequestJob) {
   BackgroundFetchRegistrationId registration_id(
       kServiceWorkerRegistrationId, url::Origin(GURL(kOrigin)), kTag);
@@ skipping 20 matching lines @@
   EXPECT_EQ(DownloadItem::DownloadState::IN_PROGRESS, request_info.state());
   EXPECT_FALSE(did_complete_job());
 
   // Update the item to be completed then update the observer. The JobController
   // should update the JobData that the request is complete.
   item->SetState(DownloadItem::DownloadState::COMPLETE);
   ItemObserver()->OnDownloadUpdated(item);
 
   EXPECT_EQ(DownloadItem::DownloadState::COMPLETE, request_info.state());
   EXPECT_TRUE(did_complete_job());
+  EXPECT_FALSE(did_abort_by_developer());
 }
 
 TEST_F(BackgroundFetchJobControllerTest, MultipleRequestJob) {
   BackgroundFetchRegistrationId registration_id(
       kServiceWorkerRegistrationId, url::Origin(GURL(kOrigin)), kTag);
 
   std::vector<BackgroundFetchRequestInfo> request_infos;
   for (int i = 0; i < 10; i++) {
     request_infos.emplace_back(GURL(kTestUrl), base::IntToString(i));
   }
@@ skipping 32 matching lines @@
   EXPECT_FALSE(did_complete_job());
 
   // Finally, update the last request to be complete. The JobController should
   // see that there are no more requests and mark the job as done.
   ASSERT_EQ(10U, download_items.size());
   item = download_items[9].get();
   item->SetState(DownloadItem::DownloadState::COMPLETE);
   ItemObserver()->OnDownloadUpdated(item);
 
   EXPECT_TRUE(did_complete_job());
+  EXPECT_FALSE(did_abort_by_developer());
 }
 
 TEST_F(BackgroundFetchJobControllerTest, UpdateStorageState) {
   BackgroundFetchRegistrationId registration_id(
       kServiceWorkerRegistrationId, url::Origin(GURL(kOrigin)), kTag);
 
   BackgroundFetchRequestInfo request_info(GURL(kTestUrl), kJobGuid);
   request_info.set_state(DownloadItem::DownloadState::IN_PROGRESS);
   data_manager()->set_next_request(&request_info);
   InitializeJobController(registration_id);
@@ skipping 14 matching lines @@
   item->SetReceivedBytes(123);
   item->SetState(DownloadItem::DownloadState::COMPLETE);
 
   // Trigger the observer. The JobController should update the JobData that the
   // request is complete and should fill in storage state.
   ItemObserver()->OnDownloadUpdated(item);
 
   EXPECT_EQ(123, request_info.received_bytes());
   EXPECT_TRUE(data_manager()->IsComplete(kJobGuid));
   EXPECT_TRUE(did_complete_job());
+  EXPECT_FALSE(did_abort_by_developer());
 }
 
 }  // namespace content