[ServiceWorker] Fetch event should return integrity value

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

Index: third_party/WebKit/Source/platform/loader/fetch/Resource.h
diff --git a/third_party/WebKit/Source/platform/loader/fetch/Resource.h b/third_party/WebKit/Source/platform/loader/fetch/Resource.h
index 1b300e9abd1bfd9a2023c62f79e442a04182e3eb..d48b09255e06ae239784a861ef1e92e94d5ed966 100644
--- a/third_party/WebKit/Source/platform/loader/fetch/Resource.h
+++ b/third_party/WebKit/Source/platform/loader/fetch/Resource.h
@@ -281,6 +281,8 @@ class PLATFORM_EXPORT Resource : public GarbageCollectedFinalized<Resource>,
   const IntegrityMetadataSet& IntegrityMetadata() const {
     return integrity_metadata_;
   }
+  void SetIntegrityValue(const String& integrity) { integrity_ = integrity; }

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

Please remove these and use ResourceRequet's members / methods instead.

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

Done. Thanks.

+  String IntegrityValue() const { return integrity_; }
   // The argument must never be |NotChecked|.
   void SetIntegrityDisposition(ResourceIntegrityDisposition);
   ResourceIntegrityDisposition IntegrityDisposition() const {
@@ -473,6 +475,7 @@ class PLATFORM_EXPORT Resource : public GarbageCollectedFinalized<Resource>,
 
   ResourceIntegrityDisposition integrity_disposition_;
   IntegrityMetadataSet integrity_metadata_;
+  String integrity_;

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

Is this necessary to be kept here?

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

Not necessary, thanks for pointing out!

 
   // Ordered list of all redirects followed while fetching this resource.
   Vector<RedirectPair> redirect_chain_;