[ServiceWorker] Fetch event should return integrity value

third_party/WebKit/Source/core/dom/ScriptLoader.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2008 Nikolas Zimmermann <zimmermann@kde.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 714 matching lines @@
                                 kClientRequestedCredentials);
   options.initiator_info.name = element_->InitiatorName();
   FetchParameters params(resource_request, options);
 
   // "... cryptographic nonce, ..."
   params.SetContentSecurityPolicyNonce(nonce);
 
   // "... integrity metadata, ..."
   params.SetIntegrityMetadata(integrity_metadata);
 
+  // "... integrity value, ..."
+  params.SetIntegrityValue(element_->IntegrityAttributeValue());

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

How about having the integrity string in IntegrityMetadata? 
I think it'd be better to have information about the integrity into one place. 

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

integrity string was converted to IntegrityMetadataSet in
SubresourceIntegrity::ParseIntegrityAttribute, even IntegrityMetadata 
 has the integrity string, it is difficult to get it from
integrity_metadata(IntegrityMetadataSet).

+
   // "... parser state, ..."
   params.SetParserDisposition(parser_state);
 
   // "... CORS setting, ..."
   if (cross_origin != kCrossOriginAttributeNotSet) {
     params.SetCrossOriginAccessControl(security_origin, cross_origin);
   }
 
   // "... and encoding."
   params.SetCharset(encoding);
@@ skipping 239 matching lines @@
   //     then abort these steps at this point. The script is not executed.
   return DeprecatedEqualIgnoringCase(event_attribute, "onload") ||
          DeprecatedEqualIgnoringCase(event_attribute, "onload()");
 }
 
 String ScriptLoader::ScriptContent() const {
   return element_->TextFromChildren();
 }
 
 }  // namespace blink