PPAPI: Implement synchronous postMessage

ppapi/proxy/message_handler.h

Index: ppapi/proxy/message_handler.h
diff --git a/ppapi/proxy/message_handler.h b/ppapi/proxy/message_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..c3e00bbbdf24224a5810423b585e98ed665e448b
--- /dev/null
+++ b/ppapi/proxy/message_handler.h
@@ -0,0 +1,73 @@
+// Copyright 2014 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 PPAPI_PROXY_MESSAGE_HANDLER_H_
+#define PPAPI_PROXY_MESSAGE_HANDLER_H_
+
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "ppapi/c/pp_resource.h"
+#include "ppapi/c/ppp_message_handler.h"
+#include "ppapi/proxy/ppapi_proxy_export.h"
+
+namespace IPC {
+class Message;
+}
+
+namespace ppapi {
+
+class ScopedPPVar;
+
+namespace proxy {
+
+class MessageLoopResource;
+
+// MessageHandler wraps a PPP_MessageHandler to encapsulate calling methods
+// on the right thread and calling the Destroy call when this MessageHandler is

[raymes, 2014/06/06 06:38:17]

Destroy call-> Destroy function?

[dmichael (off chromium), 2014/06/13 22:01:26]

Done.

+// destroyed.
+class PPAPI_PROXY_EXPORT MessageHandler {
+ public:
+  // Create a MessageHandler. If any parameters are invalid, it will return a
+  // null scoped_ptr and set |*error| appropriately.
+  //  |handler_if| is the struct of function pointers we will invoke. All of
+  //               the function pointers within must be valid, or we fail
+  //               with PP_ERROR_BADARGUMENT.
+  //  |user_data| is a pointer provided by the plugin that we pass back when we
+  //              call functions in |handler_if|.
+  //  |message_loop| is the message loop where we will invoke functions in
+  //                 |handler_if|. Must not be the main thread message loop,
+  //                 to try to force the plugin to not over-subscribe the main
+  //                 thread. If it's the main thread loop, |error| will be set
+  //                 to PP_ERROR_WRONGTHREAD.
+  //  |error| is an out-param that will be set on failure.

[raymes, 2014/06/06 06:38:17]

nit: This indentation style looks a little odd (haven't seen it much). Seems
fine though. I would suggest maybe removing the single space indent of the
parameters.

[dmichael (off chromium), 2014/06/13 22:01:26]

Done.

+  static scoped_ptr<MessageHandler> Create(
+      PP_Instance instance,
+      const PPP_MessageHandler_0_1* handler_if,
+      void* user_data,
+      PP_Resource message_loop,
+      int32_t* error);
+  ~MessageHandler();
+
+  void HandleMessage(ScopedPPVar var);
+  void HandleBlockingMessage(ScopedPPVar var,
+                             scoped_ptr<IPC::Message> reply_msg);
+
+ private:
+  MessageHandler(PP_Instance instance,
+                 const PPP_MessageHandler_0_1* handler_if,
+                 void* user_data,
+                 scoped_refptr<MessageLoopResource> message_loop);
+
+  PP_Instance instance_;
+  const PPP_MessageHandler_0_1* handler_if_;
+  void* user_data_;
+  scoped_refptr<MessageLoopResource> message_loop_;
+
+  DISALLOW_COPY_AND_ASSIGN(MessageHandler);
+};
+
+}  // namespace proxy
+}  // namespace ppapi
+
+#endif  // PPAPI_PROXY_MESSAGE_HANDLER_H_