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

Side by Side Diff: third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.cpp

Issue 2941883003: [ServiceWorker] Fetch event should return integrity value (Closed)
Patch Set: Rebase and address shimazu and yhirano's comments Created 3 years, 5 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 /* 1 /*
2 Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de) 2 Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
3 Copyright (C) 2001 Dirk Mueller (mueller@kde.org) 3 Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
4 Copyright (C) 2002 Waldo Bastian (bastian@kde.org) 4 Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
5 Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All 5 Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All
6 rights reserved. 6 rights reserved.
7 Copyright (C) 2009 Torch Mobile Inc. http://www.torchmobile.com/ 7 Copyright (C) 2009 Torch Mobile Inc. http://www.torchmobile.com/
8 8
9 This library is free software; you can redistribute it and/or 9 This library is free software; you can redistribute it and/or
10 modify it under the terms of the GNU Library General Public 10 modify it under the terms of the GNU Library General Public
(...skipping 1055 matching lines...) Expand 10 before | Expand all | Expand 10 after
1066 return new_is_with_cors_mode == existing_was_with_cors_mode; 1066 return new_is_with_cors_mode == existing_was_with_cors_mode;
1067 } 1067 }
1068 1068
1069 ResourceFetcher::RevalidationPolicy 1069 ResourceFetcher::RevalidationPolicy
1070 ResourceFetcher::DetermineRevalidationPolicy( 1070 ResourceFetcher::DetermineRevalidationPolicy(
1071 Resource::Type type, 1071 Resource::Type type,
1072 const FetchParameters& fetch_params, 1072 const FetchParameters& fetch_params,
1073 Resource* existing_resource, 1073 Resource* existing_resource,
1074 bool is_static_data) const { 1074 bool is_static_data) const {
1075 const ResourceRequest& request = fetch_params.GetResourceRequest(); 1075 const ResourceRequest& request = fetch_params.GetResourceRequest();
1076
yhirano 2017/06/30 09:11:57 Maybe you don't need this edit?
xiaofengzhang 2017/07/07 00:10:14 Done.
1077 if (!existing_resource) 1076 if (!existing_resource)
1078 return kLoad; 1077 return kLoad;
1079 1078
1080 // If the existing resource is loading and the associated fetcher is not equal 1079 // If the existing resource is loading and the associated fetcher is not equal
1081 // to |this|, we must not use the resource. Otherwise, CSP violation may 1080 // to |this|, we must not use the resource. Otherwise, CSP violation may
1082 // happen in redirect handling. 1081 // happen in redirect handling.
1083 if (existing_resource->Loader() && 1082 if (existing_resource->Loader() &&
1084 existing_resource->Loader()->Fetcher() != this) { 1083 existing_resource->Loader()->Fetcher() != this) {
1085 return kReload; 1084 return kReload;
1086 } 1085 }
(...skipping 661 matching lines...) Expand 10 before | Expand all | Expand 10 after
1748 visitor->Trace(archive_); 1747 visitor->Trace(archive_);
1749 visitor->Trace(loaders_); 1748 visitor->Trace(loaders_);
1750 visitor->Trace(non_blocking_loaders_); 1749 visitor->Trace(non_blocking_loaders_);
1751 visitor->Trace(document_resources_); 1750 visitor->Trace(document_resources_);
1752 visitor->Trace(preloads_); 1751 visitor->Trace(preloads_);
1753 visitor->Trace(matched_preloads_); 1752 visitor->Trace(matched_preloads_);
1754 visitor->Trace(resource_timing_info_map_); 1753 visitor->Trace(resource_timing_info_map_);
1755 } 1754 }
1756 1755
1757 } // namespace blink 1756 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698