Remote Assistance on Chrome OS Part II - Native Messaging renaming

chrome/browser/extensions/api/messaging/native_messaging_channel.h

+// Copyright 2014 The Chromium Authors. All rights reserved.

[Sergey Ulanov, 2014/09/15 19:53:39]

I think it's better to add this file under extensions/browser/api/messaging (and
ideally all native messaging code should be moved there)

Otherwise we would need to have circular dependencies between src/chrome and
src/remoting: chrome will depend on remoting (for the It2me host) and remoting
will depend on chrome for the interface.

[kelvinp, 2014/09/16 00:32:51]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_API_MESSAGING_NATIVE_MESSAGING_CHANNEL_H_
+#define CHROME_BROWSER_EXTENSIONS_API_MESSAGING_NATIVE_MESSAGING_CHANNEL_H_
+
+#include "base/callback.h"
+
+namespace base {
+class DictionaryValue;
+}  // namespace base
+
+namespace extensions {
+
+// Defines an interface to read messages and send responses between a native
+// component and chrome.
+class NativeMessagingChannel {
+ public:
+  // Used to send a message to the other endpoint.
+  typedef base::Callback<void(scoped_ptr<base::DictionaryValue> message)>

[Sergey Ulanov, 2014/09/15 19:53:39]

this should be base::Value instead of DictionaryValue. In remoting we send only
dictionaries, but NM allows sending any JSON values, not only dictionaries.

[kelvinp, 2014/09/16 00:32:51]

Done.

+      SendMessageCallback;
+
+  virtual ~NativeMessagingChannel() {}
+
+  // Starts reading and processing messages.
+  // |received_message| is called whenever a message is received from the other
+  // endpoint.
+  // |quit_closure| is called by the channel to shut down the native component
+  // whenever it encounter failures in sending or receiving a message.
+  virtual void Start(const SendMessageCallback& received_message,

[Sergey Ulanov, 2014/09/15 19:53:39]

I suggest you replace these two callbacks with a callback interface.

[kelvinp, 2014/09/16 00:32:51]

Done.

+                     const base::Closure& quit_closure) = 0;
+
+  // Sends a message to the other endpoint.
+  virtual void SendMessage(scoped_ptr<base::DictionaryValue> message) = 0;

[Sergey Ulanov, 2014/09/15 19:53:39]

base::Value

[kelvinp, 2014/09/16 00:32:51]

Done.

+};
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_EXTENSIONS_API_MESSAGING_NATIVE_MESSAGING_CHANNEL_H_