Introduce ServiceWorkerRegistrationOptions struct for service worker options

chrome/browser/android/webapps/add_to_homescreen_data_fetcher_unittest.cc

 // Copyright 2016 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 "chrome/browser/android/webapps/add_to_homescreen_data_fetcher.h"
 
 #include <memory>
 #include <string>
 
 #include "base/callback_forward.h"
 #include "base/files/file_path.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/strings/nullable_string16.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/time/time.h"
 #include "chrome/browser/installable/installable_manager.h"
 #include "chrome/common/web_application_info.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/browser/service_worker/embedded_worker_test_helper.h"
 #include "content/browser/service_worker/service_worker_context_core.h"

[dominickn, 2017/07/07 03:04:32]

Nit: #include "content/common/service_worker/service_worker_types.h" as well

[yuryu, 2017/07/10 03:38:10]

Done.

 #include "content/common/service_worker/service_worker_status_code.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/manifest.h"
 #include "content/test/test_web_contents.h"
 #include "net/http/http_status_code.h"
 #include "third_party/WebKit/public/platform/WebDisplayMode.h"
 #include "ui/gfx/image/image_unittest_util.h"
 #include "url/gurl.h"
@@ skipping 220 matching lines @@
     MockWebContents* mock_web_contents =
         static_cast<MockWebContents*>(web_contents());
     mock_web_contents->SetShouldManifestTimeOut(should_time_out);
   }
 
   // Registers service worker at |url|. Blocks till the service worker is
   // registered.
   void RegisterServiceWorker(const GURL& url) {
     base::RunLoop run_loop;
     embedded_worker_test_helper_->context()->RegisterServiceWorker(
-        url, GURL(url.spec() + "/service_worker.js"), nullptr,
+        GURL(url.spec() + "/service_worker.js"),
+        content::ServiceWorkerRegistrationOptions(url), nullptr,
         base::Bind(&AddToHomescreenDataFetcherTest::OnServiceWorkerRegistered,
                    base::Unretained(this), run_loop.QuitClosure()));
   }
 
  private:
   // Callback for RegisterServiceWorker() for when service worker registration
   // has completed.
   void OnServiceWorkerRegistered(const base::Closure& callback,
                                  content::ServiceWorkerStatusCode status,
                                  const std::string& status_message,
@@ skipping 185 matching lines @@
   EXPECT_TRUE(base::EqualsASCII(waiter.title(), kDefaultManifestShortName));
   EXPECT_TRUE(base::EqualsASCII(fetcher->shortcut_info().user_title,
                                 kDefaultManifestShortName));
 
   fetcher->set_weak_observer(nullptr);
 }
 
 INSTANTIATE_TEST_CASE_P(CheckWebApkCompatibility,
                         AddToHomescreenDataFetcherTestCommon,
                         ::testing::Values(false, true));