WIP: Introduce CompressibleString

third_party/WebKit/Source/core/fetch/ScriptResource.cpp

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
     Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
     Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
     Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
@@ skipping 59 matching lines @@
     ResourceClientWalker<ScriptResourceClient> walker(m_clients);
     while (ScriptResourceClient* client = walker.next())
         client->notifyAppendData(this);
 }
 
 AtomicString ScriptResource::mimeType() const
 {
     return extractMIMETypeFromMediaType(m_response.httpHeaderField("Content-Type")).lower();
 }
 
-const String& ScriptResource::script()
+RefPtrWillBeRawPtr<CompressableString> ScriptResource::script()

[haraken, 2015/10/22 16:03:31]

RefPtrWillBeRawPtr<CompressableString> => CompressableString*

(because conceptually this is not transferring ownership.)

[hajimehoshi, 2015/10/26 09:34:02]

Done.

 {
     ASSERT(!isPurgeable());
     ASSERT(isLoaded());
 
     if (!m_script && m_data) {
         String script = decodedText();
         m_data.clear();
         // We lie a it here and claim that script counts as encoded data (even though it's really decoded data).
         // That's because the MemoryCache thinks that it can clear out decoded data by calling destroyDecodedData(),
         // but we can't destroy script in destroyDecodedData because that's our only copy of the data!
         setEncodedSize(script.sizeInBytes());
-        m_script = AtomicString(script);
+        m_script = CompressableString::create(script);
     }
 
-    return m_script.string();
+    return m_script;

[haraken, 2015/10/22 16:03:31]

return m_script.get();

[hajimehoshi, 2015/10/26 09:34:02]

Done.

 }
 
 bool ScriptResource::mimeTypeAllowedByNosniff() const
 {
     return parseContentTypeOptionsHeader(m_response.httpHeaderField("X-Content-Type-Options")) != ContentTypeOptionsNosniff || MIMETypeRegistry::isSupportedJavaScriptMIMEType(mimeType());
 }
 
 bool ScriptResource::mustRefetchDueToIntegrityMetadata(const FetchRequest& request) const
 {
     if (request.integrityMetadata().isEmpty())
         return false;
 
     // TODO(jww) this integrity metadata should actually be
     // normalized so that order doesn't matter.
     return m_integrityMetadata != request.integrityMetadata();
 }
 
 } // namespace blink