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

Side by Side Diff: content/browser/service_worker/foreign_fetch_request_handler.cc

Issue 1597383002: Service Worker: (Re-commit) Add FetchEvent.clientId (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/service_worker/foreign_fetch_request_handler.h" 5 #include "content/browser/service_worker/foreign_fetch_request_handler.h"
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "content/browser/service_worker/service_worker_context_wrapper.h" 8 #include "content/browser/service_worker/service_worker_context_wrapper.h"
9 #include "content/browser/service_worker/service_worker_url_request_job.h" 9 #include "content/browser/service_worker/service_worker_url_request_job.h"
10 #include "content/common/resource_request_body.h" 10 #include "content/common/resource_request_body.h"
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 if (use_network_) { 112 if (use_network_) {
113 // TODO(mek): Determine if redirects should be able to be intercepted by 113 // TODO(mek): Determine if redirects should be able to be intercepted by
114 // other foreign fetch service workers. 114 // other foreign fetch service workers.
115 return nullptr; 115 return nullptr;
116 } 116 }
117 117
118 // It's for original request (A) or redirect case (B-a or B-b). 118 // It's for original request (A) or redirect case (B-a or B-b).
119 DCHECK(!job_.get() || job_->ShouldForwardToServiceWorker()); 119 DCHECK(!job_.get() || job_->ShouldForwardToServiceWorker());
120 120
121 ServiceWorkerURLRequestJob* job = new ServiceWorkerURLRequestJob( 121 ServiceWorkerURLRequestJob* job = new ServiceWorkerURLRequestJob(
122 request, network_delegate, blob_storage_context_, resource_context, 122 request, network_delegate, std::string(), blob_storage_context_,
123 request_mode_, credentials_mode_, redirect_mode_, false, 123 resource_context, request_mode_, credentials_mode_, redirect_mode_, false,
124 request_context_type_, frame_type_, body_, this); 124 request_context_type_, frame_type_, body_, this);
125 job_ = job->GetWeakPtr(); 125 job_ = job->GetWeakPtr();
126 126
127 context_->FindReadyRegistrationForDocument( 127 context_->FindReadyRegistrationForDocument(
128 request->url(), 128 request->url(),
129 base::Bind(&ForeignFetchRequestHandler::DidFindRegistration, 129 base::Bind(&ForeignFetchRequestHandler::DidFindRegistration,
130 weak_factory_.GetWeakPtr(), job_)); 130 weak_factory_.GetWeakPtr(), job_));
131 131
132 return job_.get(); 132 return job_.get();
133 } 133 }
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
218 // TODO(mek): Implement this. 218 // TODO(mek): Implement this.
219 return GURL(); 219 return GURL();
220 } 220 }
221 221
222 void ForeignFetchRequestHandler::ClearJob() { 222 void ForeignFetchRequestHandler::ClearJob() {
223 job_.reset(); 223 job_.reset();
224 target_worker_ = nullptr; 224 target_worker_ = nullptr;
225 } 225 }
226 226
227 } // namespace content 227 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698