CSSStyleSheetResource should cache decoded text instead of raw bytes

third_party/WebKit/Source/core/html/HTMLLinkElement.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, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2009 Rob Buis (rwlbuis@gmail.com)
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 411 matching lines @@
     m_loading = false;
     removePendingSheet();
     if (m_sheet)
       clearSheet();
     return;
   }
 
   // See the comment in PendingScript.cpp about why this check is necessary
   // here, instead of in the resource fetcher. https://crbug.com/500701.
   if (!cachedStyleSheet->errorOccurred() &&
-      m_owner->fastHasAttribute(HTMLNames::integrityAttr) &&
-      cachedStyleSheet->resourceBuffer() &&
-      !SubresourceIntegrity::CheckSubresourceIntegrity(
-          *m_owner, cachedStyleSheet->resourceBuffer()->data(),
-          cachedStyleSheet->resourceBuffer()->size(), KURL(baseURL, href),
-          *cachedStyleSheet)) {
-    m_loading = false;
-    removePendingSheet();
-    notifyLoadedSheetAndAllCriticalSubresources(
-        Node::ErrorOccurredLoadingSubresource);
-    return;
+      !m_owner->fastGetAttribute(HTMLNames::integrityAttr).isEmpty() &&
+      !cachedStyleSheet->integrityMetadata().isEmpty()) {
+    ResourceIntegrityDisposition disposition =
+        cachedStyleSheet->integrityDisposition();
+
+    DCHECK(disposition != ResourceIntegrityDisposition::NotChecked ||
+           !cachedStyleSheet->sheetText().isNull());
+
+    if (disposition == ResourceIntegrityDisposition::NotChecked &&
+        !cachedStyleSheet->loadFailedOrCanceled()) {
+      DCHECK(cachedStyleSheet->resourceBuffer());
+      if (SubresourceIntegrity::CheckSubresourceIntegrity(
+              *m_owner, cachedStyleSheet->resourceBuffer()->data(),
+              cachedStyleSheet->resourceBuffer()->size(), KURL(baseURL, href),
+              *cachedStyleSheet))
+        disposition = ResourceIntegrityDisposition::Passed;
+      else
+        disposition = ResourceIntegrityDisposition::Failed;
+
+      // TODO(kouhei): Remove this const_cast crbug.com/653502
+      const_cast<CSSStyleSheetResource*>(cachedStyleSheet)
+          ->setIntegrityDisposition(disposition);
+    }
+
+    if (disposition == ResourceIntegrityDisposition::Failed) {
+      m_loading = false;
+      removePendingSheet();
+      notifyLoadedSheetAndAllCriticalSubresources(
+          Node::ErrorOccurredLoadingSubresource);
+      return;
+    }
   }
 
   CSSParserContext parserContext(m_owner->document(), nullptr, baseURL,
                                  charset);
 
   DEFINE_STATIC_LOCAL(EnumerationHistogram, restoredCachedStyleSheetHistogram,
                       ("Blink.RestoredCachedStyleSheet", 2));
   DEFINE_STATIC_LOCAL(
       EnumerationHistogram, restoredCachedStyleSheet2Histogram,
       ("Blink.RestoredCachedStyleSheet2", StyleSheetCacheStatusCount));
@@ skipping 289 matching lines @@
     removePendingSheet();
 }
 
 DEFINE_TRACE(LinkStyle) {
   visitor->trace(m_sheet);
   LinkResource::trace(visitor);
   ResourceOwner<StyleSheetResource>::trace(visitor);
 }
 
 }  // namespace blink