Move ReportLocalLoadFailed to ExecutionContext

third_party/WebKit/Source/platform/loader/fetch/ResourceLoader.cpp

 /*
  * Copyright (C) 2006, 2007, 2010, 2011 Apple Inc. All rights reserved.
  *           (C) 2007 Graham Dennis (graham.dennis@gmail.com)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 147 matching lines @@
     // Fail as cache miss if cached response is a redirect.
     HandleError(
         ResourceError::CacheMissError(resource_->LastResourceRequest().Url()));
     return false;
   }
 
   ResourceRequest& new_request(passed_new_request.ToMutableResourceRequest());
   const ResourceResponse& redirect_response(
       passed_redirect_response.ToResourceResponse());
 
+  // For main resources, we want to ensure that every step of the redirect chain
+  // is a valid transition, so set the SecurityOrigin based on the url of the
+  // redirect source.
+  if (resource_->GetType() == Resource::kMainResource) {
+    resource_->MutableOptions().security_origin =
+        SecurityOrigin::Create(redirect_response.Url());

[kinuko, 2017/05/23 13:54:38]

For DocumentLoader case I might prefer having this check in the original place. 
BaseFetchContext's corresponding implementation is in core/ layer while we're in
platform/ layer, which makes me feel it's going to be a bit hard to follow what
we're doing here.

[Nate Chapin, 2017/05/23 21:20:50]

Moved back to DocumentLoader.

+  }
+
   new_request.SetRedirectStatus(
       ResourceRequest::RedirectStatus::kFollowedRedirect);
 
   const KURL original_url = new_request.Url();
 
   if (!IsManualRedirectFetchRequest(resource_->GetResourceRequest())) {
     ResourceRequestBlockedReason blocked_reason = Context().CanFollowRedirect(
         resource_->GetType(), new_request, new_request.Url(),
         resource_->Options(),
         /* Don't send security violation reports for unused preloads */
@@ skipping 349 matching lines @@
     return;
 
   // Don't activate if cache policy is explicitly set.
   if (request.GetCachePolicy() != WebCachePolicy::kUseProtocolCachePolicy)
     return;
 
   is_cache_aware_loading_activated_ = true;
 }
 
 }  // namespace blink