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

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

Issue 2467793002: use scoped_refptr
Patch Set: add test Created 4 years, 1 month 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/service_worker_fetch_dispatcher.h" 5 #include "content/browser/service_worker/service_worker_fetch_dispatcher.h"
6 6
7 #include <string> 7 #include <string>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 263 matching lines...) Expand 10 before | Expand all | Expand 10 after
274 net_log_.EndEvent( 274 net_log_.EndEvent(
275 net::NetLogEventType::SERVICE_WORKER_DISPATCH_FETCH_EVENT, 275 net::NetLogEventType::SERVICE_WORKER_DISPATCH_FETCH_EVENT,
276 base::Bind(&NetLogFetchEventCallback, status, fetch_result)); 276 base::Bind(&NetLogFetchEventCallback, status, fetch_result));
277 277
278 FetchCallback fetch_callback = fetch_callback_; 278 FetchCallback fetch_callback = fetch_callback_;
279 scoped_refptr<ServiceWorkerVersion> version = version_; 279 scoped_refptr<ServiceWorkerVersion> version = version_;
280 fetch_callback.Run(status, fetch_result, response, version); 280 fetch_callback.Run(status, fetch_result, response, version);
281 } 281 }
282 282
283 void ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload( 283 void ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload(
284 net::URLRequest* original_request) { 284 net::URLRequest* original_request,
285 base::WeakPtr<ServiceWorkerProviderHost> provider_host) {
285 if (resource_type_ != RESOURCE_TYPE_MAIN_FRAME && 286 if (resource_type_ != RESOURCE_TYPE_MAIN_FRAME &&
286 resource_type_ != RESOURCE_TYPE_SUB_FRAME) { 287 resource_type_ != RESOURCE_TYPE_SUB_FRAME) {
287 return; 288 return;
288 } 289 }
289 if (!version_->navigation_preload_enabled()) 290 if (!version_->navigation_preload_enabled())
290 return; 291 return;
291 // TODO(horo): Currently NavigationPreload doesn't support request body. 292 // TODO(horo): Currently NavigationPreload doesn't support request body.
292 if (!request_->blob_uuid.empty()) 293 if (!request_->blob_uuid.empty())
293 return; 294 return;
294 if (!base::FeatureList::IsEnabled( 295 if (!base::FeatureList::IsEnabled(
295 features::kServiceWorkerNavigationPreload)) { 296 features::kServiceWorkerNavigationPreload)) {
296 // TODO(horo): Check |version_|'s origin_trial_tokens() here if we use 297 // TODO(horo): Check |version_|'s origin_trial_tokens() here if we use
297 // Origin-Trial for NavigationPreload. 298 // Origin-Trial for NavigationPreload.
298 return; 299 return;
299 } 300 }
300 if (IsBrowserSideNavigationEnabled()) {
301 // TODO(horo): Support NavigationPreload with PlzNavigate.
302 NOTIMPLEMENTED();
303 return;
304 }
305 301
306 const ResourceRequestInfoImpl* original_info = 302 const ResourceRequestInfoImpl* original_info =
307 ResourceRequestInfoImpl::ForRequest(original_request); 303 ResourceRequestInfoImpl::ForRequest(original_request);
308 if (!original_info->filter()) 304 ResourceMessageFilter* filter = original_info->filter();
309 return; 305 if (!filter) {
306 DCHECK(IsBrowserSideNavigationEnabled());
307 filter = provider_host->resource_filter_for_navigation_preload();
308 }
309 DCHECK(filter);
310
310 mojom::URLLoaderFactoryPtr factory; 311 mojom::URLLoaderFactoryPtr factory;
311 URLLoaderFactoryImpl::Create(original_info->filter(), 312 URLLoaderFactoryImpl::Create(filter, mojo::GetProxy(&factory));
312 mojo::GetProxy(&factory));
313 313
314 preload_handle_ = mojom::FetchEventPreloadHandle::New(); 314 preload_handle_ = mojom::FetchEventPreloadHandle::New();
315 315
316 mojom::URLLoaderClientPtr url_loader_client; 316 mojom::URLLoaderClientPtr url_loader_client;
317 preload_handle_->url_loader_client_request = GetProxy(&url_loader_client); 317 preload_handle_->url_loader_client_request = GetProxy(&url_loader_client);
318 318
319 ResourceRequest request; 319 ResourceRequest request;
320 request.method = original_request->method(); 320 request.method = original_request->method();
321 request.url = original_request->url(); 321 request.url = original_request->url();
322 request.referrer = GURL(original_request->referrer()); 322 request.referrer = GURL(original_request->referrer());
(...skipping 19 matching lines...) Expand all
342 } 342 }
343 343
344 ServiceWorkerMetrics::EventType ServiceWorkerFetchDispatcher::GetEventType() 344 ServiceWorkerMetrics::EventType ServiceWorkerFetchDispatcher::GetEventType()
345 const { 345 const {
346 if (request_->fetch_type == ServiceWorkerFetchType::FOREIGN_FETCH) 346 if (request_->fetch_type == ServiceWorkerFetchType::FOREIGN_FETCH)
347 return ServiceWorkerMetrics::EventType::FOREIGN_FETCH; 347 return ServiceWorkerMetrics::EventType::FOREIGN_FETCH;
348 return ResourceTypeToEventType(resource_type_); 348 return ResourceTypeToEventType(resource_type_);
349 } 349 }
350 350
351 } // namespace content 351 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698