Making ResourceRequestBody part of //content's public API.

content/public/common/resource_request_body.h

+// 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.
+
+#ifndef CONTENT_PUBLIC_COMMON_RESOURCE_REQUEST_BODY_H_
+#define CONTENT_PUBLIC_COMMON_RESOURCE_REQUEST_BODY_H_
+
+#include <stdint.h>
+
+#include <vector>
+
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+// ResourceRequestBody represents body (i.e. upload data) of a HTTP request.

[Charlie Reis, 2016/06/07 22:21:01]

nit: the body
Maybe also mention that this is intentionally opaque and will be used via
OpenURL?

[Łukasz Anforowicz, 2016/06/08 16:51:56]

Done?  You're right - we should explicitly document that opaqueness is
intentional.  OTOH, I am never sure how much comments is too much (i.e. at what
point the benefits of the comment outweight concerns that the comments will at
some point become stale) - let me know if you think I went a little bit
overboard.

[Charlie Reis, 2016/06/08 22:43:19]

Looks great-- those are useful to know for understanding why this is in the
public interface.

+class CONTENT_EXPORT ResourceRequestBody
+    : public base::RefCountedThreadSafe<ResourceRequestBody> {
+ protected:
+  ResourceRequestBody();
+  virtual ~ResourceRequestBody();
+
+ private:
+  friend class base::RefCountedThreadSafe<ResourceRequestBody>;
+  DISALLOW_COPY_AND_ASSIGN(ResourceRequestBody);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_COMMON_RESOURCE_REQUEST_BODY_H_