Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(304)

Unified Diff: content/browser/service_worker/service_worker_controllee_request_handler_unittest.cc

Issue 501453002: Decouple script_url from ServiceWorkerRegistration (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review comments and add unittests Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/service_worker/service_worker_controllee_request_handler_unittest.cc
diff --git a/content/browser/service_worker/service_worker_controllee_request_handler_unittest.cc b/content/browser/service_worker/service_worker_controllee_request_handler_unittest.cc
index 552cbfaa2f041ece1c653ac05ba4de60a1f831d9..6bf4a831d649e473b86e7eb5919c54c24eb1331f 100644
--- a/content/browser/service_worker/service_worker_controllee_request_handler_unittest.cc
+++ b/content/browser/service_worker/service_worker_controllee_request_handler_unittest.cc
@@ -42,10 +42,9 @@ class ServiceWorkerControlleeRequestHandlerTest : public testing::Test {
scope_ = GURL("http://host/scope/");
script_url_ = GURL("http://host/script.js");
registration_ = new ServiceWorkerRegistration(
- scope_, script_url_, 1L, context()->AsWeakPtr());
+ scope_, 1L, context()->AsWeakPtr());
version_ = new ServiceWorkerVersion(
- registration_,
- 1L, context()->AsWeakPtr());
+ registration_, script_url_, 1L, context()->AsWeakPtr());
// An empty host.
scoped_ptr<ServiceWorkerProviderHost> host(new ServiceWorkerProviderHost(

Powered by Google App Engine
This is Rietveld 408576698