Roll third_party/inspector_protocol to 4ad35c45aca9834b67ec2cb152c816ea1b7ceb48

third_party/inspector_protocol/lib/FrontendChannel_h.template

 // 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 {{"_".join(config.protocol.namespace)}}_FrontendChannel_h
 #define {{"_".join(config.protocol.namespace)}}_FrontendChannel_h
 
 {% for namespace in config.protocol.namespace %}
 namespace {{namespace}} {
 {% endfor %}
 
+class {{config.lib.export_macro}} Serializable {
+public:
+        virtual std::unique_ptr<protocol::Value> serializeValue() const = 0;

[dgozman, 2016/11/21 22:29:16]

8 spaces?

[dgozman, 2016/11/21 22:29:16]

Why return value and not json string?
I'd call this toJSONString actually, and make values implement the interface.

[kozy, 2016/11/22 01:25:38]

Done.

[kozy, 2016/11/22 01:25:38]

Done.

+        virtual ~Serializable() = default;
+};
+
 class {{config.lib.export_macro}} FrontendChannel {
 public:
     virtual ~FrontendChannel() { }
-    virtual void sendProtocolResponse(int callId, const String& message) = 0;
-    virtual void sendProtocolNotification(const String& message) = 0;
+    virtual void sendProtocolResponse(int callId, std::unique_ptr<Serializable> message) = 0;
+    virtual void sendProtocolNotification(std::unique_ptr<Serializable> message) = 0;
     virtual void flushProtocolNotifications() = 0;
 };
 
 {% for namespace in config.protocol.namespace %}
 } // namespace {{namespace}}
 {% endfor %}
 
 #endif // !defined({{"_".join(config.protocol.namespace)}}_FrontendChannel_h)