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

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

Issue 2473263003: Set "Service-Worker-Navigation-Preload" header (Closed)
Patch Set: incorporated falken's comment 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
« no previous file with comments | « content/browser/service_worker/service_worker_browsertest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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"
11 #include "base/feature_list.h" 11 #include "base/feature_list.h"
12 #include "base/memory/ptr_util.h" 12 #include "base/memory/ptr_util.h"
13 #include "base/time/time.h" 13 #include "base/time/time.h"
14 #include "base/trace_event/trace_event.h" 14 #include "base/trace_event/trace_event.h"
15 #include "content/browser/loader/resource_dispatcher_host_impl.h" 15 #include "content/browser/loader/resource_dispatcher_host_impl.h"
16 #include "content/browser/loader/resource_message_filter.h" 16 #include "content/browser/loader/resource_message_filter.h"
17 #include "content/browser/loader/resource_request_info_impl.h" 17 #include "content/browser/loader/resource_request_info_impl.h"
18 #include "content/browser/loader/url_loader_factory_impl.h" 18 #include "content/browser/loader/url_loader_factory_impl.h"
19 #include "content/browser/service_worker/embedded_worker_status.h" 19 #include "content/browser/service_worker/embedded_worker_status.h"
20 #include "content/browser/service_worker/service_worker_version.h" 20 #include "content/browser/service_worker/service_worker_version.h"
21 #include "content/common/service_worker/fetch_event_dispatcher.mojom.h" 21 #include "content/common/service_worker/fetch_event_dispatcher.mojom.h"
22 #include "content/common/service_worker/service_worker_messages.h" 22 #include "content/common/service_worker/service_worker_messages.h"
23 #include "content/common/service_worker/service_worker_status_code.h" 23 #include "content/common/service_worker/service_worker_status_code.h"
24 #include "content/common/service_worker/service_worker_types.h" 24 #include "content/common/service_worker/service_worker_types.h"
25 #include "content/common/service_worker/service_worker_utils.h" 25 #include "content/common/service_worker/service_worker_utils.h"
26 #include "content/public/common/browser_side_navigation_policy.h" 26 #include "content/public/common/browser_side_navigation_policy.h"
27 #include "content/public/common/content_features.h" 27 #include "content/public/common/content_features.h"
28 #include "mojo/public/cpp/bindings/associated_binding.h" 28 #include "mojo/public/cpp/bindings/associated_binding.h"
29 #include "mojo/public/cpp/bindings/binding.h" 29 #include "mojo/public/cpp/bindings/binding.h"
30 #include "net/http/http_util.h"
30 #include "net/log/net_log.h" 31 #include "net/log/net_log.h"
31 #include "net/log/net_log_capture_mode.h" 32 #include "net/log/net_log_capture_mode.h"
32 #include "net/log/net_log_event_type.h" 33 #include "net/log/net_log_event_type.h"
33 #include "net/url_request/url_request.h" 34 #include "net/url_request/url_request.h"
34 35
35 namespace content { 36 namespace content {
36 37
37 namespace { 38 namespace {
38 39
39 // This class wraps a mojo::AssociatedInterfacePtr<URLLoader>. It also is a 40 // This class wraps a mojo::AssociatedInterfacePtr<URLLoader>. It also is a
(...skipping 349 matching lines...) Expand 10 before | Expand all | Expand 10 after
389 request.load_flags = original_request->load_flags(); 390 request.load_flags = original_request->load_flags();
390 // Set to SUB_RESOURCE because we shouldn't trigger NavigationResourceThrottle 391 // Set to SUB_RESOURCE because we shouldn't trigger NavigationResourceThrottle
391 // for the service worker navigation preload request. 392 // for the service worker navigation preload request.
392 request.resource_type = RESOURCE_TYPE_SUB_RESOURCE; 393 request.resource_type = RESOURCE_TYPE_SUB_RESOURCE;
393 request.priority = original_request->priority(); 394 request.priority = original_request->priority();
394 request.skip_service_worker = SkipServiceWorker::ALL; 395 request.skip_service_worker = SkipServiceWorker::ALL;
395 request.do_not_prompt_for_login = true; 396 request.do_not_prompt_for_login = true;
396 request.render_frame_id = original_info->GetRenderFrameID(); 397 request.render_frame_id = original_info->GetRenderFrameID();
397 request.is_main_frame = original_info->IsMainFrame(); 398 request.is_main_frame = original_info->IsMainFrame();
398 request.parent_is_main_frame = original_info->ParentIsMainFrame(); 399 request.parent_is_main_frame = original_info->ParentIsMainFrame();
400
401 DCHECK(net::HttpUtil::IsValidHeaderValue(
402 version_->navigation_preload_state().header));
403 request.headers = "Service-Worker-Navigation-Preload: " +
404 version_->navigation_preload_state().header;
405
399 const int request_id = ResourceDispatcherHostImpl::Get()->MakeRequestID(); 406 const int request_id = ResourceDispatcherHostImpl::Get()->MakeRequestID();
400 DCHECK_LT(request_id, -1); 407 DCHECK_LT(request_id, -1);
401 408
402 preload_handle_ = mojom::FetchEventPreloadHandle::New(); 409 preload_handle_ = mojom::FetchEventPreloadHandle::New();
403 mojom::URLLoaderClientPtr url_loader_client_ptr; 410 mojom::URLLoaderClientPtr url_loader_client_ptr;
404 preload_handle_->url_loader_client_request = 411 preload_handle_->url_loader_client_request =
405 mojo::GetProxy(&url_loader_client_ptr); 412 mojo::GetProxy(&url_loader_client_ptr);
406 auto url_loader_client = base::MakeUnique<DelegatingURLLoaderClient>( 413 auto url_loader_client = base::MakeUnique<DelegatingURLLoaderClient>(
407 std::move(url_loader_client_ptr)); 414 std::move(url_loader_client_ptr));
408 mojom::URLLoaderClientAssociatedPtrInfo url_loader_client_associated_ptr_info; 415 mojom::URLLoaderClientAssociatedPtrInfo url_loader_client_associated_ptr_info;
409 url_loader_client->Bind(&url_loader_client_associated_ptr_info, 416 url_loader_client->Bind(&url_loader_client_associated_ptr_info,
410 url_loader_factory_.associated_group()); 417 url_loader_factory_.associated_group());
411 mojom::URLLoaderAssociatedPtr url_loader_associated_ptr; 418 mojom::URLLoaderAssociatedPtr url_loader_associated_ptr;
412 419
413 // TODO(horo): Add "Service-Worker-Navigation-Preload" header.
414 // See: https://github.com/w3c/ServiceWorker/issues/920#issuecomment-251150270
415 url_loader_factory_->CreateLoaderAndStart( 420 url_loader_factory_->CreateLoaderAndStart(
416 mojo::GetProxy(&url_loader_associated_ptr, 421 mojo::GetProxy(&url_loader_associated_ptr,
417 url_loader_factory_.associated_group()), 422 url_loader_factory_.associated_group()),
418 original_info->GetRouteID(), request_id, request, 423 original_info->GetRouteID(), request_id, request,
419 std::move(url_loader_client_associated_ptr_info)); 424 std::move(url_loader_client_associated_ptr_info));
420 425
421 std::unique_ptr<DelegatingURLLoader> url_loader( 426 std::unique_ptr<DelegatingURLLoader> url_loader(
422 new DelegatingURLLoader(std::move(url_loader_associated_ptr))); 427 new DelegatingURLLoader(std::move(url_loader_associated_ptr)));
423 preload_handle_->url_loader = url_loader->CreateInterfacePtrAndBind(); 428 preload_handle_->url_loader = url_loader->CreateInterfacePtrAndBind();
424 url_loader_ = std::move(url_loader); 429 url_loader_ = std::move(url_loader);
425 url_loader_client_ = std::move(url_loader_client); 430 url_loader_client_ = std::move(url_loader_client);
426 } 431 }
427 432
428 ServiceWorkerMetrics::EventType ServiceWorkerFetchDispatcher::GetEventType() 433 ServiceWorkerMetrics::EventType ServiceWorkerFetchDispatcher::GetEventType()
429 const { 434 const {
430 if (request_->fetch_type == ServiceWorkerFetchType::FOREIGN_FETCH) 435 if (request_->fetch_type == ServiceWorkerFetchType::FOREIGN_FETCH)
431 return ServiceWorkerMetrics::EventType::FOREIGN_FETCH; 436 return ServiceWorkerMetrics::EventType::FOREIGN_FETCH;
432 return ResourceTypeToEventType(resource_type_); 437 return ResourceTypeToEventType(resource_type_);
433 } 438 }
434 439
435 } // namespace content 440 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/service_worker/service_worker_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698