Move mojo::internal::MessageBuilder out of the |internal| namespace.

mojo/public/cpp/bindings/lib/message_builder.h

Index: mojo/public/cpp/bindings/lib/message_builder.h
diff --git a/mojo/public/cpp/bindings/lib/message_builder.h b/mojo/public/cpp/bindings/lib/message_builder.h
index 86ae71db142431980d02dfef94759742dd1abaf4..535f8e47386a301c7c306f01f3bb54865ababb2b 100644
--- a/mojo/public/cpp/bindings/lib/message_builder.h
+++ b/mojo/public/cpp/bindings/lib/message_builder.h
@@ -14,14 +14,14 @@
 namespace mojo {
 class Message;
 
-namespace internal {
-
 class MessageBuilder {

[viettrungluu, 2015/11/17 23:59:38]

Are there specific tests for MessageBuilder, MessageWithRequestIDBuilder, etc.?
(If not, you should probably add some. :P)

[vardhan, 2015/11/18 22:46:40]

Done.

  public:
   MessageBuilder(uint32_t name, size_t payload_size);
   ~MessageBuilder();
 
-  Buffer* buffer() { return &buf_; }
+  // TODO(vardhan): |buffer()| is internal and only consumed by internal classes

[viettrungluu, 2015/11/17 23:59:38]

Maybe, in light of this comment, you should put this after |message()|?

[vardhan, 2015/11/18 22:46:40]

Done.

+  // and unittests.  Consider making it private + friend class its consumers?
+  internal::Buffer* buffer() { return &buf_; }
   Message* message() { return &message_; }
 
  protected:
@@ -29,7 +29,7 @@ class MessageBuilder {
   void Initialize(size_t size);
 
   Message message_;
-  FixedBuffer buf_;
+  internal::FixedBuffer buf_;
 
   MOJO_DISALLOW_COPY_AND_ASSIGN(MessageBuilder);
 };
@@ -47,7 +47,7 @@ class RequestMessageBuilder : public MessageWithRequestIDBuilder {
   RequestMessageBuilder(uint32_t name, size_t payload_size)
       : MessageWithRequestIDBuilder(name,
                                     payload_size,
-                                    kMessageExpectsResponse,
+                                    internal::kMessageExpectsResponse,
                                     0) {}
 };
 
@@ -58,11 +58,10 @@ class ResponseMessageBuilder : public MessageWithRequestIDBuilder {
                          uint64_t request_id)
       : MessageWithRequestIDBuilder(name,
                                     payload_size,
-                                    kMessageIsResponse,
+                                    internal::kMessageIsResponse,
                                     request_id) {}
 };
 
-}  // namespace internal
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_BINDINGS_LIB_MESSAGE_BUILDER_H_