Optionally have MessagePort pass data as base::Value, part 1.

content/public/common/message_port_message.h

Index: content/public/common/message_port_message.h
diff --git a/content/public/common/message_port_message.h b/content/public/common/message_port_message.h
new file mode 100644
index 0000000000000000000000000000000000000000..f0d76ae792ae6a9d7e5b4e651c993acf55abbe76
--- /dev/null
+++ b/content/public/common/message_port_message.h
@@ -0,0 +1,36 @@
+// Copyright 2015 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_MESSAGE_PORT_MESSAGE_H_
+#define CONTENT_PUBLIC_COMMON_MESSAGE_PORT_MESSAGE_H_
+
+#include <vector>
+
+#include "base/strings/string16.h"
+#include "base/values.h"
+
+namespace content {
+
+// Struct representing a message sent across a message port. This struct hides
+// the fact that messages can be serialized/encoded in multiple ways from code
+// that doesn't care about the message contents.
+struct MessagePortMessage {
+  MessagePortMessage();
+  explicit MessagePortMessage(const base::string16& message);
+  explicit MessagePortMessage(const base::Value* message);

[scheib, 2015/02/18 21:03:23]

Perhaps just take ownership of the message object instead of DeepCopy. 

[Marijn Kruisselbrink, 2015/02/26 23:12:54]

Done.

+  MessagePortMessage(const MessagePortMessage& other);
+  MessagePortMessage& operator=(const MessagePortMessage& other);
+  ~MessagePortMessage();
+
+  // Message serialized using blink::WebSerializedScriptValue. Only one of
+  // |message_as_string| and |message_as_value| should be non-empty.
+  base::string16 message_as_string;
+  // Message as a base::Value. This is either an empty list or a list
+  // containing exactly one item.
+  base::ListValue message_as_value;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_COMMON_MESSAGE_PORT_MESSAGE_H_