Use the request URL as the URL of the resources which are fetched ServiceWorker [3/3 blink]

Source/core/fetch/ResourceFetcher.cpp

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
     Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
     Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
     Copyright (C) 2009 Torch Mobile Inc. http://www.torchmobile.com/
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
@@ skipping 1329 matching lines @@
 
 void ResourceFetcher::willSendRequest(unsigned long identifier, ResourceRequest& request, const ResourceResponse& redirectResponse, const FetchInitiatorInfo& initiatorInfo)
 {
     context().dispatchWillSendRequest(m_documentLoader, identifier, request, redirectResponse, initiatorInfo);
 }
 
 void ResourceFetcher::didReceiveResponse(const Resource* resource, const ResourceResponse& response)
 {
     MixedContentChecker::checkMixedPrivatePublic(frame(), response.remoteIPAddress());
 
     // If the response is fetched via ServiceWorker, the original URL of the response could be different from the URL of the request.

[yhirano, 2014/11/19 04:06:21]

Can you add a comment here that this check aims not to fetch a resource that are
forbidden by the page CSP even in the service worker, and it is not specified
currently?

[horo, 2014/11/19 04:55:59]

Done.

     if (response.wasFetchedViaServiceWorker()) {
-        KURL originalURL = response.url();
-        // FIXME: Use response.originalURLViaServiceWorker() after the chromium side patch will land.
-        if (!response.originalURLViaServiceWorker().isEmpty())
-            originalURL = response.originalURLViaServiceWorker();
+        const KURL& originalURL = response.originalURLViaServiceWorker();
         if (!canRequest(resource->type(), resource->resourceRequest(), originalURL, resource->options(), false, FetchRequest::UseDefaultOriginRestrictionForType)) {
             resource->loader()->cancel();
             bool isInternalRequest = resource->options().initiatorInfo.name == FetchInitiatorTypeNames::internal;
             context().dispatchDidFail(m_documentLoader, resource->identifier(), ResourceError(errorDomainBlinkInternal, 0, originalURL.string(), "Unsafe attempt to load URL " + originalURL.elidedString() + " fetched by a ServiceWorker."), isInternalRequest);
             return;
         }
     }
     context().dispatchDidReceiveResponse(m_documentLoader, resource->identifier(), response, resource->loader());
 }
 
@@ skipping 210 matching lines @@
 
 void ResourceFetcher::trace(Visitor* visitor)
 {
     visitor->trace(m_document);
     visitor->trace(m_loaders);
     visitor->trace(m_multipartLoaders);
     ResourceLoaderHost::trace(visitor);
 }
 
 }