[ServiceWorker] Fetch event should return integrity value

third_party/WebKit/Source/platform/loader/fetch/FetchParameters.h

Index: third_party/WebKit/Source/platform/loader/fetch/FetchParameters.h
diff --git a/third_party/WebKit/Source/platform/loader/fetch/FetchParameters.h b/third_party/WebKit/Source/platform/loader/fetch/FetchParameters.h
index bf09ceda25ccbead76a218d444963619b63b65f8..5a877a1333d51cee0292d8b521f9ab84a1d57d6e 100644
--- a/third_party/WebKit/Source/platform/loader/fetch/FetchParameters.h
+++ b/third_party/WebKit/Source/platform/loader/fetch/FetchParameters.h
@@ -90,6 +90,9 @@ class PLATFORM_EXPORT FetchParameters {
   const String& Charset() const { return charset_; }
   void SetCharset(const String& charset) { charset_ = charset; }
 
+  const String& IntegrityValue() const { return integrity_; }

[shimazu, 2017/06/26 06:13:32]

I guess it's not used.

[yhirano, 2017/06/28 04:55:09]

Please remove these and use ResourceRequest members / methods instead.

[xiaofengzhang, 2017/06/29 02:26:06]

Yes, Done.

+  void SetIntegrityValue(const String& integrity) { integrity_ = integrity; }
+
   const ResourceLoaderOptions& Options() const { return options_; }
 
   DeferOption Defer() const { return defer_; }
@@ -165,6 +168,7 @@ class PLATFORM_EXPORT FetchParameters {
  private:
   ResourceRequest resource_request_;
   String charset_;
+  String integrity_;

[shimazu, 2017/06/26 06:13:32]

I think we don't need to keep the |integrity_| value here because you are
already adding it to ResourceRequest.

[xiaofengzhang, 2017/06/29 02:26:06]

Done.

   ResourceLoaderOptions options_;
   SpeculativePreloadType speculative_preload_type_;
   double preload_discovery_time_;