Implement a parser for Content-Disposition header fields

third_party/WebKit/Source/platform/network/ContentHeaderFieldInput.h

+// Copyright 2017 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.
+
+#ifndef ContentHeaderFieldInput_h
+#define ContentHeaderFieldInput_h
+
+#include "platform/network/ParsedContentHeaderFieldParameters.h"
+#include "platform/wtf/text/WTFString.h"
+
+namespace blink {
+
+class ContentHeaderFieldInput final {

[kinuko, 2017/05/22 07:39:44]

Please add a class-level comment.

It'd be probably helpful each of these related classes have a lightweight
comment about the relationship

[yhirano, 2017/05/22 07:51:20]

Please write a class comment.

[e_hakkinen, 2017/05/22 08:11:58]

Done.

+  STACK_ALLOCATED();
+
+ public:
+  using Mode = ParsedContentHeaderFieldParameters::Mode;
+
+  explicit ContentHeaderFieldInput(const String& header_field);
+  ContentHeaderFieldInput(ContentHeaderFieldInput&&);
+
+  bool Consume(char);

[yhirano, 2017/05/22 07:51:20]

It would be good to explain what the returned boolean means. Ditto below.

[e_hakkinen, 2017/05/22 08:11:58]

Done.

+  bool ConsumeToken(Mode, StringView& output);
+  bool ConsumeTokenOrQuotedString(Mode, String& output);
+  unsigned Index() const { return index_; }
+  bool IsConsumed() const { return index_ >= input_.length(); }
+  void SkipSpaces();
+
+ private:
+  bool ConsumeQuotedString(String& output);
+
+  unsigned index_;
+  const String input_;
+};
+
+}  // namespace blink
+
+#endif