Move multipart resource handling to core/fetch (1/2)

third_party/WebKit/Source/core/fetch/MultipartImageResourceParser.h

Index: third_party/WebKit/Source/core/fetch/MultipartImageResourceParser.h
diff --git a/third_party/WebKit/Source/core/fetch/MultipartImageResourceParser.h b/third_party/WebKit/Source/core/fetch/MultipartImageResourceParser.h
new file mode 100644
index 0000000000000000000000000000000000000000..e496e477c52610ee6a315e5817718164320ec7fd
--- /dev/null
+++ b/third_party/WebKit/Source/core/fetch/MultipartImageResourceParser.h
@@ -0,0 +1,93 @@
+// Copyright 2016 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.
+
+/* ***** BEGIN LICENSE BLOCK *****
+ * Version: MPL 1.1/GPL 2.0/LGPL 2.1
+ *
+ * The contents of this file are subject to the Mozilla Public License Version
+ * 1.1 (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ * http://www.mozilla.org/MPL/
+ *
+ * Software distributed under the License is distributed on an "AS IS" basis,
+ * WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License
+ * for the specific language governing rights and limitations under the
+ * License.
+ *
+ * The Original Code is mozilla.org code.
+ *
+ * The Initial Developer of the Original Code is
+ * Netscape Communications Corporation.
+ * Portions created by the Initial Developer are Copyright (C) 1998
+ * the Initial Developer. All Rights Reserved.
+ *
+ * Contributor(s):
+ *
+ * Alternatively, the contents of this file may be used under the terms of
+ * either the GNU General Public License Version 2 or later (the "GPL"), or
+ * the GNU Lesser General Public License Version 2.1 or later (the "LGPL"),
+ * in which case the provisions of the GPL or the LGPL are applicable instead
+ * of those above. If you wish to allow use of your version of this file only
+ * under the terms of either the GPL or the LGPL, and not to allow others to
+ * use your version of this file under the terms of the MPL, indicate your
+ * decision by deleting the provisions above and replace them with the notice
+ * and other provisions required by the GPL or the LGPL. If you do not delete
+ * the provisions above, a recipient may use your version of this file under
+ * the terms of any one of the MPL, the GPL or the LGPL.
+ *
+ * ***** END LICENSE BLOCK ***** */
+
+#ifndef MultipartImageResourceParser_h
+#define MultipartImageResourceParser_h
+
+#include "core/CoreExport.h"
+#include "platform/heap/Handle.h"
+#include "platform/network/ResourceResponse.h"
+#include "wtf/Vector.h"
+
+namespace blink {
+
+// A parser parsing mlutipart/x-mixed-replace resource.
+class CORE_EXPORT MultipartImageResourceParser : public GarbageCollectedFinalized<MultipartImageResourceParser> {
+public:
+    class CORE_EXPORT Client : public WillBeGarbageCollectedMixin {
+    public:
+        virtual ~Client() {}
+        virtual void didReceiveResponse(const ResourceResponse&) = 0;
+        virtual void didReceiveData(const char* bytes, size_t) = 0;

[hiroshige, 2016/02/25 18:39:08]

How about renaming these to:
  responseReceivedMultipart()
  appendDataMultipart()
because ThreadableLoaderClient has the same names.

ThreadableLoaderClient::didReceiveResponse() is for all response, while
Resource::didReceiveResponse() here is only for multipart responses and doesn't
correspond to ThreadableLoaderClient::didReceiveResponse(), which is confusing
for me.

Also, these methods are more related to Resource::appendData() and
Resource::responseReceived(), not ThreadableLoaderClient::didReceiveData() and
ThreadableLoaderClient::didReceiveResponse().

[yhirano, 2016/02/26 22:21:51]

I think appendData is not appropriate for a "client" class. How about
onePartInMultipartReceived / multipartDataReceived?

+        DEFINE_INLINE_VIRTUAL_TRACE() {}
+    };
+
+    MultipartImageResourceParser(const ResourceResponse&, const Vector<char>& boundary, Client*);
+    void addData(const char* bytes, size_t);

[hiroshige, 2016/02/25 18:39:08]

How about also renaming addData() to appendData(), because this is related to
Resource::appendData().

[yhirano, 2016/02/26 22:21:50]

Done.

+    void finish();
+    void cancel() { m_isCancelled = true; }
+
+    DECLARE_TRACE();
+
+    static int pushOverLineForTest(const Vector<char>& data, size_t size) { return pushOverLine(data, size); }
+    static size_t findBoundaryForTest(const Vector<char>& data, Vector<char>* boundary) { return findBoundary(data, boundary); }
+
+private:
+    bool parseHeaders();
+    bool isCancelled() const { return m_isCancelled; }
+    static int pushOverLine(const Vector<char>&, size_t);
+    // This function updates |*boundary|.
+    static size_t findBoundary(const Vector<char>& data, Vector<char>* boundary);
+
+    const ResourceResponse m_originalResponse;
+    Vector<char> m_boundary;
+    RawPtrWillBeMember<Client> m_client;
+
+    Vector<char> m_data;
+    bool m_isParsingTop = true;
+    bool m_isParsingHeaders = false;
+    bool m_sawLastBoundary = false;
+    bool m_isFirstPart = true;
+    bool m_isCancelled = false;
+};
+
+} // namespace blink
+
+#endif // MultipartImageResourceParser_h