Remove ParamStorageTraits from BeaconLoader.cpp

third_party/WebKit/Source/core/loader/BeaconLoader.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/loader/BeaconLoader.h"
 
 #include "core/dom/DOMArrayBufferView.h"
 #include "core/dom/Document.h"
 #include "core/fetch/CrossOriginAccessControl.h"
 #include "core/fetch/FetchContext.h"
@@ skipping 13 matching lines @@
 #include "wtf/Functional.h"
 
 namespace blink {
 
 namespace {
 
 class Beacon {
 public:
     virtual bool serialize(ResourceRequest&, int, int&) const = 0;
     virtual unsigned long long size() const = 0;
-
-protected:
-    static unsigned long long beaconSize(const String&);
-    static unsigned long long beaconSize(Blob*);
-    static unsigned long long beaconSize(DOMArrayBufferView*);
-    static unsigned long long beaconSize(FormData*);
-
-    static bool serialize(const String&, ResourceRequest&, int, int&);
-    static bool serialize(Blob*, ResourceRequest&, int, int&);
-    static bool serialize(DOMArrayBufferView*, ResourceRequest&, int, int&);
-    static bool serialize(FormData*, ResourceRequest&, int, int&);
 };
 
-template<typename Payload>
-class BeaconData final : public Beacon {
+class BeaconString final : public Beacon {
 public:
-    BeaconData(const Payload& data)
+    BeaconString(const String& data)
         : m_data(data)
     {
     }
 
-    bool serialize(ResourceRequest& request, int allowance, int& payloadLength) const override
+private:

[sof, 2016/04/27 14:05:07]

let's just have "private:" limit visibility for the m_data field in this and the
other local classes here?

[hiroshige, 2016/04/28 05:52:10]

Done.

+    unsigned long long size() const override
     {
-        return Beacon::serialize(m_data, request, allowance, payloadLength);
+        return m_data.sizeInBytes();
     }
 
-    unsigned long long size() const override
+    bool serialize(ResourceRequest& request, int, int&) const override
     {
-        return beaconSize(m_data);
+        RefPtr<EncodedFormData> entityBody = EncodedFormData::create(m_data.utf8());
+        request.setHTTPBody(entityBody);
+        request.setHTTPContentType("text/plain;charset=UTF-8");
+        return true;
+    }
+
+    const String m_data;
+};
+
+class BeaconBlob final : public Beacon {
+public:
+    BeaconBlob(Blob* data)
+        : m_data(data)
+    {
     }
 
 private:
-    const typename WTF::ParamStorageTraits<Payload>::StorageType m_data;
+    unsigned long long size() const override
+    {
+        return m_data->size();
+    }
+
+    bool serialize(ResourceRequest& request, int, int&) const override
+    {
+        ASSERT(m_data);
+        RefPtr<EncodedFormData> entityBody = EncodedFormData::create();
+        if (m_data->hasBackingFile())
+            entityBody->appendFile(toFile(m_data)->path());
+        else
+            entityBody->appendBlob(m_data->uuid(), m_data->blobDataHandle());
+
+        request.setHTTPBody(entityBody.release());
+
+        const String& blobType = m_data->type();
+        if (!blobType.isEmpty() && isValidContentType(blobType))
+            request.setHTTPContentType(AtomicString(blobType));
+
+        return true;
+    }
+
+    const Persistent<Blob> m_data;
+};
+
+class BeaconDOMArrayBufferView final : public Beacon {
+public:
+    BeaconDOMArrayBufferView(DOMArrayBufferView* data)
+        : m_data(data)
+    {
+    }
+
+private:
+    unsigned long long size() const override
+    {
+        return m_data->byteLength();
+    }
+
+    bool serialize(ResourceRequest& request, int, int&) const override
+    {
+        ASSERT(m_data);
+        RefPtr<EncodedFormData> entityBody = EncodedFormData::create(m_data->baseAddress(), m_data->byteLength());
+        request.setHTTPBody(entityBody.release());
+
+        // FIXME: a reasonable choice, but not in the spec; should it give a default?
+        AtomicString contentType = AtomicString("application/octet-stream");
+        request.setHTTPContentType(contentType);
+
+        return true;
+    }
+
+    const Persistent<DOMArrayBufferView> m_data;
+};
+
+class BeaconFormData final : public Beacon {
+public:
+    BeaconFormData(FormData* data)
+        : m_data(data)
+    {
+    }
+
+private:
+    unsigned long long size() const override
+    {
+        // FormData's size cannot be determined until serialized.
+        return 0;
+    }
+
+    bool serialize(ResourceRequest& request, int allowance, int& payloadLength) const override
+    {
+        ASSERT(m_data);
+        RefPtr<EncodedFormData> entityBody = m_data->encodeMultiPartFormData();
+        unsigned long long entitySize = entityBody->sizeInBytes();
+        if (allowance > 0 && static_cast<unsigned long long>(allowance) < entitySize)
+            return false;
+
+        AtomicString contentType = AtomicString("multipart/form-data; boundary=") + entityBody->boundary().data();
+        request.setHTTPBody(entityBody.release());
+        request.setHTTPContentType(contentType);
+
+        payloadLength = entitySize;
+        return true;
+    }
+
+    const Persistent<FormData> m_data;
 };
 
 } // namespace
 
 class BeaconLoader::Sender {
 public:
     static bool send(LocalFrame* frame, int allowance, const KURL& beaconURL, const Beacon& beacon, int& payloadLength)
     {
         if (!frame->document())
             return false;
@@ skipping 22 matching lines @@
 
         // The loader keeps itself alive until it receives a response and disposes itself.
         BeaconLoader* loader = new BeaconLoader(frame, request, initiatorInfo, AllowStoredCredentials);
         ASSERT_UNUSED(loader, loader);
         return true;
     }
 };
 
 bool BeaconLoader::sendBeacon(LocalFrame* frame, int allowance, const KURL& beaconURL, const String& data, int& payloadLength)
 {
-    BeaconData<String> beacon(data);
+    BeaconString beacon(data);
     return Sender::send(frame, allowance, beaconURL, beacon, payloadLength);
 }
 
 bool BeaconLoader::sendBeacon(LocalFrame* frame, int allowance, const KURL& beaconURL, DOMArrayBufferView* data, int& payloadLength)
 {
-    BeaconData<decltype(data)> beacon(data);
+    BeaconDOMArrayBufferView beacon(data);
     return Sender::send(frame, allowance, beaconURL, beacon, payloadLength);
 }
 
 bool BeaconLoader::sendBeacon(LocalFrame* frame, int allowance, const KURL& beaconURL, FormData* data, int& payloadLength)
 {
-    BeaconData<decltype(data)> beacon(data);
+    BeaconFormData beacon(data);
     return Sender::send(frame, allowance, beaconURL, beacon, payloadLength);
 }
 
 bool BeaconLoader::sendBeacon(LocalFrame* frame, int allowance, const KURL& beaconURL, Blob* data, int& payloadLength)
 {
-    BeaconData<decltype(data)> beacon(data);
+    BeaconBlob beacon(data);
     return Sender::send(frame, allowance, beaconURL, beacon, payloadLength);
 }
 
 BeaconLoader::BeaconLoader(LocalFrame* frame, ResourceRequest& request, const FetchInitiatorInfo& initiatorInfo, StoredCredentials credentialsAllowed)
     : PingLoader(frame, request, initiatorInfo, credentialsAllowed)
     , m_beaconOrigin(frame->document()->getSecurityOrigin())
 {
 }
 
 void BeaconLoader::willFollowRedirect(WebURLLoader*, WebURLRequest& passedNewRequest, const WebURLResponse& passedRedirectResponse)
@@ skipping 14 matching lines @@
                 localFrame->document()->addConsoleMessage(ConsoleMessage::create(JSMessageSource, ErrorMessageLevel, errorDescription));
         }
         // Cancel the load and self destruct.
         dispose();
         return;
     }
     // FIXME: http://crbug.com/427429 is needed to correctly propagate
     // updates of Origin: following this successful redirect.
 }
 
-namespace {
-
-unsigned long long Beacon::beaconSize(const String& data)
-{
-    return data.sizeInBytes();
-}
-
-bool Beacon::serialize(const String& data, ResourceRequest& request, int, int&)
-{
-    RefPtr<EncodedFormData> entityBody = EncodedFormData::create(data.utf8());
-    request.setHTTPBody(entityBody);
-    request.setHTTPContentType("text/plain;charset=UTF-8");
-    return true;
-}
-
-unsigned long long Beacon::beaconSize(Blob* data)
-{
-    return data->size();
-}
-
-bool Beacon::serialize(Blob* data, ResourceRequest& request, int, int&)
-{
-    ASSERT(data);
-    RefPtr<EncodedFormData> entityBody = EncodedFormData::create();
-    if (data->hasBackingFile())
-        entityBody->appendFile(toFile(data)->path());
-    else
-        entityBody->appendBlob(data->uuid(), data->blobDataHandle());
-
-    request.setHTTPBody(entityBody.release());
-
-    const String& blobType = data->type();
-    if (!blobType.isEmpty() && isValidContentType(blobType))
-        request.setHTTPContentType(AtomicString(blobType));
-
-    return true;
-}
-
-unsigned long long Beacon::beaconSize(DOMArrayBufferView* data)
-{
-    return data->byteLength();
-}
-
-bool Beacon::serialize(DOMArrayBufferView* data, ResourceRequest& request, int, int&)
-{
-    ASSERT(data);
-    RefPtr<EncodedFormData> entityBody = EncodedFormData::create(data->baseAddress(), data->byteLength());
-    request.setHTTPBody(entityBody.release());
-
-    // FIXME: a reasonable choice, but not in the spec; should it give a default?
-    AtomicString contentType = AtomicString("application/octet-stream");
-    request.setHTTPContentType(contentType);
-
-    return true;
-}
-
-unsigned long long Beacon::beaconSize(FormData*)
-{
-    // FormData's size cannot be determined until serialized.
-    return 0;
-}
-
-bool Beacon::serialize(FormData* data, ResourceRequest& request, int allowance, int& payloadLength)
-{
-    ASSERT(data);
-    RefPtr<EncodedFormData> entityBody = data->encodeMultiPartFormData();
-    unsigned long long entitySize = entityBody->sizeInBytes();
-    if (allowance > 0 && static_cast<unsigned long long>(allowance) < entitySize)
-        return false;
-
-    AtomicString contentType = AtomicString("multipart/form-data; boundary=") + entityBody->boundary().data();
-    request.setHTTPBody(entityBody.release());
-    request.setHTTPContentType(contentType);
-
-    payloadLength = entitySize;
-    return true;
-}
-
-} // namespace
-
 } // namespace blink